diff options
author | Guy Harris <guy@alum.mit.edu> | 2015-04-13 22:04:40 -0700 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2015-04-14 05:05:06 +0000 |
commit | d074ed17e1431e90f3fd3baeee5925866d514432 (patch) | |
tree | 13233581d9c8af9a8a1e20b2f726397c4ce70286 /epan | |
parent | eeed4d1121f122aa5579153872193b9a91d8ad52 (diff) |
Just use g_strdup_printf() to create a g_mallocated formatted string.
And, as it's g_mallocated, we don't need to g_mallocate it again with
g_strdup().
The same is true of strings allocated with g_strconcat(). Don't make an
extra copy, and don't free up the original.
Change-Id: Iea470a4424a811934f7fb8b0ba8dd110d337e420
Reviewed-on: https://code.wireshark.org/review/8058
Reviewed-by: Guy Harris <guy@alum.mit.edu>
Diffstat (limited to 'epan')
-rw-r--r-- | epan/ext_menubar.c | 18 |
1 files changed, 5 insertions, 13 deletions
diff --git a/epan/ext_menubar.c b/epan/ext_menubar.c index e716b4d58b..04482bf264 100644 --- a/epan/ext_menubar.c +++ b/epan/ext_menubar.c @@ -64,7 +64,7 @@ extern ext_menu_t * ext_menubar_register_menu(int proto_id, const gchar * menula entry->is_plugin = is_plugin; /* Create a name for this submenu */ - entry->name = g_strdup(name); + entry->name = name; entry->label = g_strdup(menulabel); entry->tooltip = g_strdup(menulabel); @@ -73,7 +73,6 @@ extern ext_menu_t * ext_menubar_register_menu(int proto_id, const gchar * menula menubar_entries = g_list_append(menubar_entries, entry); menubar_menunames = g_list_append(menubar_menunames, name); - g_free(name); return entry; } @@ -92,9 +91,7 @@ extern ext_menu_t * ext_menubar_add_submenu(ext_menu_t * parent, const gchar *me parent->submenu_cnt++; /* Create unique name, which is used by GTK to provide the menu */ - name = (gchar *)g_malloc0(strlen(parent->name) + 4); - g_snprintf(name, strlen(parent->name) + 4, "%sS%02d", - parent->name, parent->submenu_cnt ); + name = g_strdup_printf("%sS%02d", parent->name, parent->submenu_cnt); /* Create submenu entry */ entry = (ext_menubar_t *)g_malloc0(sizeof(ext_menubar_t)); @@ -103,12 +100,11 @@ extern ext_menu_t * ext_menubar_add_submenu(ext_menu_t * parent, const gchar *me /* Just a convenience */ entry->proto = parent->proto; entry->is_plugin = parent->is_plugin; - entry->name = g_strdup(name); + entry->name = name; entry->label = g_strdup(menulabel); entry->tooltip = g_strdup(menulabel); parent->children = g_list_append(parent->children, entry); - g_free(name); return entry; } @@ -128,14 +124,12 @@ static void ext_menubar_add_generic_entry ( parent->item_cnt++; /* Create unique name, which is used by GTK to provide the menu */ - name = (gchar *)g_malloc0(strlen(parent->name) + 4); - g_snprintf(name, strlen(parent->name) + 4, "%sI%02d", - parent->name, parent->item_cnt ); + name = g_strdup_printf("%sI%02d", parent->name, parent->item_cnt); /* Create menu entry */ entry = (ext_menubar_t*)g_malloc0(sizeof(ext_menubar_t)); entry->type = type; - entry->name = g_strdup(name); + entry->name = name; entry->label = g_strdup(label); if ( tooltip != NULL && strlen(tooltip) > 0 ) @@ -145,8 +139,6 @@ static void ext_menubar_add_generic_entry ( entry->user_data = user_data; parent->children = g_list_append(parent->children, entry); - - g_free(name); } extern void ext_menubar_add_entry(ext_menu_t * parent, const gchar *label, |