[asterisk-commits] qwell: branch 1.2 r45332 - /branches/1.2/channels/chan_skinny.c

asterisk-commits at lists.digium.com asterisk-commits at lists.digium.com
Tue Oct 17 10:50:36 MST 2006


Author: qwell
Date: Tue Oct 17 12:50:36 2006
New Revision: 45332

URL: http://svn.digium.com/view/asterisk?rev=45332&view=rev
Log:
Fix an integer signedness problem.


Modified:
    branches/1.2/channels/chan_skinny.c

Modified: branches/1.2/channels/chan_skinny.c
URL: http://svn.digium.com/view/asterisk/branches/1.2/channels/chan_skinny.c?rev=45332&r1=45331&r2=45332&view=diff
==============================================================================
--- branches/1.2/channels/chan_skinny.c (original)
+++ branches/1.2/channels/chan_skinny.c Tue Oct 17 12:50:36 2006
@@ -2863,6 +2863,10 @@
 			return -1;
 		}
 		dlen = letohl(*(int *)s->inbuf);
+		if (dlen < 0) {
+			ast_log(LOG_WARNING, "Skinny Client sent invalid data.\n");
+			return -1;
+		}
 		if (dlen+8 > sizeof(s->inbuf)) {
 			dlen = sizeof(s->inbuf) - 8;
 		}



More information about the asterisk-commits mailing list