Wireshark-bugs: [Wireshark-bugs] [Bug 7235] Fixes for FPSpotlightRPC AFP dissector
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: bugzilla-daemon@xxxxxxxxxxxxx
Date: Wed, 9 May 2012 02:16:32 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7235

--- Comment #4 from Frank Lahm <franklahm@xxxxxxxxxxxxxx> 2012-05-09 02:16:32 PDT ---
(In reply to comment #3)
> @@ -4331,7 +4331,6 @@
>      offset = spotlight_dissect_query_loop(tvb, sub_tree_queries, offset,
> SQ_CPX_TYPE_ARRAY, INT_MAX, offset + (gint)toc_offset + 8, encoding);
> 
>      /* ToC */
> -    offset += (gint)toc_offset;
>      if (toc_entries < 1) {
>          proto_tree_add_text(tree,
>                      tvb,
> Why remove this line of code ?

Because offset is already update in the previous line by
spotlight_dissect_query_loop().

-- 
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.