Re: [Live-devel] Race condition bug fix

2007-09-14 Thread Ross Finlayson
>when two DESCRIBE requests are pipelined, But that's not what you showed in the trace that you included in your earlier message. That trace showed DESCRIBE, SETUP, PLAY, DESCRIBE, SETUP, PLAY - all handled correctly - with only a trailing TEARDOWN not handled correctly (because the URL was ba

Re: [Live-devel] Race condition bug fix

2007-09-14 Thread Geoff Cleary
See embedded comment below. Geoff > -Original Message- > From: [EMAIL PROTECTED] [mailto:live-devel- > [EMAIL PROTECTED] On Behalf Of Ross Finlayson > Sent: Friday, September 14, 2007 10:49 AM > To: LIVE555 Streaming Media - development & use > Subject: Re: [Live

Re: [Live-devel] Race condition bug fix

2007-09-14 Thread Ross Finlayson
First, there is non't really a "race condition" going on here, because our server code is single-threaded. Also, you are using an old version of the "LIVE555 Streaming Media" code. You will need to upgrade to the latest version of the code. (No support is given for old versions of the code.)

[Live-devel] Race condition bug fix

2007-09-14 Thread Geoff Cleary
Ross I have found out more about what the client (Genetec V4.0) is doing. It is trying to pipeline requests for two Unicast streams. It expects the two streams to play concurrently, however the server assigns the same session IDs. What do you think standard behavior should be in this regard? I hav

Re: [Live-devel] Race condition bug fix

2007-09-13 Thread Ross Finlayson
I agree that in the case of two reads to build a request, my code does not pass the correct length, however there is still a problem. Sorry I did not explain it fully in my original mail. In our use of the server, the client often makes two requests for a pair of streams from the server. There

Re: [Live-devel] Race condition bug fix

2007-09-13 Thread Geoff Cleary
Behalf Of Ross Finlayson Sent: Wednesday, September 12, 2007 9:01 PM To: LIVE555 Streaming Media - development & use Subject: Re: [Live-devel] Race condition bug fix The following fix prevents a bug when two requests are processed in close proximity. No, the original code was correct.

Re: [Live-devel] Race condition bug fix

2007-09-12 Thread Ross Finlayson
The following fix prevents a bug when two requests are processed in close proximity. No, the original code was correct. It's possible that an incoming RTSP request will require more than one socket read in order to obtain all the data for the request. In that case, the length of the request

[Live-devel] Race condition bug fix

2007-09-12 Thread Geoff Cleary
The following fix prevents a bug when two requests are processed in close proximity. Should pass the parser the pointer to the new request not the address of the buffer. *** lib/live/liveMedia/RTSPServer.cpp 2007-06-15 14:44:10.811882441 -0400 --- ./lib/live/liveMedia/RTSPServer.cpp 2007-0