"Ronald S. Bultje" <[email protected]> writes:

> From: "Ronald S. Bultje" <[email protected]>
>
> Prevents crashes if the return value is used as an index in arrays
> without being checked. Before, it could reach -1.
> ---
>  libavcodec/golomb.h |    6 ++++--
>  1 files changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/libavcodec/golomb.h b/libavcodec/golomb.h
> index 5f720c0..ae3f530 100644
> --- a/libavcodec/golomb.h
> +++ b/libavcodec/golomb.h
> @@ -123,7 +123,7 @@ static inline int svq3_get_ue_golomb(GetBitContext *gb){

Why is the return type of this function signed in the first place?

-- 
Måns Rullgård
[email protected]
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to