[Asterisk-code-review] Change in asterisk[11]: cdr_pgsql: Fix CLI "cdr show pgsql status" command.

Matt Jordan (Code Review) asteriskteam at digium.com
Tue Apr 14 14:38:47 CDT 2015


Matt Jordan has submitted this change and it was merged.

Change subject: cdr_pgsql: Fix CLI "cdr show pgsql status" command.
......................................................................


cdr_pgsql: Fix CLI "cdr show pgsql status" command.

The command always showed the usage information.

* Fix the error in command validation for CLI_SHOWUSAGE.

ASTERISK-24959 #close
Reported by: Rodrigo Ramirez Norambuena

Change-Id: I584f0936bb01001336a468a55c1d05d79fe795d5
(cherry picked from commit 23a180cade51e84b9def65b05759c3cb9feba225)
---
M cdr/cdr_pgsql.c
1 file changed, 1 insertion(+), 1 deletion(-)

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



diff --git a/cdr/cdr_pgsql.c b/cdr/cdr_pgsql.c
index 4b536ab..c7a6644 100644
--- a/cdr/cdr_pgsql.c
+++ b/cdr/cdr_pgsql.c
@@ -124,7 +124,7 @@
 		return NULL;
 	}
 
-	if (a->argc != 3)
+	if (a->argc != e->args)
 		return CLI_SHOWUSAGE;
 
 	if (connected) {

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I584f0936bb01001336a468a55c1d05d79fe795d5
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 11
Gerrit-Owner: Matt Jordan <mjordan at digium.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>



More information about the asterisk-code-review mailing list