diff options
author | Jeff Foster <jfoste@woodward.com> | 2001-10-17 21:24:28 +0000 |
---|---|---|
committer | Jeff Foster <jfoste@woodward.com> | 2001-10-17 21:24:28 +0000 |
commit | 0945259b596309141eb6c9121a4bfca3da5352c5 (patch) | |
tree | 66b6136bb0e006481ac63963447af9442b7489ce /packet-smb-pipe.c | |
parent | c2cdf8acd044fa2f3e38d460049f8fc56aea053a (diff) | |
download | wireshark-0945259b596309141eb6c9121a4bfca3da5352c5.tar.gz wireshark-0945259b596309141eb6c9121a4bfca3da5352c5.tar.bz2 wireshark-0945259b596309141eb6c9121a4bfca3da5352c5.zip |
fixed bug in packet-smb-pipe.c dissect_transact_data routine. If the
aux_count_p value was NULL the program would SIGSEGV when *aux_count_p
was initially set to 0. Added NULL pointer test.
svn path=/trunk/; revision=4036
Diffstat (limited to 'packet-smb-pipe.c')
-rw-r--r-- | packet-smb-pipe.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/packet-smb-pipe.c b/packet-smb-pipe.c index daeb4c3883..60e831b1ed 100644 --- a/packet-smb-pipe.c +++ b/packet-smb-pipe.c @@ -8,7 +8,7 @@ XXX Fixme : shouldnt show [malformed frame] for long packets * significant rewrite to tvbuffify the dissector, Ronnie Sahlberg and * Guy Harris 2001 * - * $Id: packet-smb-pipe.c,v 1.35 2001/09/14 07:10:05 guy Exp $ + * $Id: packet-smb-pipe.c,v 1.36 2001/10/17 21:24:28 jfoster Exp $ * * Ethereal - Network traffic analyzer * By Gerald Combs <gerald@ethereal.com> @@ -1601,7 +1601,9 @@ dissect_transact_data(tvbuff_t *tvb, int offset, int convert, const char *string; gint string_len; - *aux_count_p = 0; + if( aux_count_p != NULL) + *aux_count_p = 0; + while ((c = *desc++) != '\0') { switch (c) { |