aboutsummaryrefslogtreecommitdiffstats
path: root/configure.ac
diff options
context:
space:
mode:
authorDavid Creswick <dcrewi@gyrae.net>2015-04-05 19:13:57 -0500
committerGerald Combs <gerald@wireshark.org>2015-05-05 23:49:28 +0000
commitb5ef92efc067eb172ab71cc39b0aa3f32ab5f01d (patch)
tree06db956fea686fc6ea7de34cbd6835396e57687c /configure.ac
parentfa04dfdd776e950467aee7f538139fb3fd8a75bb (diff)
GTK UI: remove eo_saveable_pathname in favor of eo_massage_str
The function had several deficiencies: - When "" is passed as the filename, the function returns a null pointer (due to the quirky behavior of g_strsplit_set), which causes a segfault when it's dereferenced later. I'm not sure what the correct return should be, but it shouldn't be NULL. - It leaks memory. (The array of strings returned by g_strsplit_set are never freed.) - It only strips out backslashes. That is not the only character that is disallowed in filenames, even on windows. - The functionality is already provided by eo_massage_str and it does a more complete job of it. Change-Id: I4d9eab7506048c5d04de8f163b8af1e3f67c163d Reviewed-on: https://code.wireshark.org/review/7996 Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com> Reviewed-by: Gerald Combs <gerald@wireshark.org>
Diffstat (limited to 'configure.ac')
0 files changed, 0 insertions, 0 deletions