diff options
Diffstat (limited to 'epan')
30 files changed, 104 insertions, 170 deletions
diff --git a/epan/addr_resolv.c b/epan/addr_resolv.c index 590c496ea9..ea32c6f2ba 100644 --- a/epan/addr_resolv.c +++ b/epan/addr_resolv.c @@ -1961,8 +1961,7 @@ read_hosts_file (const char *hostspath) add_ipv4_name(host_addr[0], cp); } } - if (line != NULL) - g_free(line); + g_free(line); fclose(hf); return TRUE; @@ -2028,8 +2027,7 @@ read_subnets_file (const char *subnetspath) subnet_entry_set(host_addr, (guint32)mask_length, cp); } - if (line != NULL) - g_free(line); + g_free(line); fclose(hf); return TRUE; diff --git a/epan/dfilter/dfilter-macro.c b/epan/dfilter/dfilter-macro.c index a6601c4ce4..531aff9111 100644 --- a/epan/dfilter/dfilter-macro.c +++ b/epan/dfilter/dfilter-macro.c @@ -61,7 +61,7 @@ void dump_dfilter_macro_t(const dfilter_macro_t *m, const char *function, const static gboolean free_value(gpointer k _U_, gpointer v, gpointer u _U_) { fvt_cache_entry_t* e = v; - if (e->repr) g_free(e->repr); + g_free(e->repr); g_free(e); return TRUE; } @@ -465,12 +465,10 @@ static void macro_update(void* mp, const gchar** error) { done: g_ptr_array_add(parts,NULL); - if (m->parts) g_free(m->parts); - + g_free(m->parts); m->parts = (gchar**)parts->pdata; - if (m->args_pos) g_free(m->args_pos); - + g_free(m->args_pos); m->args_pos = (int*)(void *)args_pos->data; g_ptr_array_free(parts,FALSE); diff --git a/epan/dfilter/dfilter.c b/epan/dfilter/dfilter.c index 13e489110c..6bc1bcd12d 100644 --- a/epan/dfilter/dfilter.c +++ b/epan/dfilter/dfilter.c @@ -150,9 +150,7 @@ dfilter_free(dfilter_t *df) free_insns(df->consts); } - if (df->interesting_fields) { - g_free(df->interesting_fields); - } + g_free(df->interesting_fields); /* clear registers */ for (i = 0; i < df->max_registers; i++) { diff --git a/epan/dissectors/packet-dcerpc.c b/epan/dissectors/packet-dcerpc.c index 3c43b37719..6fee1c1d06 100644 --- a/epan/dissectors/packet-dcerpc.c +++ b/epan/dissectors/packet-dcerpc.c @@ -1679,9 +1679,7 @@ init_ndr_pointer_list(packet_info *pinfo) npd=g_slist_nth_data(ndr_pointer_list, 0); ndr_pointer_list=g_slist_remove(ndr_pointer_list, npd); - if(npd){ - g_free(npd); - } + g_free(npd); } ndr_pointer_list=NULL; diff --git a/epan/dissectors/packet-gsm_sms.c b/epan/dissectors/packet-gsm_sms.c index cad7db2ca0..1521ee7d15 100644 --- a/epan/dissectors/packet-gsm_sms.c +++ b/epan/dissectors/packet-gsm_sms.c @@ -2747,8 +2747,7 @@ dis_field_ud(tvbuff_t *tvb, proto_tree *tree, guint32 offset, guint32 length, gb } } - if(utf8_text) - g_free(utf8_text); + g_free(utf8_text); g_iconv_close(cd); } else diff --git a/epan/dissectors/packet-ieee80211.c b/epan/dissectors/packet-ieee80211.c index 3355d6b433..1afb415c2f 100644 --- a/epan/dissectors/packet-ieee80211.c +++ b/epan/dissectors/packet-ieee80211.c @@ -12171,7 +12171,8 @@ static tvbuff_t *try_decrypt_wep(tvbuff_t *tvb, guint32 offset, guint32 len) { } } - if ((!decr_tvb) && (tmp)) g_free(tmp); + if (!decr_tvb) + g_free(tmp); #if 0 printf("de-wep %p\n", decr_tvb); @@ -12275,7 +12276,7 @@ void set_airpdcap_keys(void) } } } - if(tmpk != NULL) g_free(tmpk); + g_free(tmpk); } /* Now set the keys */ @@ -12384,9 +12385,7 @@ static void init_wepkeys(void) { g_free(wep_keys[i]); g_free(wep_keys); } - - if (wep_keylens) - g_free(wep_keylens); + g_free(wep_keylens); #ifdef USE_ENV guint8 *buf; @@ -12437,9 +12436,7 @@ static void init_wepkeys(void) { #endif #endif - if (wep_keys[keyidx]) { - g_free(wep_keys[keyidx]); - } + g_free(wep_keys[keyidx]); res = wep_str_to_bytes(tmp, bytes); if (tmp && res && bytes->len > 0) { diff --git a/epan/dissectors/packet-k12.c b/epan/dissectors/packet-k12.c index 58982518a1..ca53bd2c0e 100644 --- a/epan/dissectors/packet-k12.c +++ b/epan/dissectors/packet-k12.c @@ -318,8 +318,7 @@ k12_update_cb(void* r, const char** err) for (num_protos = 0; protos[num_protos]; num_protos++) g_strstrip(protos[num_protos]); - if (h->handles) g_free(h->handles); - + g_free(h->handles); h->handles = g_malloc0(sizeof(dissector_handle_t)*(num_protos < 2 ? 2 : num_protos)); for (i = 0; i < num_protos; i++) { @@ -355,9 +354,10 @@ static void k12_free_cb(void* r) { k12_handles_t* h = r; - if (h->match) g_free(h->match); - if (h->protos) g_free(h->protos); - if (h->handles) g_free(h->handles); + + g_free(h->match); + g_free(h->protos); + g_free(h->handles); } diff --git a/epan/dissectors/packet-kerberos.c b/epan/dissectors/packet-kerberos.c index 06f3d89454..42b9942963 100644 --- a/epan/dissectors/packet-kerberos.c +++ b/epan/dissectors/packet-kerberos.c @@ -533,9 +533,7 @@ decrypt_krb5_data(proto_tree *tree, packet_info *pinfo, input.ciphertext.data = (guint8 *)cryptotext; data.length = length; - if(data.data){ - g_free(data.data); - } + g_free(data.data); data.data = g_malloc(length); key.key.enctype=ek->keytype; @@ -748,8 +746,10 @@ clear_keytab(void) { for(ske = service_key_list; ske != NULL; ske = g_slist_next(ske)){ sk = (service_key_t *) ske->data; - if (sk && sk->contents) g_free(sk->contents); - if (sk) g_free(sk); + if (sk) { + g_free(sk->contents); + g_free(sk); + } } g_slist_free(service_key_list); service_key_list = NULL; diff --git a/epan/dissectors/packet-multipart.c b/epan/dissectors/packet-multipart.c index e10af37a0e..b41bfdd594 100644 --- a/epan/dissectors/packet-multipart.c +++ b/epan/dissectors/packet-multipart.c @@ -435,8 +435,7 @@ cleanup_multipart_info(void *data) { multipart_info_t *m_info = data; if (m_info) { - if (m_info->boundary) - g_free(m_info->boundary); + g_free(m_info->boundary); g_free(m_info); } } @@ -792,10 +791,8 @@ process_body_part(proto_tree *tree, tvbuff_t *tvb, const guint8 *boundary, } } - if(filename) - g_free(filename); - if(typename) - g_free(typename); + g_free(filename); + g_free(typename); return boundary_start + boundary_line_len; } diff --git a/epan/dissectors/packet-rtps.c b/epan/dissectors/packet-rtps.c index 2ff4da4a1e..141e6a3dd4 100644 --- a/epan/dissectors/packet-rtps.c +++ b/epan/dissectors/packet-rtps.c @@ -1577,9 +1577,8 @@ static gint rtps_util_add_string(proto_tree *tree, /* Can be NULL */ g_snprintf(buffer, buffer_size, "%s", retVal); } } - if (retVal != NULL) { - g_free(retVal); - } + g_free(retVal); + /* NDDS align strings at 4-bytes word. So: * string_length: 4 -> buffer_length = 4; * string_length: 5 -> buffer_length = 8; diff --git a/epan/dissectors/packet-rtps2.c b/epan/dissectors/packet-rtps2.c index f64126a2f3..5e2eec8155 100644 --- a/epan/dissectors/packet-rtps2.c +++ b/epan/dissectors/packet-rtps2.c @@ -1795,9 +1795,8 @@ static gint rtps_util_add_string(proto_tree *tree, g_snprintf(buffer, buffer_size, "%s", retVal); } } - if (retVal != NULL) { - g_free(retVal); - } + g_free(retVal); + /* NDDS align strings at 4-bytes word. So: * string_length: 4 -> buffer_length = 4; * string_length: 5 -> buffer_length = 8; diff --git a/epan/dissectors/packet-snmp.c b/epan/dissectors/packet-snmp.c index d56019f9ae..864afef329 100644 --- a/epan/dissectors/packet-snmp.c +++ b/epan/dissectors/packet-snmp.c @@ -3005,12 +3005,12 @@ static void* snmp_users_copy_cb(void* dest, const void* orig, unsigned len _U_) static void snmp_users_free_cb(void* p) { snmp_ue_assoc_t* ue = p; - if (ue->user.userName.data) g_free(ue->user.userName.data); - if (ue->user.authPassword.data) g_free(ue->user.authPassword.data); - if (ue->user.privPassword.data) g_free(ue->user.privPassword.data); - if (ue->user.authKey.data) g_free(ue->user.authKey.data); - if (ue->user.privKey.data) g_free(ue->user.privKey.data); - if (ue->engine.data) g_free(ue->engine.data); + g_free(ue->user.userName.data); + g_free(ue->user.authPassword.data); + g_free(ue->user.privPassword.data); + g_free(ue->user.authKey.data); + g_free(ue->user.privKey.data); + g_free(ue->engine.data); } static void snmp_users_update_cb(void* p _U_, const char** err) { diff --git a/epan/dissectors/packet-ssl-utils.c b/epan/dissectors/packet-ssl-utils.c index ad6fd045b9..be6c40707f 100644 --- a/epan/dissectors/packet-ssl-utils.c +++ b/epan/dissectors/packet-ssl-utils.c @@ -2697,12 +2697,10 @@ ssl_common_init(GHashTable **session_hash, StringInfo *decrypted_data, StringInf g_hash_table_destroy(*session_hash); *session_hash = g_hash_table_new(ssl_hash, ssl_equal); - if (decrypted_data->data) - g_free(decrypted_data->data); + g_free(decrypted_data->data); ssl_data_alloc(decrypted_data, 32); - if (compressed_data->data) - g_free(compressed_data->data); + g_free(compressed_data->data); ssl_data_alloc(compressed_data, 32); } diff --git a/epan/dissectors/packet-user_encap.c b/epan/dissectors/packet-user_encap.c index 26d54ebb82..7859883f5a 100644 --- a/epan/dissectors/packet-user_encap.c +++ b/epan/dissectors/packet-user_encap.c @@ -142,9 +142,9 @@ static void user_free_cb(void* record) { user_encap_t *u = record; - if (u->payload_proto_name) g_free(u->payload_proto_name); - if (u->header_proto_name) g_free(u->header_proto_name); - if (u->trailer_proto_name) g_free(u->trailer_proto_name); + g_free(u->payload_proto_name); + g_free(u->header_proto_name); + g_free(u->trailer_proto_name); } UAT_VS_DEF(user_encap, encap, user_encap_t, WTAP_ENCAP_USER0, ENCAP0_STR) diff --git a/epan/dissectors/packet-xml.c b/epan/dissectors/packet-xml.c index cc09fc0a02..a2d6152b60 100644 --- a/epan/dissectors/packet-xml.c +++ b/epan/dissectors/packet-xml.c @@ -793,15 +793,13 @@ static void add_xmlpi_namespace(gpointer k _U_, gpointer v, gpointer p) { } static void destroy_dtd_data(dtd_build_data_t* dtd_data) { - - if(dtd_data->proto_name) g_free(dtd_data->proto_name); - if(dtd_data->media_type) g_free(dtd_data->media_type); - if(dtd_data->description) g_free(dtd_data->description); - if(dtd_data->proto_root) g_free(dtd_data->proto_root); + g_free(dtd_data->proto_name); + g_free(dtd_data->media_type); + g_free(dtd_data->description); + g_free(dtd_data->proto_root); g_string_free(dtd_data->error,TRUE); - while(dtd_data->elements->len) { dtd_named_list_t* nl = g_ptr_array_remove_index_fast(dtd_data->elements,0); g_ptr_array_free(nl->list,TRUE); @@ -819,10 +817,8 @@ static void destroy_dtd_data(dtd_build_data_t* dtd_data) { g_ptr_array_free(dtd_data->attributes,TRUE); g_free(dtd_data); - } - static void copy_attrib_item(gpointer k, gpointer v _U_, gpointer p) { gchar* key = g_strdup(k); int* value = g_malloc(sizeof(int)); @@ -1036,8 +1032,7 @@ next_attribute: /* if a proto_root is defined in the dtd we'll use that as root */ if( dtd_data->proto_root ) { - if(root_name) - g_free(root_name); + g_free(root_name); root_name = g_strdup(dtd_data->proto_root); } @@ -1186,9 +1181,7 @@ next_attribute: g_hash_table_destroy(elements); destroy_dtd_data(dtd_data); - - if (root_name) - g_free(root_name); + g_free(root_name); } # define DIRECTORY_T GDir diff --git a/epan/filesystem.c b/epan/filesystem.c index cf1a347720..5b522d6b76 100644 --- a/epan/filesystem.c +++ b/epan/filesystem.c @@ -812,9 +812,7 @@ char *getenv_utf8(const char *varname) void set_profile_name(const gchar *profilename) { - if (persconfprofile) { - g_free (persconfprofile); - } + g_free (persconfprofile); if (profilename && strlen(profilename) > 0 && strcmp(profilename, DEFAULT_PROFILE) != 0) { @@ -934,10 +932,7 @@ get_profiles_dir(void) { static char *profiles_dir = NULL; - if (profiles_dir) { - g_free (profiles_dir); - } - + g_free (profiles_dir); profiles_dir = g_strdup_printf ("%s%s%s", get_persconffile_dir_no_profile (), G_DIR_SEPARATOR_S, PROFILES_DIR); @@ -949,9 +944,7 @@ get_persconffile_dir(const gchar *profilename) { static char *persconffile_profile_dir = NULL; - if (persconffile_profile_dir) { - g_free (persconffile_profile_dir); - } + g_free (persconffile_profile_dir); if (profilename && strlen(profilename) > 0 && strcmp(profilename, DEFAULT_PROFILE) != 0) { diff --git a/epan/ftypes/ftype-pcre.c b/epan/ftypes/ftype-pcre.c index 7b9c717a09..01b01f1bb1 100644 --- a/epan/ftypes/ftype-pcre.c +++ b/epan/ftypes/ftype-pcre.c @@ -86,10 +86,10 @@ static void pcre_tuple_free(pcre_tuple_t *tuple) { if (tuple) { - if (tuple->string) g_free(tuple->string); - if (tuple->re) g_free(tuple->re); - if (tuple->ex) g_free(tuple->ex); - if (tuple->error) g_free(tuple->error); + g_free(tuple->string); + g_free(tuple->re); + g_free(tuple->ex); + g_free(tuple->error); g_free(tuple); } } diff --git a/epan/ftypes/ftype-string.c b/epan/ftypes/ftype-string.c index 4a41c55dcf..0e60bc2a2a 100644 --- a/epan/ftypes/ftype-string.c +++ b/epan/ftypes/ftype-string.c @@ -45,9 +45,7 @@ string_fvalue_new(fvalue_t *fv) static void string_fvalue_free(fvalue_t *fv) { - if (fv->value.string) { - g_free(fv->value.string); - } + g_free(fv->value.string); } static void diff --git a/epan/geoip_db.c b/epan/geoip_db.c index 8becf3e232..b7339e324b 100644 --- a/epan/geoip_db.c +++ b/epan/geoip_db.c @@ -118,7 +118,7 @@ static void* geoip_db_path_copy_cb(void* dest, const void* orig, unsigned len _U static void geoip_db_path_free_cb(void* p) { geoip_db_path_t *m = p; - if (m->path) g_free(m->path); + g_free(m->path); } /** diff --git a/epan/ipv4.c b/epan/ipv4.c index 5fe49a0e7e..0472a5be3d 100644 --- a/epan/ipv4.c +++ b/epan/ipv4.c @@ -49,8 +49,7 @@ ipv4_addr_new(void) void ipv4_addr_free(ipv4_addr *ipv4) { - if (ipv4) - g_free(ipv4); + g_free(ipv4); } void diff --git a/epan/oids.c b/epan/oids.c index fd9bc50360..e4c40096f2 100644 --- a/epan/oids.c +++ b/epan/oids.c @@ -243,7 +243,7 @@ static void* smi_mod_copy_cb(void* dest, const void* orig, unsigned len _U_) { static void smi_mod_free_cb(void* p) { smi_module_t* m = p; - if (m->name) g_free(m->name); + g_free(m->name); } diff --git a/epan/prefs.c b/epan/prefs.c index c7269a6708..e525a1ba98 100644 --- a/epan/prefs.c +++ b/epan/prefs.c @@ -1206,33 +1206,21 @@ void prefs_reset(void) { prefs_initialized = FALSE; - if (prefs.pr_file) - g_free(prefs.pr_file); - if (prefs.pr_cmd) - g_free(prefs.pr_cmd); + + g_free(prefs.pr_file); + g_free(prefs.pr_cmd); free_col_info(&prefs); - if (prefs.gui_font_name) - g_free(prefs.gui_font_name); - if (prefs.gui_colorized_fg) - g_free(prefs.gui_colorized_fg); - if (prefs.gui_colorized_bg) - g_free(prefs.gui_colorized_bg); - if (prefs.gui_fileopen_dir) - g_free(prefs.gui_fileopen_dir); - if (prefs.gui_webbrowser) - g_free(prefs.gui_webbrowser); - if (prefs.gui_window_title) - g_free(prefs.gui_window_title); - if (prefs.gui_start_title) - g_free(prefs.gui_start_title); - if (prefs.capture_device) - g_free(prefs.capture_device); - if (prefs.capture_devices_linktypes) - g_free(prefs.capture_devices_linktypes); - if (prefs.capture_devices_descr) - g_free(prefs.capture_devices_descr); - if (prefs.capture_devices_hide) - g_free(prefs.capture_devices_hide); + g_free(prefs.gui_font_name); + g_free(prefs.gui_colorized_fg); + g_free(prefs.gui_colorized_bg); + g_free(prefs.gui_fileopen_dir); + g_free(prefs.gui_webbrowser); + g_free(prefs.gui_window_title); + g_free(prefs.gui_start_title); + g_free(prefs.capture_device); + g_free(prefs.capture_devices_linktypes); + g_free(prefs.capture_devices_descr); + g_free(prefs.capture_devices_hide); uat_unload_all(); init_prefs(); @@ -1788,10 +1776,10 @@ set_pref(gchar *pref_name, gchar *value, void *private_data _U_) return PREFS_SET_SYNTAX_ERR; } } else if (strcmp(pref_name, PRS_PRINT_FILE) == 0) { - if (prefs.pr_file) g_free(prefs.pr_file); + g_free(prefs.pr_file); prefs.pr_file = g_strdup(value); } else if (strcmp(pref_name, PRS_PRINT_CMD) == 0) { - if (prefs.pr_cmd) g_free(prefs.pr_cmd); + g_free(prefs.pr_cmd); prefs.pr_cmd = g_strdup(value); } else if (strcmp(pref_name, PRS_COL_FMT) == 0) { col_l = prefs_get_string_list(value); @@ -1927,8 +1915,7 @@ set_pref(gchar *pref_name, gchar *value, void *private_data _U_) } else if (strcmp(pref_name, PRS_GUI_FONT_NAME_1) == 0) { /* GTK1 font name obsolete */ } else if (strcmp(pref_name, PRS_GUI_FONT_NAME_2) == 0) { - if (prefs.gui_font_name != NULL) - g_free(prefs.gui_font_name); + g_free(prefs.gui_font_name); prefs.gui_font_name = g_strdup(value); } else if (strcmp(pref_name, PRS_GUI_MARKED_FG) == 0) { cval = strtoul(value, NULL, 16); @@ -1943,12 +1930,10 @@ set_pref(gchar *pref_name, gchar *value, void *private_data _U_) prefs.gui_marked_bg.green = GREEN_COMPONENT(cval); prefs.gui_marked_bg.blue = BLUE_COMPONENT(cval); } else if (strcmp(pref_name, PRS_GUI_COLORIZED_FG) == 0) { - if (prefs.gui_colorized_fg != NULL) - g_free(prefs.gui_colorized_fg); + g_free(prefs.gui_colorized_fg); prefs.gui_colorized_fg = g_strdup(value); } else if (strcmp(pref_name, PRS_GUI_COLORIZED_BG) == 0) { - if (prefs.gui_colorized_bg != NULL) - g_free(prefs.gui_colorized_bg); + g_free(prefs.gui_colorized_bg); prefs.gui_colorized_bg = g_strdup(value); } else if (strcmp(pref_name, PRS_GUI_GEOMETRY_SAVE_POSITION) == 0) { if (g_ascii_strcasecmp(value, "true") == 0) { @@ -2003,8 +1988,7 @@ set_pref(gchar *pref_name, gchar *value, void *private_data _U_) find_index_from_string_array(value, gui_fileopen_style_text, FO_STYLE_LAST_OPENED); } else if (strcmp(pref_name, PRS_GUI_FILEOPEN_DIR) == 0) { - if (prefs.gui_fileopen_dir != NULL) - g_free(prefs.gui_fileopen_dir); + g_free(prefs.gui_fileopen_dir); prefs.gui_fileopen_dir = g_strdup(value); } else if (strcmp(pref_name, PRS_GUI_FILEOPEN_REMEMBERED_DIR) == 0) { /* deprecated */ } else if (strcmp(pref_name, PRS_GUI_FILEOPEN_PREVIEW) == 0) { @@ -2034,12 +2018,10 @@ set_pref(gchar *pref_name, gchar *value, void *private_data _U_) g_free(prefs.gui_webbrowser); prefs.gui_webbrowser = g_strdup(value); } else if (strcmp(pref_name, PRS_GUI_WINDOW_TITLE) == 0) { - if (prefs.gui_window_title != NULL) - g_free(prefs.gui_window_title); + g_free(prefs.gui_window_title); prefs.gui_window_title = g_strdup(value); } else if (strcmp(pref_name, PRS_GUI_START_TITLE) == 0) { - if (prefs.gui_start_title != NULL) - g_free(prefs.gui_start_title); + g_free(prefs.gui_start_title); prefs.gui_start_title = g_strdup(value); } else if (strcmp(pref_name, PRS_GUI_VERSION_IN_START_PAGE) == 0) { if (g_ascii_strcasecmp(value, "true") == 0) { @@ -2069,20 +2051,16 @@ set_pref(gchar *pref_name, gchar *value, void *private_data _U_) /* handle the capture options */ } else if (strcmp(pref_name, PRS_CAP_DEVICE) == 0) { - if (prefs.capture_device != NULL) - g_free(prefs.capture_device); + g_free(prefs.capture_device); prefs.capture_device = g_strdup(value); } else if (strcmp(pref_name, PRS_CAP_DEVICES_LINKTYPES) == 0) { - if (prefs.capture_devices_linktypes != NULL) - g_free(prefs.capture_devices_linktypes); + g_free(prefs.capture_devices_linktypes); prefs.capture_devices_linktypes = g_strdup(value); } else if (strcmp(pref_name, PRS_CAP_DEVICES_DESCR) == 0) { - if (prefs.capture_devices_descr != NULL) - g_free(prefs.capture_devices_descr); + g_free(prefs.capture_devices_descr); prefs.capture_devices_descr = g_strdup(value); } else if (strcmp(pref_name, PRS_CAP_DEVICES_HIDE) == 0) { - if (prefs.capture_devices_hide != NULL) - g_free(prefs.capture_devices_hide); + g_free(prefs.capture_devices_hide); prefs.capture_devices_hide = g_strdup(value); } else if (strcmp(pref_name, PRS_CAP_PROM_MODE) == 0) { prefs.capture_prom_mode = ((g_ascii_strcasecmp(value, "true") == 0)?TRUE:FALSE); @@ -3127,11 +3105,10 @@ free_col_info(e_prefs *pr) while (pr->col_list != NULL) { cfmt = pr->col_list->data; + g_free(cfmt->title); g_free(cfmt->fmt); - if (cfmt->custom_field) { - g_free(cfmt->custom_field); - } + g_free(cfmt->custom_field); g_free(cfmt); pr->col_list = g_list_remove_link(pr->col_list, pr->col_list); } diff --git a/epan/proto.c b/epan/proto.c index 6a0673e66a..fe81914cde 100644 --- a/epan/proto.c +++ b/epan/proto.c @@ -407,8 +407,7 @@ proto_cleanup(void) g_free(gpa_hfinfo.hfi); gpa_hfinfo.hfi=NULL; } - if (tree_is_expanded != NULL) - g_free(tree_is_expanded); + g_free(tree_is_expanded); } diff --git a/epan/reassemble.c b/epan/reassemble.c index e3d21e6a3f..7b75c21076 100644 --- a/epan/reassemble.c +++ b/epan/reassemble.c @@ -850,8 +850,7 @@ fragment_add_work(fragment_data *fd_head, tvbuff_t *tvb, int offset, } } - if( old_data ) - g_free(old_data); + g_free(old_data); /* mark this packet as defragmented. allows us to skip any trailing fragments */ fd_head->flags |= FD_DEFRAGMENTED; @@ -1134,9 +1133,7 @@ fragment_defragment_and_free (fragment_data *fd_head, fragment_data *fd, g_free(fd_i->data); fd_i->data=NULL; } - - if( old_data ) - g_free(old_data); + g_free(old_data); /* mark this packet as defragmented. allows us to skip any trailing fragments */ diff --git a/epan/stats_tree.c b/epan/stats_tree.c index ffdb0ae49c..d3f0fd1f78 100644 --- a/epan/stats_tree.c +++ b/epan/stats_tree.c @@ -169,10 +169,8 @@ free_stat_node(stat_node *node) if (node->hash) g_hash_table_destroy(node->hash); - if (node->rng) g_free(node->rng); - - if (node->name) g_free(node->name); - + g_free(node->rng); + g_free(node->name); g_free(node); } diff --git a/epan/tvbuff.c b/epan/tvbuff.c index 83e322c7e2..a47d7792e6 100644 --- a/epan/tvbuff.c +++ b/epan/tvbuff.c @@ -168,10 +168,8 @@ tvb_free(tvbuff_t* tvb) g_slist_free(composite->tvbs); - if (composite->start_offsets) - g_free(composite->start_offsets); - if (composite->end_offsets) - g_free(composite->end_offsets); + g_free(composite->start_offsets); + g_free(composite->end_offsets); if (tvb->real_data) { /* * XXX - do this with a union? diff --git a/epan/uat.h b/epan/uat.h index 1ba3bfa5e0..7977e78e14 100644 --- a/epan/uat.h +++ b/epan/uat.h @@ -324,7 +324,7 @@ gboolean uat_fld_chk_str_ ## what (void* u1 _U_, const char* strptr, unsigned le #define UAT_CSTRING_CB_DEF(basename,field_name,rec_t) \ static void basename ## _ ## field_name ## _set_cb(void* rec, const char* buf, unsigned len, void* u1 _U_, void* u2 _U_) {\ char* new_buf = g_strndup(buf,len); \ - if ((((rec_t*)rec)->field_name)) g_free((((rec_t*)rec)->field_name)); \ + g_free((((rec_t*)rec)->field_name)); \ (((rec_t*)rec)->field_name) = new_buf; } \ static void basename ## _ ## field_name ## _tostr_cb(void* rec, const char** out_ptr, unsigned* out_len, void* u1 _U_, void* u2 _U_) {\ if (((rec_t*)rec)->field_name ) { \ @@ -354,7 +354,7 @@ static void basename ## _ ## field_name ## _tostr_cb(void* rec, const char** out #define UAT_LSTRING_CB_DEF(basename,field_name,rec_t,ptr_element,len_element) \ static void basename ## _ ## field_name ## _set_cb(void* rec, const char* buf, unsigned len, void* u1 _U_, void* u2 _U_) {\ char* new_val = uat_unesc(buf,len,&(((rec_t*)rec)->len_element)); \ - if ((((rec_t*)rec)->ptr_element)) g_free((((rec_t*)rec)->ptr_element)); \ + g_free((((rec_t*)rec)->ptr_element)); \ (((rec_t*)rec)->ptr_element) = new_val; }\ static void basename ## _ ## field_name ## _tostr_cb(void* rec, const char** out_ptr, unsigned* out_len, void* u1 _U_, void* u2 _U_) {\ if (((rec_t*)rec)->ptr_element ) { \ @@ -376,9 +376,9 @@ static void basename ## _ ## field_name ## _tostr_cb(void* rec, const char** out #define UAT_BUFFER_CB_DEF(basename,field_name,rec_t,ptr_element,len_element) \ static void basename ## _ ## field_name ## _set_cb(void* rec, const char* buf, unsigned len, void* u1 _U_, void* u2 _U_) {\ char* new_buf = len ? g_memdup(buf,len) : NULL; \ - if ((((rec_t*)rec)->ptr_element) ) g_free((((rec_t*)rec)->ptr_element)); \ - (((rec_t*)rec)->ptr_element) = new_buf; \ - (((rec_t*)rec)->len_element) = len; } \ + g_free((((rec_t*)rec)->ptr_element)); \ + (((rec_t*)rec)->ptr_element) = new_buf; \ + (((rec_t*)rec)->len_element) = len; } \ static void basename ## _ ## field_name ## _tostr_cb(void* rec, const char** out_ptr, unsigned* out_len, void* u1 _U_, void* u2 _U_) {\ *out_ptr = ((rec_t*)rec)->ptr_element ? ep_memdup(((rec_t*)rec)->ptr_element,((rec_t*)rec)->len_element) : ""; \ *out_len = ((rec_t*)rec)->len_element; } diff --git a/epan/wslua/wslua_listener.c b/epan/wslua/wslua_listener.c index bd103a95fe..41931a5312 100644 --- a/epan/wslua/wslua_listener.c +++ b/epan/wslua/wslua_listener.c @@ -212,7 +212,7 @@ WSLUA_CONSTRUCTOR Listener_new(lua_State* L) { error = register_tap_listener(tap_type, tap, tap->filter, lua_tap_reset, lua_tap_packet, lua_tap_draw); if (error) { - if (tap->filter) g_free(tap->filter); + g_free(tap->filter); g_free(tap->name); g_free(tap); /* WSLUA_ERROR(new_tap,"tap registration error"); */ diff --git a/epan/wslua/wslua_pinfo.c b/epan/wslua/wslua_pinfo.c index ff0eaf0d9f..5e03867eae 100644 --- a/epan/wslua/wslua_pinfo.c +++ b/epan/wslua/wslua_pinfo.c @@ -223,7 +223,7 @@ static int Address__gc(lua_State* L) { Address addr = checkAddress(L,1); if (addr) { - if (addr->data) g_free((void*)addr->data); + g_free((void*)addr->data); g_free((void*)addr); } diff --git a/epan/wslua/wslua_proto.c b/epan/wslua/wslua_proto.c index 08841cc8bf..1cd0169f9a 100644 --- a/epan/wslua/wslua_proto.c +++ b/epan/wslua/wslua_proto.c @@ -225,9 +225,10 @@ static int Pref_gc(lua_State* L) { Pref pref = checkPref(L,1); if (pref && ! pref->name) { - if (pref->label) g_free(pref->label); - if (pref->desc) g_free(pref->desc); - if (pref->type == PREF_STRING) g_free((void*)pref->value.s); + g_free(pref->label); + g_free(pref->desc); + if (pref->type == PREF_STRING) + g_free((void*)pref->value.s); g_free(pref); } |