[Home]

Summary:ASTERISK-14917: [patch] Reset entire span request can result in a crash
Reporter:Torrey Searle (tsearle)Labels:
Date Opened:2009-10-01 04:10:05Date Closed:2009-10-06 17:34:57
Priority:CriticalRegression?No
Status:Closed/CompleteComponents:Channels/chan_dahdi
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) dahdi_reset_crash.patch
Description:Upon getting a request to reset an entire span, asterisk can crash with a core dump.

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

Analysis shows that asterisk is using an incorrect (and initialized) variable in the reset logic.  Attached is a patch that prevents this crash from happening.
Comments:By: Torrey Searle (tsearle) 2009-10-01 04:16:06

please remove the chan_dahdi uploaded it by mistake :-)
By: Digium Subversion (svnbot) 2009-10-06 17:30:28

Repository: asterisk
Revision: 222393

U   branches/1.4/channels/chan_dahdi.c

------------------------------------------------------------------------
r222393 | jpeeler | 2009-10-06 17:30:28 -0500 (Tue, 06 Oct 2009) | 11 lines

Fix potential crash when entire span request is received.

The variable index used in this scenario for accessing the dahdi_pvts was
wrong and was most likely copied from the several other places it is used
correctly.

(closes issue ASTERISK-14917)
Reported by: tsearle
Patches:
     dahdi_reset_crash.patch uploaded by tsearle (license 373)

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

http://svn.digium.com/view/asterisk?view=rev&revision=222393
By: Digium Subversion (svnbot) 2009-10-06 17:30:59

Repository: asterisk
Revision: 222394

_U  trunk/

------------------------------------------------------------------------
r222394 | jpeeler | 2009-10-06 17:30:59 -0500 (Tue, 06 Oct 2009) | 17 lines

Blocked revisions 222393 via svnmerge

........
 r222393 | jpeeler | 2009-10-06 17:27:13 -0500 (Tue, 06 Oct 2009) | 11 lines
 
 Fix potential crash when entire span request is received.
 
 The variable index used in this scenario for accessing the dahdi_pvts was
 wrong and was most likely copied from the several other places it is used
 correctly.
 
 (closes issue ASTERISK-14917)
 Reported by: tsearle
 Patches:
       dahdi_reset_crash.patch uploaded by tsearle (license 373)
........

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

http://svn.digium.com/view/asterisk?view=rev&revision=222394
By: Digium Subversion (svnbot) 2009-10-06 17:32:35

Repository: asterisk
Revision: 222395

U   branches/1.6.0/channels/chan_dahdi.c

------------------------------------------------------------------------
r222395 | jpeeler | 2009-10-06 17:32:35 -0500 (Tue, 06 Oct 2009) | 14 lines

Fix potential crash when entire span request is received.

The variable index used in this scenario for accessing the dahdi_pvts was
wrong and was most likely copied from the several other places it is used
correctly.

(closes issue ASTERISK-14917)
Reported by: tsearle
Patches:
    dahdi_reset_crash.patch uploaded by tsearle (license 373)

Modified:
  branches/1.4/channels/chan_dahdi.c

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

http://svn.digium.com/view/asterisk?view=rev&revision=222395
By: Digium Subversion (svnbot) 2009-10-06 17:33:26

Repository: asterisk
Revision: 222396

U   branches/1.6.1/channels/chan_dahdi.c

------------------------------------------------------------------------
r222396 | jpeeler | 2009-10-06 17:33:26 -0500 (Tue, 06 Oct 2009) | 14 lines

Fix potential crash when entire span request is received.

The variable index used in this scenario for accessing the dahdi_pvts was
wrong and was most likely copied from the several other places it is used
correctly.

(closes issue ASTERISK-14917)
Reported by: tsearle
Patches:
    dahdi_reset_crash.patch uploaded by tsearle (license 373)

Modified:
  branches/1.4/channels/chan_dahdi.c

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

http://svn.digium.com/view/asterisk?view=rev&revision=222396
By: Digium Subversion (svnbot) 2009-10-06 17:34:56

Repository: asterisk
Revision: 222397

U   branches/1.6.2/channels/chan_dahdi.c

------------------------------------------------------------------------
r222397 | jpeeler | 2009-10-06 17:34:56 -0500 (Tue, 06 Oct 2009) | 11 lines

Fix potential crash when entire span request is received.

The variable index used in this scenario for accessing the dahdi_pvts was
wrong and was most likely copied from the several other places it is used
correctly.

(closes issue ASTERISK-14917)
Reported by: tsearle
Patches:
    dahdi_reset_crash.patch uploaded by tsearle (license 373)

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

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