Fix an issue where a caller to ast_write on a MulticastRTP channel would determine...
authorJoshua Colp <jcolp@digium.com>
Tue, 25 Sep 2012 12:12:20 +0000 (12:12 +0000)
committerJoshua Colp <jcolp@digium.com>
Tue, 25 Sep 2012 12:12:20 +0000 (12:12 +0000)
commitcdcbffeed0a1347e385364fe83e04f7bd9b543e6
tree772790b5b03ec89052d1cd5c47bda025dfb64bd3
parentda8c22fe451e43d0de02d6b74f74b20f3b921c20
Fix an issue where a caller to ast_write on a MulticastRTP channel would determine it failed when in reality it did not.

When sending RTP packets via multicast the amount of data sent is stored in a variable and returned
from the write function. This is incorrect as any non-zero value returned is considered a failure while
a return value of 0 is success. For callers (such as ast_streamfile) that checked the return value
they would have considered it a failure when in reality nothing went wrong and it was actually a success.

The write function for the multicast RTP engine now returns -1 on failure and 0 on success, as it should.

(closes issue ASTERISK-17254)
Reported by: wybecom
........

Merged revisions 373550 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........

Merged revisions 373551 from http://svn.asterisk.org/svn/asterisk/branches/10
........

Merged revisions 373552 from http://svn.asterisk.org/svn/asterisk/branches/11

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