aboutsummaryrefslogtreecommitdiffstats
path: root/file.c
diff options
context:
space:
mode:
authorStig Bjørlykke <stig@bjorlykke.org>2009-03-15 18:08:46 +0000
committerStig Bjørlykke <stig@bjorlykke.org>2009-03-15 18:08:46 +0000
commit0258be0b0544d4ccb08f2b01ef2b59f7c7afd6fd (patch)
tree44e6b5fab5e07e13c8e55052f5fa45e0c61c2e8c /file.c
parent0f9f5caf9deb0dda5ea5fe1642d2e9bc7e7fe43f (diff)
From Jakub Zawadzki:
More remove checking for NULL before g_free(). svn path=/trunk/; revision=27728
Diffstat (limited to 'file.c')
-rw-r--r--file.c12
1 files changed, 4 insertions, 8 deletions
diff --git a/file.c b/file.c
index 28fd5a9b99..bea4c0cf53 100644
--- a/file.c
+++ b/file.c
@@ -1438,8 +1438,7 @@ cf_filter_packets(capture_file *cf, gchar *dftext, gboolean force)
}
/* We have a valid filter. Replace the current filter. */
- if (cf->dfilter != NULL)
- g_free(cf->dfilter);
+ g_free(cf->dfilter);
cf->dfilter = dftext;
/* Now rescan the packet list, applying the new filter, but not
@@ -2198,12 +2197,9 @@ cf_print_packets(capture_file *cf, print_args_t *print_args)
"selected packets", TRUE, print_packet,
&callback_args);
- if (callback_args.header_line_buf != NULL)
- g_free(callback_args.header_line_buf);
- if (callback_args.line_buf != NULL)
- g_free(callback_args.line_buf);
- if (callback_args.col_widths != NULL)
- g_free(callback_args.col_widths);
+ g_free(callback_args.header_line_buf);
+ g_free(callback_args.line_buf);
+ g_free(callback_args.col_widths);
switch (ret) {