[asterisk-commits] russell: trunk r76486 - in /trunk: ./ channels/chan_iax2.c

[SVN commits to the Asterisk project]

Author: russell
Date: Mon Jul 23 07:29:46 2007
New Revision: 76486

URL: http://svn.digium.com/view/asterisk?view=rev&rev=76486
Log:
Merged revisions 76485 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r76485 | russell | 2007-07-23 07:25:01 -0500 (Mon, 23 Jul 2007) | 6 lines

Use a signed integer for storing the number of bytes in the packet read from
the network.  Using an unsigned value here made it impossible to handle an
error returned from recvfrom().  Furthermore, in the case that recvfrom()
did return an error, this would cause a crash due to a heap overflow.
(closes issue #10265, reported by and fix suggested by timrobbins)

........

Modified:
    trunk/   (props changed)
    trunk/channels/chan_iax2.c

Propchange: trunk/
------------------------------------------------------------------------------
Binary property 'branch-1.4-merged' - no diff available.

Modified: trunk/channels/chan_iax2.c
URL: http://svn.digium.com/view/asterisk/trunk/channels/chan_iax2.c?view=diff&rev=76486&r1=76485&r2=76486
==============================================================================
--- trunk/channels/chan_iax2.c (original)
+++ trunk/channels/chan_iax2.c Mon Jul 23 07:29:46 2007
@@ -735,7 +735,7 @@
 	struct sockaddr_in iosin;
 	unsigned char readbuf[4096]; 
 	unsigned char *buf;
-	size_t buf_len;
+	ssize_t buf_len;
 	size_t buf_size;
 	int iofd;
 	time_t checktime;