[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:04:09 CDT 2015


Matt Jordan has uploaded a new change for review.

  https://gerrit.asterisk.org/108

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(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/08/108/1

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: newchange
Gerrit-Change-Id: I584f0936bb01001336a468a55c1d05d79fe795d5
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 11
Gerrit-Owner: Matt Jordan <mjordan at digium.com>
Gerrit-Reviewer: Rodrigo Ramirez Norambuena <decipher.hk at gmail.com>



More information about the asterisk-code-review mailing list