[Home]

Summary:ASTERISK-16210: [patch] rt(c)p set debug ip takes wrong argument
Reporter:Walter Doekes (wdoekes)Labels:
Date Opened:2010-06-04 08:16:55Date Closed:2010-06-30 12:32:23
Priority:TrivialRegression?No
Status:Closed/CompleteComponents:Core/RTP
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) ast1628-rtp-set-debug-ip.patch
( 1) astsvn-rtp-set-debug-ip.patch
Description:Hi,

CLI> rtp set debug ip 1.2.3.4
Lookup failed for 'ip'
Command 'rtp set debug ip 1.2.3.4' failed.

CLI> rtcp set debug ip 1.2.3.4
Lookup failed for 'ip'
Command 'rtcp set debug ip 1.2.3.4' failed.


See the attached patches.

Regards,
Walter Doekes
OSSO B.V.
Comments:By: Walter Doekes (wdoekes) 2010-06-04 08:19:30

Oops, haha. It was already fixed in SVN, but not in 1.6.2.8.

Oh well. The svn patch reduces the differences between the two branches then :P
By: David Woolley (davidw) 2010-06-04 08:24:41

I thought this was fixed by ASTERISK-14648

That only seems to have failed to apply for 1.6.2.

(Collided with previous comment.)


By: Leif Madsen (lmadsen) 2010-06-09 09:49:00

I wonder why mvanbaak blocked this initially? He blocked it specifically for 1.6.1 and 1.6.2 it looks like, but I have no idea why...
By: Leif Madsen (lmadsen) 2010-06-09 09:49:33

Is there a particular reason you blocked this change in 1.6.2?
By: Paul Belanger (pabelanger) 2010-06-25 08:52:51

Ping!  Any reason this cannot go into 1.6.2?
By: Digium Subversion (svnbot) 2010-06-30 12:28:04

Repository: asterisk
Revision: 273233

U   trunk/res/res_rtp_asterisk.c

------------------------------------------------------------------------
r273233 | pabelanger | 2010-06-30 12:28:03 -0500 (Wed, 30 Jun 2010) | 11 lines

Fix rt(c)p set debug ip taking wrong argument

Also clean up some coding errors.

(closes issue ASTERISK-16210)
Reported by: wdoekes
Patches:
     astsvn-rtp-set-debug-ip.patch uploaded by wdoekes (license 717)
Tested by: wdoekes, pabelanger


------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=273233
By: Digium Subversion (svnbot) 2010-06-30 12:32:23

Repository: asterisk
Revision: 273234

_U  branches/1.6.2/
U   branches/1.6.2/main/rtp.c

------------------------------------------------------------------------
r273234 | pabelanger | 2010-06-30 12:32:22 -0500 (Wed, 30 Jun 2010) | 17 lines

Merged revisions 273233 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
 r273233 | pabelanger | 2010-06-30 13:28:04 -0400 (Wed, 30 Jun 2010) | 11 lines
 
 Fix rt(c)p set debug ip taking wrong argument
 
 Also clean up some coding errors.
 
 (closes issue ASTERISK-16210)
 Reported by: wdoekes
 Patches:
       astsvn-rtp-set-debug-ip.patch uploaded by wdoekes (license 717)
 Tested by: wdoekes, pabelanger
........

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=273234