Wireshark-bugs: [Wireshark-bugs] [Bug 5858] some 'set but not used' compilation fixes
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: bugzilla-daemon@xxxxxxxxxxxxx
Date: Tue, 26 Apr 2011 13:27:01 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5858

--- Comment #25 from Yaniv Kaul <mykaul@xxxxxxxxx> 2011-04-26 13:27:01 PDT ---
(In reply to comment #22)
> Hi,
> The packet-ber.c patch looks bogus next_tvb is used at line 3843
> count=seq->func(imp_tag, next_tvb, 0, actx, tree, *seq->p_id)-hoffset;
> Regards
> Anders

Note that 'count' was set but not used, and when I removed it, seq->func() was
kind of pointless, so I removed it, and then next_tvb was a bit useless.

I hope I did not err here, though.

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