Merged revisions 76485 via svnmerge from
authorRussell Bryant <russell@russellbryant.com>
Mon, 23 Jul 2007 12:29:46 +0000 (12:29 +0000)
committerRussell Bryant <russell@russellbryant.com>
Mon, 23 Jul 2007 12:29:46 +0000 (12:29 +0000)
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)

........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@76486 65c4cc65-6c06-0410-ace0-fbb531ad65f3

channels/chan_iax2.c

index 05c2306..33d3b81 100644 (file)
@@ -735,7 +735,7 @@ struct iax2_thread {
        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;