[Asterisk-code-review] res_indications: Fix indications remove command autocomplete (...asterisk[master])

[George Joseph]

George Joseph has submitted this change and it was merged. ( https://gerrit.asterisk.org/c/asterisk/+/11300 )

Change subject: res_indications: Fix indications remove command autocomplete
......................................................................

res_indications: Fix indications remove command autocomplete

We changed the validation of autocomplete parameter in the "indications
remove" command to avoid continue the execution of the command after
asking for autocomplete out of range parameters.

ASTERISK-28391
Reported by: lmendes86

Change-Id: I92b24131fd02f2e3c7fec966eea6f7a663310d40
---
M main/indications.c
1 file changed, 3 insertions(+), 1 deletion(-)

Approvals:
  Sean Bright: Looks good to me, but someone else must approve
  George Joseph: Looks good to me, approved; Approved for Submit



diff --git a/main/indications.c b/main/indications.c
index 1f77ca1..c074844 100644
--- a/main/indications.c
+++ b/main/indications.c
@@ -764,9 +764,11 @@
 	case CLI_GENERATE:
 		if (a->pos == 2) {
 			return complete_country(a);
-		} else if (a->pos == 3) {
+		}
+		if (a->pos == 3) {
 			return complete_indications(a);
 		}
+		return NULL;
 	}
 
 	if (a->argc != 3 && a->argc != 4) {

-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/11300
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: I92b24131fd02f2e3c7fec966eea6f7a663310d40
Gerrit-Change-Number: 11300
Gerrit-PatchSet: 1
Gerrit-Owner: Lucas Mendes <lucas.mendes at wearespindle.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Sean Bright <sean.bright at gmail.com>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20190422/4d8310c9/attachment.html>