[Asterisk-code-review] main/cli: Use proper string methods to check existence of co... (asterisk[master])

Matt Jordan asteriskteam at digium.com
Fri Nov 20 22:02:20 CST 2015


Matt Jordan has uploaded a new change for review.

  https://gerrit.asterisk.org/1672

Change subject: main/cli: Use proper string methods to check existence of context/exten/app
......................................................................

main/cli: Use proper string methods to check existence of context/exten/app

Because the context, extension, and application are stored in stringfields,
checking for them being NULL doesn't work so well. This patch uses the
appropriate string library call, ast_strlen_zero, to see if there is a value
in the context/exten/app values.

Change-Id: Ie09623bfdf35f5a8d3b23dd596647fe3c97b9a23
---
M main/cli.c
1 file changed, 4 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/72/1672/1

diff --git a/main/cli.c b/main/cli.c
index b3d4cac..9173055 100644
--- a/main/cli.c
+++ b/main/cli.c
@@ -1076,10 +1076,12 @@
 				char locbuf[40] = "(None)";
 				char appdata[40] = "(None)";
 
-				if (!cs->context && !cs->exten)
+				if (!ast_strlen_zero(cs->context) && !ast_strlen_zero(cs->exten)) {
 					snprintf(locbuf, sizeof(locbuf), "%s@%s:%d", cs->exten, cs->context, cs->priority);
-				if (cs->appl)
+				}
+				if (!ast_strlen_zero(cs->appl)) {
 					snprintf(appdata, sizeof(appdata), "%s(%s)", cs->appl, S_OR(cs->data, ""));
+				}
 				ast_cli(a->fd, FORMAT_STRING, cs->name, locbuf, ast_state2str(cs->state), appdata);
 			}
 		}

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie09623bfdf35f5a8d3b23dd596647fe3c97b9a23
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Matt Jordan <mjordan at digium.com>



More information about the asterisk-code-review mailing list