[Home]

Summary:ASTERISK-01155: pri->pvt[chan]->owner deferenced when NULL during PRI hangup
Reporter:Simon Kirby (sim)Labels:
Date Opened:2004-03-04 16:08:49.000-0600Date Closed:2008-01-15 14:45:56.000-0600
Priority:CriticalRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) asterisk-v1-0_stable+pri_hangup_fix.patch
Description:This line of code:

pri->pvt[chan]->owner->hangupcause = hangup_pri2cause(e->hangup.cause);

is executed directly before a check to see if ->owner is not NULL.  In our case, ->owner is NULL, perhaps because we're hanging up a line forward in and out of the same PRI.

The attached patch moves the assignment to within the "if" block.

All versions seem to be affected.
Comments:By: James Golovich (jamesgolovich) 2004-03-04 23:25:34.000-0600

Fixed in CVS.  Thanks
By: Digium Subversion (svnbot) 2008-01-15 14:45:56.000-0600

Repository: asterisk
Revision: 2341

U   trunk/channels/chan_zap.c

------------------------------------------------------------------------
r2341 | citats | 2008-01-15 14:45:55 -0600 (Tue, 15 Jan 2008) | 2 lines

Fix pri->pvt[chan]->owner dereference (bug ASTERISK-1155)

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

http://svn.digium.com/view/asterisk?view=rev&revision=2341
By: Digium Subversion (svnbot) 2008-01-15 14:45:56.000-0600

Repository: asterisk
Revision: 2342

U   branches/v1-0_stable/channels/chan_zap.c

------------------------------------------------------------------------
r2342 | citats | 2008-01-15 14:45:56 -0600 (Tue, 15 Jan 2008) | 2 lines

Fix pri->pvt[chan]->owner dereference (bug ASTERISK-1155)

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

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