Wireshark-bugs: [Wireshark-bugs] [Bug 5051] Patch to packet_bacapp.c

Date: Mon, 9 Aug 2010 15:48:17 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5051

Gerald Combs <gerald@xxxxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
                 CC|                            |gerald@xxxxxxxxxxxxx
           See Also|                            |https://bugs.wireshark.org/
                   |                            |bugzilla/show_bug.cgi?id=50
                   |                            |90
         Resolution|FIXED                       |

--- Comment #11 from Gerald Combs <gerald@xxxxxxxxxxxxx> 2010-08-09 15:48:15 PDT ---
Looks like this triggered bug 5090. Is there a particular reason a bunch of
while loop checks were removed? E.g.

    while ((tvb_reported_length_remaining(tvb, offset) >
0)&&(offset>lastoffset))

was changed to

    while (tvb_reported_length_remaining(tvb, offset))

in a lot of places. This seems to have created an infinite loop at line 7508:

    while (tvb_reported_length_remaining(tvb, offset)) {
        lastoffset = offset;
        fTagHeader (tvb, offset, &tag_no, &tag_info, &lvt);
        if (tag_is_closing(tag_info)) {
            continue;
        }

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