diff options
| author | Guy Harris <guy@alum.mit.edu> | 2015-01-03 14:35:01 -0800 |
|---|---|---|
| committer | Guy Harris <guy@alum.mit.edu> | 2015-01-03 22:35:27 +0000 |
| commit | 938485436e7b7c647323a147c8ea1abc33e07564 (patch) | |
| tree | 2c0ce27222b3a8f08c950b1d8655f04557286d21 /ui/gtk/file_import_dlg.c | |
| parent | 9df73862fbb2ad7eecdad335714fa6a0487c1973 (diff) | |
Use g_strdup_printf(), rather than a fixed-length buffer, for the app name.
Add some missing g_free()s while we're at it.
Change-Id: Id38acc21d3c0b337e5d05baaf5ebbcd63699ff50
Reviewed-on: https://code.wireshark.org/review/6287
Reviewed-by: Guy Harris <guy@alum.mit.edu>
Diffstat (limited to 'ui/gtk/file_import_dlg.c')
| -rw-r--r-- | ui/gtk/file_import_dlg.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/ui/gtk/file_import_dlg.c b/ui/gtk/file_import_dlg.c index f861f7c81f..69ae7b6833 100644 --- a/ui/gtk/file_import_dlg.c +++ b/ui/gtk/file_import_dlg.c @@ -461,7 +461,7 @@ file_import_open(text_import_info_t *info) wtapng_iface_descriptions_t *idb_inf; wtapng_if_descr_t int_data; GString *os_info_str; - char appname[100]; + char *appname; /* Choose a random name for the temporary import buffer */ import_file_fd = create_tempfile(&tmpname, "import"); @@ -471,7 +471,7 @@ file_import_open(text_import_info_t *info) os_info_str = g_string_new(""); get_os_version_info(os_info_str); - g_snprintf(appname, sizeof(appname), "Wireshark %s", get_ws_vcs_version_info()); + appname = g_strdup_printf("Wireshark %s", get_ws_vcs_version_info()); shb_hdr = g_new(wtapng_section_t,1); shb_hdr->section_length = -1; @@ -562,6 +562,8 @@ end: g_free(info->date_timestamp_format); g_free(info); g_free(capfile_name); + g_free(shb_hdr); + g_free(appname); window_destroy(file_import_dlg_w); } |