Wireshark-bugs: [Wireshark-bugs] [Bug 5531] add support for ANSI C12.22 protocol
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: bugzilla-daemon@xxxxxxxxxxxxx
Date: Tue, 25 Oct 2011 09:18:24 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5531

--- Comment #17 from Guy Harris <guy@xxxxxxxxxxxx> 2011-10-25 09:18:22 PDT ---
Sorry, what I meant was

    if (*length < 20) {
        put an error indication in the protocol tree);
        break out of the loop or return from the routine;
    }
    password = tvb_get_ephemeral_string(tvb, *offset, 20);
    proto_tree_add_item(tree, hf_c1222_security_password, tvb, *offset, 20,
ENC_NA);
    *offset += 20;
    *length -= 20;

"Put an error indication into the protocol tree" should be something such as
"put an expert info item on the parent protocol tree item, indicating that the
item should have been long enough to include the field in question, but
wasn't".

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