[asterisk-commits] Change in asterisk[master]: cdr_pgsql: Fix CLI "cdr show pgsql status" command.

[SVN commits to the Asterisk project]

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
---
M cdr/cdr_pgsql.c
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Matt Jordan: Looks good to me, approved; Verified
  Jared K. Smith: Looks good to me, but someone else must approve



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

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I584f0936bb01001336a468a55c1d05d79fe795d5
Gerrit-PatchSet: 2
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Rodrigo Ramirez Norambuena <decipher.hk at gmail.com>
Gerrit-Reviewer: Jared K. Smith <jaredsmith at jaredsmith.net>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>