[FFmpeg-devel] [PATCH 4/6] avutil/common: warn about possible move of the data pointer after the last 0 byte in GET_UTF8

[Paul B Mahol]

lgtm

On 1/30/20, Marton Balint <cus at passwd.hu> wrote:
> Signed-off-by: Marton Balint <cus at passwd.hu>
> ---
>  libavutil/common.h | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/libavutil/common.h b/libavutil/common.h
> index e6f076a13c..142ff9abe7 100644
> --- a/libavutil/common.h
> +++ b/libavutil/common.h
> @@ -373,7 +373,9 @@ static av_always_inline av_const int
> av_parity_c(uint32_t v)
>   * @param GET_BYTE Expression reading one byte from the input.
>   *                 Evaluated up to 7 times (4 for the currently
>   *                 assigned Unicode range).  With a memory buffer
> - *                 input, this could be *ptr++.
> + *                 input, this could be *ptr++, or if you want to make sure
> + *                 that *ptr stops at the end of a NULL terminated string
> then
> + *                 *ptr ? *ptr++ : 0
>   * @param ERROR    Expression to be evaluated on invalid input,
>   *                 typically a goto statement.
>   *
> --
> 2.16.4
>
> _______________________________________________
> 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".