[Home]

Summary:ASTERISK-14930: [patch] Crash on unregister SIP realtime peer - double free
Reporter:Cristian Dimache (cristiandimache)Labels:
Date Opened:2009-10-02 05:09:42Date Closed:2009-11-10 15:44:27.000-0600
Priority:CriticalRegression?No
Status:Closed/CompleteComponents:Resources/res_config_pgsql
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) gdb.txt
( 1) gdb2.txt
( 2) issue16011.diff
Description:When a realtime SIP peer unregisters, Asterisk crashes.
Comments:By: Leif Madsen (lmadsen) 2009-10-02 13:09:19

Can you verify you have DONT_OPTIMIZE enabled in your build?

By: Cristian Dimache (cristiandimache) 2009-10-03 07:32:52

Attached a debug with DONT_OPTIMIZE

By: Leif Madsen (lmadsen) 2009-10-05 12:00:08

Thanks!

By: David Vossel (dvossel) 2009-11-09 17:37:07.000-0600

I uploaded a patch, let me know if that resolves the issue.

By: Digium Subversion (svnbot) 2009-11-10 09:33:39.000-0600

Repository: asterisk
Revision: 229093

U   trunk/res/res_config_pgsql.c

------------------------------------------------------------------------
r229093 | dvossel | 2009-11-10 09:33:38 -0600 (Tue, 10 Nov 2009) | 11 lines

fixes pgsql double free of threadstorage

A thread storage variable was being freed incorrectly, which
resulted in a double free if two queries were made in the same thread.

(closes issue ASTERISK-14930)
Reported by: cristiandimache
Patches:
     issue16011.diff uploaded by dvossel (license 671)


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

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

By: Digium Subversion (svnbot) 2009-11-10 09:34:16.000-0600

Repository: asterisk
Revision: 229094

_U  branches/1.6.2/
U   branches/1.6.2/res/res_config_pgsql.c

------------------------------------------------------------------------
r229094 | dvossel | 2009-11-10 09:34:16 -0600 (Tue, 10 Nov 2009) | 17 lines

Merged revisions 229093 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
 r229093 | dvossel | 2009-11-10 09:27:45 -0600 (Tue, 10 Nov 2009) | 11 lines
 
 fixes pgsql double free of threadstorage
 
 A thread storage variable was being freed incorrectly, which
 resulted in a double free if two queries were made in the same thread.
 
 (closes issue ASTERISK-14930)
 Reported by: cristiandimache
 Patches:
       issue16011.diff uploaded by dvossel (license 671)
........

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

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

By: Digium Subversion (svnbot) 2009-11-10 09:34:51.000-0600

Repository: asterisk
Revision: 229095

_U  branches/1.6.1/
U   branches/1.6.1/res/res_config_pgsql.c

------------------------------------------------------------------------
r229095 | dvossel | 2009-11-10 09:34:51 -0600 (Tue, 10 Nov 2009) | 17 lines

Merged revisions 229093 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
 r229093 | dvossel | 2009-11-10 09:27:45 -0600 (Tue, 10 Nov 2009) | 11 lines
 
 fixes pgsql double free of threadstorage
 
 A thread storage variable was being freed incorrectly, which
 resulted in a double free if two queries were made in the same thread.
 
 (closes issue ASTERISK-14930)
 Reported by: cristiandimache
 Patches:
       issue16011.diff uploaded by dvossel (license 671)
........

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

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

By: Digium Subversion (svnbot) 2009-11-10 09:36:04.000-0600

Repository: asterisk
Revision: 229096

_U  branches/1.6.0/

------------------------------------------------------------------------
r229096 | dvossel | 2009-11-10 09:36:04 -0600 (Tue, 10 Nov 2009) | 16 lines

Blocked revisions 229093 via svnmerge

........
 r229093 | dvossel | 2009-11-10 09:27:45 -0600 (Tue, 10 Nov 2009) | 11 lines
 
 fixes pgsql double free of threadstorage
 
 A thread storage variable was being freed incorrectly, which
 resulted in a double free if two queries were made in the same thread.
 
 (closes issue ASTERISK-14930)
 Reported by: cristiandimache
 Patches:
       issue16011.diff uploaded by dvossel (license 671)
........

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

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