Fixed bug in rtsp streaming caused by a signed - unsigned conversion.

[manupap1]

This patch solves a streaming issue with one of my AXIS encoders which sends SDES packets on the RTSP control stream.
In some case the value of contentLen can be badly reinterpreted and this may lead to a nearly infinit loop.

[connortechnology]

OOoh neat. Thanks for this.

[connortechnology]

Wow, those lines calculating paddedLen offend me. Useless adds and subtracts. Do you understand the values and calcs that are being done? Can you document them, and merge those two lines that calculate paddedLen and get rid of anything useless? I know the add and sub are only nanoseconds but really... a good documenting line help a lot.

[manupap1]

I am not a specialist in RSTP streaming, I can't help you efficiently about the documentation,
I tracked this bug because I am testing some modified code which consists in inplementing a plugin management feature based on nextime work.

[sogood007]

                 paddedLen = (((paddedLen-1)/4)+1)*4;

Is the question about this code? It seems to me it is to roundup to the nearest multiple by 4 number. Usually, I will code in this way instead
x = (x + (n - 1) ) / (n) * n ; // to round to the nearest multiple of n
The reason for the division and multiple is to take advantage of the integer division which will drop of the remainder. The pre addition of (n-1) is to account for the integer division so as the final effect of rounding up.

[connortechnology]

That makes sense. I will just add a comment saying so.