[Asterisk-code-review] pbx: Update device and presence state when changing a hint e... (asterisk[11])

Richard Mudgett asteriskteam at digium.com
Wed Sep 16 13:42:47 CDT 2015


Richard Mudgett has posted comments on this change.

Change subject: pbx: Update device and presence state when changing a hint extension.
......................................................................


Patch Set 1:

(1 comment)

https://gerrit.asterisk.org/#/c/1262/1/main/pbx.c
File main/pbx.c:

Line 6030: 	if ((presence_state = extension_presence_state_helper(ne, &subtype, &message)) > 0) {
Could calling extension_presence_state_helper() with the hints container locked have deadlock potential?  There are no other callers of the function holding the hints lock.


-- 
To view, visit https://gerrit.asterisk.org/1262
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: If268f1110290e502c73dd289c9e7e7b27bc8432f
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 11
Gerrit-Owner: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
Gerrit-HasComments: Yes



More information about the asterisk-code-review mailing list