ANNOUNCEMENT: Live Wireshark University & Allegro Packets online APAC Wireshark Training Session
April 17th, 2024 | 14:30-16:00 SGT (UTC+8) | Online

Wireshark-bugs: [Wireshark-bugs] [Bug 5531] add support for ANSI C12.22 protocol

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.