From: George Joseph <gjoseph@digium.com>
Date: Fri, 21 Sep 2018 20:23:34 +0000 (-0600)
Subject: channel.c:  Address stack overflow in does_id_conflict()
X-Git-Tag: 16.0.0~1^2~4
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=87227386befbd781a7df40e8e9c86cd5a352db27;p=thirdparty%2Fasterisk.git

channel.c:  Address stack overflow in does_id_conflict()

does_id_conflict() was passing a pointer to an int to a callback
that expected a pointer to a size_t.

Found by the Address Sanitizer.

Change-Id: I0ff542067eef63a14a60301654d65d34fe2ad503
---

diff --git a/main/channel.c b/main/channel.c
index c0e3cb43ec..74cd65f892 100644
--- a/main/channel.c
+++ b/main/channel.c
@@ -751,7 +751,7 @@ static int ast_channel_by_uniqueid_cb(void *obj, void *arg, void *data, int flag
 static int does_id_conflict(const char *uniqueid)
 {
 	struct ast_channel *conflict;
-	int length = 0;
+	size_t length = 0;
 
 	if (ast_strlen_zero(uniqueid)) {
 		return 0;