[Asterisk-code-review] Change in asterisk[master]: bridge.c: NULL app causes crash during attended transfer

[Richard Mudgett (Code Review)]

Richard Mudgett has posted comments on this change.

Change subject: bridge.c: NULL app causes crash during attended transfer
......................................................................


Patch Set 3:

(1 comment)

https://gerrit.asterisk.org/#/c/134/3/main/bridge.c
File main/bridge.c:

Line 4476: 	/* There is a race condition that makes it possible for the app to be NULL.
         : 	   Retrieve the app first thing in order to give the greatest chance of
         : 	   having a value */
Now need to update this comment with the one from the other review.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5134b84c4524906d8148817719d76ffb306488ac
Gerrit-PatchSet: 3
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
Gerrit-Reviewer: Scott Griepentrog <sgriepentrog at digium.com>
Gerrit-HasComments: Yes