[Asterisk-code-review] manager: Fix build due to missing variable usage. (asterisk[master])

Matt Jordan asteriskteam at digium.com
Wed May 6 07:06:11 CDT 2015


Matt Jordan has submitted this change and it was merged.

Change subject: manager: Fix build due to missing variable usage.
......................................................................


manager: Fix build due to missing variable usage.

Change-Id: I26d4d2cb9cee924632ff59ef0b30a7e6a1e2b00d
---
M main/manager.c
1 file changed, 2 insertions(+), 1 deletion(-)

Approvals:
  Matt Jordan: Looks good to me, approved; Verified
  Corey Farrell: Looks good to me, but someone else must approve



diff --git a/main/manager.c b/main/manager.c
index 846f6e6..17a2738 100644
--- a/main/manager.c
+++ b/main/manager.c
@@ -8981,7 +8981,8 @@
 
 				ast_append_acl(var->name, var->value, &user->acl, &acl_error, &acl_subscription_flag);
 				if (acl_error) {
-					ast_log(LOG_ERROR, "Invalid ACL '%s' for manager user '%s' on line %d. Deleting user\n");
+					ast_log(LOG_ERROR, "Invalid ACL '%s' for manager user '%s' on line %d. Deleting user\n",
+						var->value, user->username, var->lineno);
 					user->keep = 0;
 				}
 			}  else if (!strcasecmp(var->name, "read") ) {

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I26d4d2cb9cee924632ff59ef0b30a7e6a1e2b00d
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>



More information about the asterisk-code-review mailing list