[FFmpeg-devel] [PATCH 2/2] rtsp: make ff_sdp_parse return value forwarded

Luca Barbato lu_zero
Fri Jan 28 15:46:51 CET 2011


On 01/28/2011 09:17 AM, Martin Storsj? wrote:
> On Fri, 28 Jan 2011, Luca Barbato wrote:
> 
>> the sdp demuxer did not forward it at all while the rtsp demuxer assumed
>> a single kind of error
>> ---
>>  libavformat/rtsp.c    |    3 ++-
>>  libavformat/rtspdec.c |    2 +-
>>  2 files changed, 3 insertions(+), 2 deletions(-)
>>
>> diff --git a/libavformat/rtsp.c b/libavformat/rtsp.c
>> index a56ff99..dec6995 100644
>> --- a/libavformat/rtsp.c
>> +++ b/libavformat/rtsp.c
>> @@ -1795,8 +1795,9 @@ static int sdp_read_header(AVFormatContext *s, AVFormatParameters *ap)
>>      }
>>      content[size] ='\0';
>>  
>> -    ff_sdp_parse(s, content);
>> +    err = ff_sdp_parse(s, content);
>>      av_free(content);
>> +    if (err) goto fail;
>>  
>>      /* open each RTP stream */
>>      for (i = 0; i < rt->nb_rtsp_streams; i++) {
>> diff --git a/libavformat/rtspdec.c b/libavformat/rtspdec.c
>> index 55996e3..f95f7c3 100644
>> --- a/libavformat/rtspdec.c
>> +++ b/libavformat/rtspdec.c
>> @@ -136,7 +136,7 @@ int ff_rtsp_setup_input_streams(AVFormatContext *s, RTSPMessageHeader *reply)
>>      ret = ff_sdp_parse(s, (const char *)content);
>>      av_freep(&content);
>>      if (ret < 0)
>> -        return AVERROR_INVALIDDATA;
>> +        return ret;
>>  
>>      return 0;
>>  }
>> -- 
>> 1.7.2.2
>>
> 
> Looks sane to me
> 

Queued, thanks for the help =)


-- 

Luca Barbato
Gentoo/linux
http://dev.gentoo.org/~lu_zero




More information about the ffmpeg-devel mailing list