[asterisk-bugs] [Asterisk 0014111]: [patch] app_chanspy crashed so-as "chanspy_ds.lock" has random values

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=14111 
====================================================================== 
Reported By:                ys
Assigned To:                russell
====================================================================== 
Project:                    Asterisk
Issue ID:                   14111
Category:                   Applications/app_chanspy
Reproducibility:            sometimes
Severity:                   minor
Priority:                   normal
Status:                     closed
Asterisk Version:           SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  1.4  
SVN Revision (number only!): 165881 
Disclaimer on File?:        N/A 
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2008-12-19 05:32 CST
Last Modified:              2008-12-19 09:08 CST
====================================================================== 
Summary:                    [patch] app_chanspy crashed so-as "chanspy_ds.lock"
has random values
Description: 
Hi.

 Inside function common_exec() declared struct chanspy_ds chanspy_ds, that
contain "ast_mutex_t lock".

 This lock is not initialized and contain random values from stack heap.
This may cause random crash in call to:
ast_mutex_init(&chanspy_ds.lock);
or 
ast_mutex_lock(&chanspy_ds.lock);

In my BSD box with DEBUG_THREADS options enabled, this application crashed
permanently on exit in ast_mutex_lock(&chanspy_ds.lock); point. 






====================================================================== 

---------------------------------------------------------------------- 
 (0096712) svnbot (reporter) - 2008-12-19 09:08
 http://bugs.digium.com/view.php?id=14111#c96712 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 165892

_U  branches/1.6.1/
U   branches/1.6.1/apps/app_chanspy.c

------------------------------------------------------------------------
r165892 | russell | 2008-12-19 09:08:54 -0600 (Fri, 19 Dec 2008) | 25
lines

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

................
r165890 | russell | 2008-12-19 09:05:09 -0600 (Fri, 19 Dec 2008) | 17
lines

Merged revisions 165889 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r165889 | russell | 2008-12-19 09:03:02 -0600 (Fri, 19 Dec 2008) | 9 lines

Ensure that the chanspy datastore is fully initialized.

This patch resolved some random crash issues observed by a user on a BSD
system

(closes issue http://bugs.digium.com/view.php?id=14111)
Reported by: ys
Patches:
      app_chanspy.c.diff uploaded by ys (license 281)

........

................

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

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

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2008-12-19 09:08 svnbot         Checkin                                      
2008-12-19 09:08 svnbot         Note Added: 0096712                          
======================================================================