stun: correct attribute string padding to match rfc
authorScott Griepentrog <sgriepentrog@digium.com>
Fri, 14 Nov 2014 15:52:21 +0000 (15:52 +0000)
committerScott Griepentrog <sgriepentrog@digium.com>
Fri, 14 Nov 2014 15:52:21 +0000 (15:52 +0000)
commitba811ae1c3f80cb0d691fbe0dbdac2db1d4d960a
tree22feeedc09aab2cdfd39ad58efc26ad5f41255c3
parent2d9471ab1f18d12674ff29392f1fac2ee9129518
stun: correct attribute string padding to match rfc

When sending the USERNAME attribute in an RTP STUN
response, the implementation in append_attr_string
passed the actual length, instead of padding it up
to a multiple of four bytes as required by the RFC
3489.  This change adds separate variables for the
string and padded attributed lengths, and performs
padding correctly.

Reported by: Thomas Arimont
Review: https://reviewboard.asterisk.org/r/4139/
........

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

Merged revisions 427875 from http://svn.asterisk.org/svn/asterisk/branches/12
........

Merged revisions 427876 from http://svn.asterisk.org/svn/asterisk/branches/13

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