diff options
author | Jeff Morriss <jeff.morriss.ws@gmail.com> | 2013-05-24 15:28:04 +0000 |
---|---|---|
committer | Jeff Morriss <jeff.morriss.ws@gmail.com> | 2013-05-24 15:28:04 +0000 |
commit | 7dddfb1784667e65616b02834469b834819161cb (patch) | |
tree | cd5a6284c5d561945ae14c3ec250624c5588e7ec /tshark.c | |
parent | f93348a2f71de4b2b06d23467de444c89ef8ee09 (diff) |
Fix https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8714 :
If we're not doing dissection (in 2-pass mode) then don't try to mark frames
as depended upon: in that case epan has not been initialized so we shouldn't
be looking in the edt (and anyway without dissection there won't be any
dependent frames).
(I'm not convinced there's any reason to run 2-pass mode without dissection,
however...)
svn path=/trunk/; revision=49554
Diffstat (limited to 'tshark.c')
-rw-r--r-- | tshark.c | 8 |
1 files changed, 7 insertions, 1 deletions
@@ -2713,7 +2713,13 @@ process_packet_first_pass(capture_file *cf, frame_data_set_after_dissect(&fdlocal, &cum_bytes); prev_cap = prev_dis = frame_data_sequence_add(cf->frames, &fdlocal); - g_slist_foreach(edt.pi.dependent_frames, find_and_mark_frame_depended_upon, cf->frames); + /* If we're not doing dissection then there won't be any dependent frames. + * More importantly, edt.pi.dependent_frames won't be initialized because + * epan hasn't been initialized. + */ + if (do_dissection) { + g_slist_foreach(edt.pi.dependent_frames, find_and_mark_frame_depended_upon, cf->frames); + } cf->count++; } else { |