[FFmpeg-devel] [PATCH] avformat/url: check the result of the strrchr
Steven Liu
lingjiujianke at gmail.com
Fri May 15 14:30:11 EEST 2020
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".
More information about the ffmpeg-devel
mailing list