[FFmpeg-devel] [PATCH] avformat/url: check the result of the strrchr

[Steven Liu]

> 2020年5月15日 下午6:26，Steven Liu <lq at chinaffmpeg.org> 写道：
> 
> 
> 
>> 2020年5月15日 下午6:01，Steven Liu <lq at chinaffmpeg.org> 写道：
>> 
>> because it need be check for success, is should not
>> change the old way if it failure.
> Change the comments: 
> because it need be checked for success status, it should not
> change the old way if it failure.
>> 
>> Signed-off-by: Steven Liu <lq at chinaffmpeg.org>
>> ---
>> libavformat/url.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>> 
>> diff --git a/libavformat/url.c b/libavformat/url.c
>> index 7cd9e0c705..6956f6dc10 100644
>> --- a/libavformat/url.c
>> +++ b/libavformat/url.c
>> @@ -182,7 +182,7 @@ void ff_make_absolute_url(char *buf, int size, const char *base,
>> 
>>   /* Remove the file name from the base url */
>>   sep = strrchr(buf, '/');
>> -    if (sep <= root)
>> +    if (sep && sep <= root)
>>       sep = root;
>> 
>>   if (sep)
>> -- 
>> 2.25.0
>> 
>> 
>> 
>> 
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel at ffmpeg.org
>> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>> 
>> To unsubscribe, visit link above, or email
>> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".
> 
> Thanks
> 
> Steven Liu

Ping

Thanks

Steven Liu