Fix various instances of "unreachable code".
authorwmeier <wmeier@f5534014-38df-0310-8fa8-9805f1628bb7>
Sun, 30 Jan 2011 23:27:57 +0000 (23:27 +0000)
committerwmeier <wmeier@f5534014-38df-0310-8fa8-9805f1628bb7>
Sun, 30 Jan 2011 23:27:57 +0000 (23:27 +0000)
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@35713 f5534014-38df-0310-8fa8-9805f1628bb7

epan/dissectors/packet-rtps2.c
epan/filesystem.c
epan/gcp.c
epan/tvbuff.c
gtk/hostlist_table.c
gtk/packet_list_store.c
wiretap/buffer.c
wiretap/file_access.c

index 865112ce2a40258a6b818bac5e317bba47551224..115a70766d2b15771a166e01e91238e630c4f6a1 100644 (file)
@@ -5511,7 +5511,7 @@ static gint dissect_parameter_sequence(proto_tree *tree,
     offset += param_length;
 
   } /* for all the parameters */
-  return offset;
+  g_assert_not_reached();
 }
 #undef ENSURE_LENGTH
 
index a662c796c4b3edbcb9af1cd83a7c9535594085e8..10a849edfcc54c6c82f71459685991097f8d8cff 100644 (file)
@@ -119,8 +119,8 @@ find_last_pathname_separator(const char *path)
        return strchr(path, ':');
 #else
        separator = strrchr(path, '/');
-#endif
        return separator;
+#endif
 }
 
 /*
index 928638c960e2cd1fb572b6c4a4df5205c993bb2f..f0d8e8a366fb57d1400c4548cfa3360d92a84bec 100644 (file)
@@ -511,7 +511,6 @@ gcp_term_t* gcp_cmd_add_term(gcp_msg_t* m, gcp_trx_t* tr, gcp_cmd_t* c, gcp_term
             }
 
             DISSECTOR_ASSERT_NOT_REACHED();
-            return NULL;
         }
     } else {
         ct = ep_new(gcp_terms_t);
index 52cecb481c911d432cf1f36f73f9b6cd4aae4548..32f8225868d807df1b31a890768d6f9447fbd4a1 100644 (file)
@@ -873,7 +873,6 @@ composite_ensure_contiguous_no_exception(tvbuff_t *tvb, const guint abs_offset,
        }
 
        DISSECTOR_ASSERT_NOT_REACHED();
-       return NULL;
 }
 
 static const guint8*
@@ -1035,7 +1034,6 @@ composite_memcpy(tvbuff_t *tvb, guint8* target, guint abs_offset, size_t abs_len
        }
 
        DISSECTOR_ASSERT_NOT_REACHED();
-       return NULL;
 }
 
 void*
index 21b301eb3090f765389639e0ff1e1e06a606ea70..da7376eb11471c3fb56d7262b27b6e884aac7445 100644 (file)
@@ -337,7 +337,6 @@ hostlist_sort_column(GtkTreeModel *model,
 #endif
     }
     g_assert_not_reached();
-    return 0;
 }
 
 static void
index cc6f7703bc317458f1c167544ba7dce2e6483b66..3eccc9e1d16a565d8eed086d86072f562127f85d 100644 (file)
@@ -970,7 +970,7 @@ packet_list_compare_records(gint sort_id, PacketListRecord *a,
        } else
                return (a->fdata->col_text[sort_id] == NULL) ? -1 : 1;
 
-       g_return_val_if_reached(0);
+       g_assert_not_reached();
 }
 
 static gint
index f7f977c06f4679eece8ac4d679bfa382499c1128..eae751cbd7bbb3fc9eda942cc449362b900877e7 100644 (file)
@@ -100,7 +100,7 @@ void buffer_remove_start(Buffer* buffer, gsize bytes)
                g_error("buffer_remove_start trying to remove %" G_GINT64_MODIFIER "u bytes. s=%" G_GINT64_MODIFIER "u ff=%" G_GINT64_MODIFIER "u!\n",
                        (guint64)bytes, (guint64)buffer->start,
                        (guint64)buffer->first_free);
-               exit(1);
+               /** g_error() does an abort() and thus never returns **/
        }
        buffer->start += bytes;
 
index aea8d60ed3f6a6d7c15b2e63162f6894203d252a..b51015579edfe583d892c051091628a142ba2331 100644 (file)
@@ -669,7 +669,7 @@ const char *wtap_file_type_string(int filetype)
 {
        if (filetype < 0 || filetype >= wtap_num_file_types) {
                g_error("Unknown capture file type %d", filetype);
-               return NULL;
+               /** g_error() does an abort() and thus never returns **/
        } else
                return dump_open_table[filetype].name;
 }