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

[Steven Liu]

> 2020年5月17日 上午10:35，Steven Liu <lq at chinaffmpeg.org> 写道：
> 
> 
> 
>> 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
>>> 
> 

will apply after 24 hours if no objections.
> Ping
> 

Thanks

Steven Liu