Re: [libav-devel] [PATCH] hvcc: zero initialize the nal buffers past the last written byte

2018-02-22 Thread Luca Barbato

On 23/02/2018 04:17, James Almer wrote:

Bug-Id: 1116
Cc: libav-sta...@libav.org

Signed-off-by: James Almer 
---
  libavformat/hevc.c | 2 ++
  1 file changed, 2 insertions(+)

diff --git a/libavformat/hevc.c b/libavformat/hevc.c
index f8bfeebd3..1f8a7bb3c 100644
--- a/libavformat/hevc.c
+++ b/libavformat/hevc.c
@@ -657,6 +657,8 @@ static uint8_t *nal_unit_extract_rbsp(const uint8_t *src, 
uint32_t src_len,
  while (i < src_len)
  dst[len++] = src[i++];
  
+memset(dst + len, 0, AV_INPUT_BUFFER_PADDING_SIZE);

+
  *dst_len = len;
  return dst;
  }



Sounds good.

lu
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

[libav-devel] [PATCH] hvcc: zero initialize the nal buffers past the last written byte

2018-02-22 Thread James Almer
Bug-Id: 1116
Cc: libav-sta...@libav.org

Signed-off-by: James Almer 
---
 libavformat/hevc.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/libavformat/hevc.c b/libavformat/hevc.c
index f8bfeebd3..1f8a7bb3c 100644
--- a/libavformat/hevc.c
+++ b/libavformat/hevc.c
@@ -657,6 +657,8 @@ static uint8_t *nal_unit_extract_rbsp(const uint8_t *src, 
uint32_t src_len,
 while (i < src_len)
 dst[len++] = src[i++];
 
+memset(dst + len, 0, AV_INPUT_BUFFER_PADDING_SIZE);
+
 *dst_len = len;
 return dst;
 }
-- 
2.16.2

___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH] lavc/qsv: default PicStruct to progressive.

2018-02-22 Thread Luca Barbato

On 22/02/2018 05:57, Song, Ruiling wrote:

-Original Message-
From: libav-devel [mailto:libav-devel-boun...@libav.org] On Behalf Of Luca
Barbato
Sent: Monday, February 12, 2018 5:29 PM
To: libav-devel@libav.org
Subject: Re: [libav-devel] [PATCH] lavc/qsv: default PicStruct to progressive.

On 12/02/2018 09:37, Ruiling Song wrote:

the PicStruct is required by MediaSDK, so give a default value.
hwupload does not work without this.

Signed-off-by: Ruiling Song 
---
   libavutil/hwcontext_qsv.c | 1 +
   1 file changed, 1 insertion(+)

diff --git a/libavutil/hwcontext_qsv.c b/libavutil/hwcontext_qsv.c
index 9fa603c..73b5f24 100644
--- a/libavutil/hwcontext_qsv.c
+++ b/libavutil/hwcontext_qsv.c
@@ -313,6 +313,7 @@ static int qsv_init_surface(AVHWFramesContext *ctx,

mfxFrameSurface1 *surf)

   surf->Info.CropH  = ctx->height;
   surf->Info.FrameRateExtN  = 25;
   surf->Info.FrameRateExtD  = 1;
+surf->Info.PicStruct  = MFX_PICSTRUCT_PROGRESSIVE;

   return 0;
   }



Sure.

Can you help merge?



Merged, sorry for the delay :)

___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel