diff options
| author | Ronnie Sahlberg <ronnie_sahlberg@ozemail.com.au> | 2006-06-22 11:23:26 +0000 |
|---|---|---|
| committer | Ronnie Sahlberg <ronnie_sahlberg@ozemail.com.au> | 2006-06-22 11:23:26 +0000 |
| commit | 6fe839a61a6e8bfc2d18a0f25c0ab34cd8bedbc9 (patch) | |
| tree | 5d7ab8b13812dfe34dd4765bf8584b19a74feccd /gtk/proto_draw.c | |
| parent | 6ceb4d9b46a74a1c5d632c70a5e4493bc8279997 (diff) | |
coverity 187
i dont think this can trigger in the current code but better safe than sorry and it safeguards against future changes that might break this requirement.
dereferencing a nullpointer would cause crash of wireshark in case a new bug is introduced making it possible for this variuable to be null.
svn path=/trunk/; revision=18543
Diffstat (limited to 'gtk/proto_draw.c')
| -rw-r--r-- | gtk/proto_draw.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/gtk/proto_draw.c b/gtk/proto_draw.c index 8511daa4db..636bf957a6 100644 --- a/gtk/proto_draw.c +++ b/gtk/proto_draw.c @@ -824,6 +824,7 @@ copy_hex_cb(GtkWidget * w _U_, gpointer data _U_, int data_type) } data_p = get_byte_view_data_and_length(GTK_WIDGET(bv), &len); + g_assert(data_p != NULL); g_string_sprintfa(byte_str,"%04x ",i); /* Offset 0000 */ for (i=0; i<len; i++){ |