[Home]

Summary:ASTERISK-15726: [patch] chan_mgcp crash Adtran 624 asterisk 1.8.0 Beta 2
Reporter:Stacy Vinson (svinson)Labels:
Date Opened:2010-08-02 01:49:13Date Closed:2010-08-04 09:05:10
Priority:CriticalRegression?No
Status:Closed/CompleteComponents:Channels/chan_mgcp
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) 20100804__issue17775.diff.txt
( 1) backtrace.txt
( 2) valgrind.txt
Description:crash with Adtran 624  (MGCP) no problem with asterisk 1.6.x



Loaded symbols for /lib/libnss_files.so.2
Core was generated by `asterisk -gccvvvvv'.
Program terminated with signal 11, Segmentation fault.
#0  0x00b11a0e in find_and_retrans (id=0xb7500508, fd=17, events=1, ignore=0x0)
   at chan_mgcp.c:3583
3583            for (cur = sub->parent->parent->responses, next = cur->next; cur; cur = next, next = cur->next) {
Missing separate debuginfos, use: debuginfo-install SDL-1.2.14-4.fc13.i686 cyrus-sasl-lib-2.1.23-11.fc13.i686 glibc-2.12-3.i686 keyutils-libs-1.2-6.fc12.i686 krb5-libs-1.7.1-10.fc13.i686 libX11-1.3.1-3.fc13.i686 libXau-1.0.5-1.fc12.i686 libattr-2.4.44-3.fc13.i686 libcap-2.17-1.fc13.i686 libcom_err-1.41.10-7.fc13.i686 libcurl-7.20.1-3.fc13.i686 libidn-1.16-1.fc13.i686 libogg-1.2.0-1.fc13.i686 libselinux-2.0.90-5.fc13.i686 libssh2-1.2.4-1.fc13.i686 libvorbis-1.3.1-1.fc13.i686 libxcb-1.5-1.fc13.i686 libxml2-2.7.7-1.fc13.i686 ncurses-libs-5.7-7.20100130.fc13.i686 nspr-4.8.4-2.fc13.i686 nss-3.12.6-7.fc13.i686 nss-softokn-freebl-3.12.6-3.fc13.i686 nss-util-3.12.6-1.fc13.i686 openldap-2.4.21-9.fc13.i686 openssl-1.0.0a-1.fc13.i686 sqlite-3.6.22-1.fc13.i686 zlib-1.2.3-23.fc12.i686

 

****** ADDITIONAL INFORMATION ******

(Fedora 13) Linux  2.6.33.6-147.fc13.i686.PAE #1 SMP Tue Jul 6 22:24:44 UTC 2010 i686 i686 i386 GNU/Linux

adtran TA 624  Gen3 Firmware D.05.08.00
Comments:By: Tilghman Lesher (tilghman) 2010-08-02 10:18:43

Could you upload the 'bt full' please?
By: Stacy Vinson (svinson) 2010-08-03 17:20:39

if you need anything more just let me know, Thanks
By: Tilghman Lesher (tilghman) 2010-08-04 00:26:57

Patch uploaded; please verify that this fixes your crash.
By: Stacy Vinson (svinson) 2010-08-04 00:55:55

Yes, the patch fixes the issue. Thanks for you work!
By: Digium Subversion (svnbot) 2010-08-04 09:04:06

Repository: asterisk
Revision: 280879

U   branches/1.8/channels/chan_mgcp.c

------------------------------------------------------------------------
r280879 | tilghman | 2010-08-04 09:04:05 -0500 (Wed, 04 Aug 2010) | 14 lines

Check cur value before attempting a deref.

(closes issue ASTERISK-15726)
Reported by: svinson
Patches:
      20100804__issue17775.diff.txt uploaded by tilghman (license 14)
Tested by: svinson

(closes issue ASTERISK-16457)
Reported by: tgruenberg
Patches:
      20100804__issue17775.diff.txt uploaded by tilghman (license 14)
Tested by: tgruenberg

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=280879
By: Digium Subversion (svnbot) 2010-08-04 09:05:10

Repository: asterisk
Revision: 280880

_U  trunk/
U   trunk/channels/chan_mgcp.c

------------------------------------------------------------------------
r280880 | tilghman | 2010-08-04 09:05:09 -0500 (Wed, 04 Aug 2010) | 21 lines

Merged revisions 280879 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
 r280879 | tilghman | 2010-08-04 09:04:07 -0500 (Wed, 04 Aug 2010) | 14 lines
 
 Check cur value before attempting a deref.
 
 (closes issue ASTERISK-15726)
  Reported by: svinson
  Patches:
        20100804__issue17775.diff.txt uploaded by tilghman (license 14)
  Tested by: svinson
 
 (closes issue ASTERISK-16457)
  Reported by: tgruenberg
  Patches:
        20100804__issue17775.diff.txt uploaded by tilghman (license 14)
  Tested by: tgruenberg
........

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=280880