From 0bb316de2832b94872ab3b7aa2e90b8430d9dd38 Mon Sep 17 00:00:00 2001 From: Doug Bailey Date: Thu, 10 May 2007 21:25:05 +0000 Subject: [PATCH] Added check for negative offset in cid spill to prevent infinite loops git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@63786 65c4cc65-6c06-0410-ace0-fbb531ad65f3 --- main/callerid.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/main/callerid.c b/main/callerid.c index 002666aa93..74b8d9200a 100644 --- a/main/callerid.c +++ b/main/callerid.c @@ -636,6 +636,12 @@ int callerid_feed(struct callerid_state *cid, unsigned char *ubuf, int len, int default: ast_log(LOG_NOTICE, "Unknown IE %d\n", cid->rawdata[x - 1]); } + if(0 > cid->rawdata[x]){ /* Negative offset in the CID Spill */ + ast_log(LOG_NOTICE, "IE %d has bad field length of %d at offset %d\n", cid->rawdata[x-1], cid->rawdata[x], x); + /* Try again */ + cid->sawflag = 0; + break; /* Exit the loop */ + } x += cid->rawdata[x]; x++; } -- 2.47.2