[Home]

Summary:ASTERISK-10773: [patch] missing locks while calling astman_send_error()
Reporter:Eliel Sardanons (eliel)Labels:
Date Opened:2007-11-15 09:54:07.000-0600Date Closed:2007-11-16 13:39:04.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Core/ManagerInterface
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) manager.c.patch
( 1) manager.c.patch2
Description:I found to places where we need to lock the mansession before calling astman_send_error().
Comments:By: snuffy (snuffy) 2007-11-15 17:54:12.000-0600

this appears to have typo..
he locks twice, one should be unlock
By: Eliel Sardanons (eliel) 2007-11-15 18:55:43.000-0600

You are right I upload the wrong patch
By: Digium Subversion (svnbot) 2007-11-16 13:39:04.000-0600

Repository: asterisk
Revision: 89331

U   trunk/main/manager.c

------------------------------------------------------------------------
r89331 | mmichelson | 2007-11-16 13:39:03 -0600 (Fri, 16 Nov 2007) | 6 lines

Added some locks that should have been around astman_send_error,
at least according to the comments.

(closes issue ASTERISK-10773, reported and patched by eliel)


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