[Asterisk-code-review] app queue: Silence GCC 8 compiler warning (asterisk[13])

Sean Bright asteriskteam at digium.com
Tue Aug 21 13:51:36 CDT 2018


Sean Bright has uploaded this change for review. ( https://gerrit.asterisk.org/9980


Change subject: app_queue: Silence GCC 8 compiler warning
......................................................................

app_queue: Silence GCC 8 compiler warning

Change-Id: I18577590da46829c1ea7d8b82e41d69f105baa10
---
M apps/app_queue.c
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/80/9980/1

diff --git a/apps/app_queue.c b/apps/app_queue.c
index 31580b4..12f09ec 100644
--- a/apps/app_queue.c
+++ b/apps/app_queue.c
@@ -9932,7 +9932,7 @@
 	case 6: /* only one possible match, "penalty" */
 		return state == 0 ? ast_strdup("penalty") : NULL;
 	case 7:
-		if (state < 100) {      /* 0-99 */
+		if (0 <= state && state < 100) {      /* 0-99 */
 			char *num;
 			if ((num = ast_malloc(3))) {
 				sprintf(num, "%d", state);

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

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-MessageType: newchange
Gerrit-Change-Id: I18577590da46829c1ea7d8b82e41d69f105baa10
Gerrit-Change-Number: 9980
Gerrit-PatchSet: 1
Gerrit-Owner: Sean Bright <sean.bright at gmail.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180821/d93be25e/attachment.html>


More information about the asterisk-code-review mailing list