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 /summary.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 'summary.c')
-rw-r--r-- | summary.c | 6 |
1 files changed, 1 insertions, 5 deletions
@@ -123,10 +123,6 @@ summary_fill_in_capture(capture_options *capture_opts, summary_tally *st) { st->cfilter = capture_opts->cfilter; st->iface = capture_opts->iface; - if(st->iface) { - st->iface_descr = get_interface_descriptive_name(st->iface); - } else { - st->iface_descr = NULL; - } + st->iface_descr = GET_IFACE_DESCR(capture_opts); } #endif |