Added Mark H. Wood's <mwood@IUPUI.edu> fix for unknown SAP types. It was

an off-by-one error. I replicated his fix to another part of the code
that looks up the SAP types (when adding the information to the proto_tree).

svn path=/trunk/; revision=681
This commit is contained in:
Gilbert Ramirez 1999-09-15 22:33:17 +00:00
parent be8c6c16f8
commit ec376ff57f
1 changed files with 3 additions and 3 deletions

View File

@ -2,7 +2,7 @@
* Routines for NetWare's IPX
* Gilbert Ramirez <gram@verdict.uthscsa.edu>
*
* $Id: packet-ipx.c,v 1.26 1999/09/02 23:17:57 guy Exp $
* $Id: packet-ipx.c,v 1.27 1999/09/15 22:33:17 gram Exp $
*
* Ethereal - Network traffic analyzer
* By Gerald Combs <gerald@unicom.net>
@ -570,7 +570,7 @@ dissect_sap(const u_char *pd, int offset, frame_data *fd, proto_tree *tree) {
if (check_col(fd, COL_PROTOCOL))
col_add_str(fd, COL_PROTOCOL, "SAP");
if (check_col(fd, COL_INFO)) {
if (query.query_type < 4) {
if (query.query_type >= 1 && query.query_type <= 4) {
col_add_str(fd, COL_INFO, sap_type[query.query_type - 1]);
}
else {
@ -582,7 +582,7 @@ dissect_sap(const u_char *pd, int offset, frame_data *fd, proto_tree *tree) {
ti = proto_tree_add_item(tree, proto_sap, offset, END_OF_FRAME, NULL);
sap_tree = proto_item_add_subtree(ti, ETT_IPXSAP);
if (query.query_type < 4) {
if (query.query_type >= 1 && query.query_type <= 4) {
proto_tree_add_text(sap_tree, offset, 2, sap_type[query.query_type - 1]);
}
else {