Huge thanks to our Platinum Members Endace and LiveAction,
and our Silver Member Veeam, for supporting the Wireshark Foundation and project.

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.