[Asterisk-code-review] main: Use ast strdup instead of strdup (asterisk[master])

Mark Michelson asteriskteam at digium.com
Fri Jan 8 14:46:51 CST 2016


Mark Michelson has submitted this change and it was merged.

Change subject: main: Use ast_strdup instead of strdup
......................................................................


main: Use ast_strdup instead of strdup

Fix compile error in main/utils.c because strdup was used in dummy_start

Change-Id: Id61a6cf4f3cbf235450441e10e7da101a6335793
---
M main/utils.c
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Mark Michelson: Looks good to me, approved
  Richard Mudgett: Looks good to me, but someone else must approve
  Anonymous Coward #1000019: Verified
  Joshua Colp: Looks good to me, but someone else must approve



diff --git a/main/utils.c b/main/utils.c
index 5cebed1..8a9f910 100644
--- a/main/utils.c
+++ b/main/utils.c
@@ -1208,7 +1208,7 @@
 
 	lock_info->thread_id = pthread_self();
 	lock_info->lwp = ast_get_tid();
-	lock_info->thread_name = strdup(a.name);
+	lock_info->thread_name = ast_strdup(a.name);
 
 	pthread_mutexattr_init(&mutex_attr);
 	pthread_mutexattr_settype(&mutex_attr, AST_MUTEX_KIND);

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Id61a6cf4f3cbf235450441e10e7da101a6335793
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Diederik de Groot <dkgroot at talon.nl>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>



More information about the asterisk-code-review mailing list