Fixed problem with handling empty parameters.

Problem was pointed out by Alejandro Vaquero who
provided a fix.  Implemented a slightly different fix.

svn path=/trunk/; revision=3653
This commit is contained in:
Ed Warnicke 2001-07-05 04:27:10 +00:00
parent 537d308abe
commit 902b66b368
2 changed files with 3 additions and 5 deletions

View File

@ -13,5 +13,5 @@
#endif #endif
/* Version number of package */ /* Version number of package */
#define VERSION "0.0.6" #define VERSION "0.0.7"

View File

@ -2,7 +2,7 @@
* Routines for mgcp packet disassembly * Routines for mgcp packet disassembly
* RFC 2705 * RFC 2705
* *
* $Id: packet-mgcp.c,v 1.23 2001/07/04 22:01:22 hagbard Exp $ * $Id: packet-mgcp.c,v 1.24 2001/07/05 04:27:10 hagbard Exp $
* *
* Copyright (c) 2000 by Ed Warnicke <hagbard@physics.rutgers.edu> * Copyright (c) 2000 by Ed Warnicke <hagbard@physics.rutgers.edu>
* *
@ -840,9 +840,7 @@ static gint tvb_parse_param(tvbuff_t* tvb, gint offset, gint len, int** hf){
tvb_current_offset++; tvb_current_offset++;
tvb_current_offset = tvb_skip_wsp(tvb,tvb_current_offset, tvb_current_offset = tvb_skip_wsp(tvb,tvb_current_offset,
(len - tvb_current_offset + offset)); (len - tvb_current_offset + offset));
if(len > tvb_current_offset - offset){ returnvalue = tvb_current_offset;
returnvalue = tvb_current_offset;
}
} }
else { else {
*hf = &hf_mgcp_param_invalid; *hf = &hf_mgcp_param_invalid;