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

[Steven Liu]

Steven Liu <lq at chinaffmpeg.org> 于2020年5月15日周五 下午6:02写道：
>
> 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".