[Asterisk-code-review] core unreal.c: Add clarification comment about channel ref. (asterisk[13])

Anonymous Coward asteriskteam at digium.com
Thu Mar 31 00:48:47 CDT 2016


Anonymous Coward #1000019 has submitted this change and it was merged.

Change subject: core_unreal.c: Add clarification comment about channel ref.
......................................................................


core_unreal.c: Add clarification comment about channel ref.

Change-Id: I0be0627260cd8d6b6c3cc345949dcfdf32eff1f3
---
M main/core_unreal.c
1 file changed, 2 insertions(+), 0 deletions(-)

Approvals:
  Anonymous Coward #1000019: Verified
  Joshua Colp: Looks good to me, approved
  George Joseph: Looks good to me, but someone else must approve



diff --git a/main/core_unreal.c b/main/core_unreal.c
index 028e371..fc311ec 100644
--- a/main/core_unreal.c
+++ b/main/core_unreal.c
@@ -808,9 +808,11 @@
 		return -1;
 	}
 
+	/* The bridge thread now controls the chan ref from the ast_unreal_pvt */
 	ao2_lock(p);
 	ast_set_flag(p, AST_UNREAL_CARETAKER_THREAD);
 	ao2_unlock(p);
+
 	ast_channel_unref(chan);
 
 	return 0;

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I0be0627260cd8d6b6c3cc345949dcfdf32eff1f3
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Richard Mudgett <rmudgett at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: George Joseph <george.joseph at fairview5.com>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>



More information about the asterisk-code-review mailing list