[FFmpeg-devel] [PATCH] libavformat/mpeg.c: Initialize the buffer uses to read the PTS.

[Michael Niedermayer]

On Thu, Aug 13, 2020 at 02:21:55PM -0700, Thierry Foucu wrote:
> Fixed an Use-of-uninitialized-value
> ---
>  libavformat/mpeg.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavformat/mpeg.c b/libavformat/mpeg.c
> index 265b2bd1ad..15a768e6e8 100644
> --- a/libavformat/mpeg.c
> +++ b/libavformat/mpeg.c
> @@ -146,7 +146,7 @@ static int mpegps_read_header(AVFormatContext *s)
>  
>  static int64_t get_pts(AVIOContext *pb, int c)
>  {
> -    uint8_t buf[5];
> +    uint8_t buf[5] = {};
>  
>      buf[0] = c < 0 ? avio_r8(pb) : c;
>      avio_read(pb, buf + 1, 4);

this avoids the uninitialized use but it doenst return the correct value
ill post a different solution

thx
[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Dictatorship naturally arises out of democracy, and the most aggravated
form of tyranny and slavery out of the most extreme liberty. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20200815/dc9ec235/attachment.sig>