Re: [Live-devel] Bug / patch in parseRTSPRequestString

2008-11-13 Thread Sébastien Escudier
Quoting Ross Finlayson <[EMAIL PROTECTED]>: > Thanks, yes. I have now released a new version (2008.11.13) of the > code that fixes this problem. Please let me know if this works OK > for you now. Thank you. I can't tell you for sure, because I don't know how to reproduce this. But if this happe

Re: [Live-devel] Bug / patch in parseRTSPRequestString

2008-11-13 Thread Ross Finlayson
I looked into the code, and here is some comments : - The following line appears to be useless, as j is set to i+1 next line : while (j < reqStrSize && (reqStr[j] == ' ' || reqStr[j] == '\t')) ++j; // skip over any additional white space for (j = i+1; (int)j < (int)(reqStrSize-8); ++j) {

Re: [Live-devel] Bug / patch in parseRTSPRequestString

2008-11-13 Thread Sébastien Escudier
Sorry, I forgot to show you this from gdb, which confirm k1 was probably 0. (gdb) p k $4 = 80 (gdb) p k1 $5 = 4294967295 (gdb) p k2 $6 = 4294967281 (gdb) p k3 $7 = (gdb) p n $8 = 4120 (gdb) p reqStrSize $9 = 167 ___ live-devel mailing list live-devel@l

[Live-devel] Bug / patch in parseRTSPRequestString

2008-11-13 Thread Sébastien Escudier
Hi I had a segfault in parseRTSPRequestString. To see gdb details, please go te the end of the mail. I looked into the code, and here is some comments : - The following line appears to be useless, as j is set to i+1 next line : while (j < reqStrSize && (reqStr[j] == ' ' || reqStr[j] == '\t')