diff options
author | David Creswick <dcrewi@gyrae.net> | 2015-04-05 19:13:57 -0500 |
---|---|---|
committer | Gerald Combs <gerald@wireshark.org> | 2015-05-05 23:49:28 +0000 |
commit | b5ef92efc067eb172ab71cc39b0aa3f32ab5f01d (patch) | |
tree | 06db956fea686fc6ea7de34cbd6835396e57687c /configure.ac | |
parent | fa04dfdd776e950467aee7f538139fb3fd8a75bb (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