Thanks Didier. I've posted the bugs. Regards, Josephine
On Mon, Feb 8, 2010 at 1:11 PM, Didier Barvaux < [email protected]> wrote: > Hi Josephine, > > > Looking at rtp_code_dynamic_rtp_part() in c_rtp.c, it appears that > > there's a small error in line 568: > > > > rohc_debugf(3, "send ts_stride = %04x on 4 bytes\n", > > ts_stride); > > > > Shouldn't it be > > > > " ... send ts_stride = %08x .. " ? > > Yes, that's a mistake. There should even be a '0x' prefix in front of > the '%08x' format string so that we know that it is a hexadecimal value. > > Moreover reading the source code around the debug trace you mention, > make me discover another problem. Thanks for that Josephine. > > The TS_STRIDE field must be encoded with SDVL according to §4.5.6 of > RFC3095, but the ROHC library does not. With SDVL encoding, the length > of the TS_STRIDE field may vary from 1 byte to 4 bytes. That's why one > reads "1-4 octets, if TSS = 1" along the TS_Stride field in §5.7.7.6 of > RFC3095. The ROHC decompressor is also affected by the problem. > > Josephine, could you please open a bug in the bugtracker of the ROHC > library for both problems ? Go to https://bugs.launchpad.net/rohc/+filebug > and answer the different questions. > > Regards, > > Didier Barvaux > Viveris Technologies > http://www.tech.viveris.com/opensource/ > > > _______________________________________________ > Mailing list: https://launchpad.net/~rohc <https://launchpad.net/%7Erohc> > Post to : [email protected] > Unsubscribe : https://launchpad.net/~rohc <https://launchpad.net/%7Erohc> > More help : https://help.launchpad.net/ListHelp >
_______________________________________________ Mailing list: https://launchpad.net/~rohc Post to : [email protected] Unsubscribe : https://launchpad.net/~rohc More help : https://help.launchpad.net/ListHelp
