diff options
author | Guy Harris <guy@alum.mit.edu> | 2005-09-28 09:31:37 +0000 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2005-09-28 09:31:37 +0000 |
commit | e2d774be167d2ff4d2759c5218b49def4ba7e265 (patch) | |
tree | b1d6194d402b9fbc0eda9704fe7b6ac25cfdad53 /epan/dissectors/packet-hsrp.c | |
parent | 16bd74ab0c6960ea48c82ecb815ef3f548cfdada (diff) | |
download | wireshark-e2d774be167d2ff4d2759c5218b49def4ba7e265.tar.gz |
Don't change the Info column if we have an unknown opcode; we've already
set it to a value that should indicate that the opcode is unknown, and
also indicates what the value is. (Especially don't do it without
calling check_col() to check whether we *should* change the info column,
as we'll dereference a null pointer if we shouldn't; this change should
fix bug 489.)
svn path=/trunk/; revision=16038
Diffstat (limited to 'epan/dissectors/packet-hsrp.c')
-rw-r--r-- | epan/dissectors/packet-hsrp.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/epan/dissectors/packet-hsrp.c b/epan/dissectors/packet-hsrp.c index 88ed8e7f74..d4460b8107 100644 --- a/epan/dissectors/packet-hsrp.c +++ b/epan/dissectors/packet-hsrp.c @@ -185,8 +185,6 @@ dissect_hsrp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) col_append_fstr(pinfo->cinfo, COL_INFO, " (state %s)", val_to_str(state, hsrp_adv_state_vals, "Unknown")); } - } else { - col_set_str(pinfo->cinfo, COL_INFO, "Unknown"); } if (tree) { |