Hi Ross,

Even if I have not identify what's wrong with the GET_PARAMETER response sent 
by the camera, I confirmed that your modification fix the problem.

Thanks,

                Michel.

[@@THALES GROUP RESTRICTED@@]

De : [email protected] 
[mailto:[email protected]] De la part de Ross Finlayson
Envoyé : vendredi 8 février 2013 18:48
À : LIVE555 Streaming Media - development & use
Objet : Re: [Live-devel] SEGV handling GET_PARAMETER response.

In this context we are in RTSPClient.cpp around 1555 :

      // If we saw a "Content-Length:" header, then make sure that we have the 
amount of data that it specified:
      unsigned bodyOffset = nextLineStart - headerDataCopy;
      bodyStart = &fResponseBuffer[bodyOffset];
      numBodyBytes = fResponseBytesAlreadySeen - bodyOffset;
      if (contentLength > numBodyBytes) {

Gdb says that nextLineStart is NULL, bodyOffset is a big number, and finally 
bodyStart point a non allocated memory (that raise a SEGV).

Do you think if is possible to add check nextLineStart before using it :

Instead, please replace the line
            unsigned bodyOffset = nextLineStart - headerDataCopy;
with
            unsigned bodyOffset = nextLineStart == NULL ? 
fResponseBytesAlreadySeen : nextLineStart - headerDataCopy;

If you still have a problem after this change, please let me know.  (Otherwise 
I'll make that change in the next release of the software.)

Ross Finlayson
Live Networks, Inc.
http://www.live555.com/

_______________________________________________
live-devel mailing list
[email protected]
http://lists.live555.com/mailman/listinfo/live-devel

Reply via email to