From Cvetan Ivanov:

1) dissect_isup_calling_party_number_parameter reported malformed packet
if the calling party number IE contains empty number (possibly with
presentation set to 'number not available')

I've basicly added if(length > 0) around the digits tree dissector,
which may not be obvious from the patch becase of the indenting of the
entire block.

2) As the tap_calling_number is static, if there is call with missing
calling_party_number IE, it gets assigned the calling party number of
the previuos call containing such IE.

fixed by adding tap_calling_number = NULL;
in the beginning of dissect_isup_message, and

tap_rec.calling_number=tap_calling_number?tap_calling_number:g_strdup("");

in the end to avoid segfault

Those things may not happen too often in the telecom world, but happen
all the time with VoIP.


svn path=/trunk/; revision=14268

1 files changed, 1 insertions, 0 deletions

diff --git a/AUTHORS b/AUTHORS
index 8fdb7679f5..db008a7dc9 100644
--- a/AUTHORS
+++ b/AUTHORS
@@ -2440,6 +2440,7 @@ Steve Karg		<skarg [AT] users.sourceforge.net>
 Steve Packet		<packetsteve [AT] hotmail.com>
 Javier Acuna		<javier.acuna [AT] sixbell.cl>
 Miklos Szurdi		<szurdimiklos [AT] yahoo.com>
+Cvetan Ivanov		<zezo [AT] spnet.net>
 
 Alain Magloire <alainm[AT]rcsm.ece.mcgill.ca> was kind enough to
 give his permission to use his version of snprintf.c.