[Ccrtp-devel] Bug report RTPDataQueue::getCurrentTimestamp()

[Guillaume Fraysse]

Hello everyone,

While having a go at ccRTP with a co-worker of mine, we had a little 
synchronization problem which led us to find a bug in the 
uint32 RTPDataQueue::getCurrentTimestamp() const (in queue.cpp) method :

The following line :
uint32 result = now.tv_usec - getInitialTime().tv_usec;

gives an incorrect result if getInitialTime().tv_usec > now.tv_usec as you 
expect to get an unsigned integer. 

We fixed it by removing the unsigned :
int32 result = now.tv_usec - getInitialTime().tv_usec;

And now it works better.

Hope this helps, please tell us if we got something wrong,
Cheers

Guillaume & Guillaume