diff options
author | morriss <morriss@f5534014-38df-0310-8fa8-9805f1628bb7> | 2007-08-22 16:30:16 +0000 |
---|---|---|
committer | morriss <morriss@f5534014-38df-0310-8fa8-9805f1628bb7> | 2007-08-22 16:30:16 +0000 |
commit | 94d4a3a89ddc2982d28b43465e0e85da638f8ff9 (patch) | |
tree | 2c5d954d066dd1bc7b7b81af5e15b2ebea283dcd /capture_ui_utils.c | |
parent | 7e467dcdec10219e01ddc7e0806b9b190bc4d9e8 (diff) |
Fix bug http://bugs.wireshark.org/bugzilla/show_bug.cgi?id=1757 :
Try to call get_interface_descriptive_name() as little as possible (storing
the result in capture_opts) to avoid a performance hit during live capture
(especially if you have lots of interfaces) and to avoid leaking memory.
One issue with this is that capture_opts.c cannot (without adding significant
dependencies) set the iface_descr so readers of that field (only gtk/main.c
and tshark.c) use a macro to (set if not already set and) get the value of
that field.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@22587 f5534014-38df-0310-8fa8-9805f1628bb7
Diffstat (limited to 'capture_ui_utils.c')
-rw-r--r-- | capture_ui_utils.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/capture_ui_utils.c b/capture_ui_utils.c index 67e659219a..33a4b9c1fe 100644 --- a/capture_ui_utils.c +++ b/capture_ui_utils.c @@ -100,6 +100,8 @@ capture_dev_user_descr_find(const gchar *if_name) * If the user has specified a comment, use that. Otherwise, * if get_interface_list() supplies a description, use that, * otherwise use the interface name. + * + * The result must be g_free()'d when you're done with it. */ char * get_interface_descriptive_name(const char *if_name) |