Merged revisions 154060 via svnmerge from
authorTilghman Lesher <tilghman@meg.abyt.es>
Mon, 3 Nov 2008 21:57:14 +0000 (21:57 +0000)
committerTilghman Lesher <tilghman@meg.abyt.es>
Mon, 3 Nov 2008 21:57:14 +0000 (21:57 +0000)
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r154060 | tilghman | 2008-11-03 15:48:21 -0600 (Mon, 03 Nov 2008) | 3 lines

  Remove the potential for a division by zero error.
  (Closes issue #13810)
........

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

main/rtp.c

index 8261048..33ea83d 100644 (file)
@@ -68,8 +68,8 @@ ASTERISK_FILE_VERSION(__FILE__, "$Revision$")
 
 static int dtmftimeout = DEFAULT_DTMF_TIMEOUT;
 
-static int rtpstart;                   /*!< First port for RTP sessions (set in rtp.conf) */
-static int rtpend;                     /*!< Last port for RTP sessions (set in rtp.conf) */
+static int rtpstart = 5000;     /*!< First port for RTP sessions (set in rtp.conf) */
+static int rtpend = 31000;      /*!< Last port for RTP sessions (set in rtp.conf) */
 static int rtpdebug;                   /*!< Are we debugging? */
 static int rtcpdebug;                  /*!< Are we debugging RTCP? */
 static int rtcpstats;                  /*!< Are we debugging RTCP? */
@@ -2457,7 +2457,7 @@ struct ast_rtp *ast_rtp_new_with_bindaddr(struct sched_context *sched, struct io
         * next one, cannot be enforced in presence of a NAT box because the
         * mapping is not under our control.
         */
-       x = (ast_random() % (rtpend-rtpstart)) + rtpstart;
+       x = (rtpend == rtpstart) ? rtpstart : (ast_random() % (rtpend - rtpstart)) + rtpstart;
        x = x & ~1;             /* make it an even number */
        startplace = x;         /* remember the starting point */
        /* this is constant across the loop */