Re: [libav-devel] [PATCH] vc1: Use codec ID from AVCodecContext while parsing frame header

2012-10-12 Thread Mashiat Sarker Shakkhar

On 10/11/2012 1:23 PM, Kostya Shishkov wrote:

On Thu, Oct 11, 2012 at 01:13:56PM -0400, Mashiat Sarker Shakkhar wrote:

This fixes a segfault with samples that I have (both of them MPEG-TS). Looks 
like
avctx-codec is not being set during parsing.
---
  libavcodec/vc1.c |2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavcodec/vc1.c b/libavcodec/vc1.c
index c7edc25..a8dd38a 100644
--- a/libavcodec/vc1.c
+++ b/libavcodec/vc1.c
@@ -576,7 +576,7 @@ int ff_vc1_parse_frame_header(VC1Context *v, GetBitContext* 
gb)

  if (v-finterpflag)
  v-interpfrm = get_bits1(gb);
-if (v-s.avctx-codec-id == AV_CODEC_ID_MSS2)
+if (v-s.avctx-codec_id == AV_CODEC_ID_MSS2)
  v-respic   =
  v-rangered =
  v-multires = get_bits(gb, 2) == 1;
--


probably OK


Someone's gotta push it. It's been OK'd and it's trivial.

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


[libav-devel] [PATCH] vc1: Use codec ID from AVCodecContext while parsing frame header

2012-10-11 Thread Mashiat Sarker Shakkhar
This fixes a segfault with samples that I have (both of them MPEG-TS). Looks 
like
avctx-codec is not being set during parsing.
---
 libavcodec/vc1.c |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavcodec/vc1.c b/libavcodec/vc1.c
index c7edc25..a8dd38a 100644
--- a/libavcodec/vc1.c
+++ b/libavcodec/vc1.c
@@ -576,7 +576,7 @@ int ff_vc1_parse_frame_header(VC1Context *v, GetBitContext* 
gb)
 
 if (v-finterpflag)
 v-interpfrm = get_bits1(gb);
-if (v-s.avctx-codec-id == AV_CODEC_ID_MSS2)
+if (v-s.avctx-codec_id == AV_CODEC_ID_MSS2)
 v-respic   =
 v-rangered =
 v-multires = get_bits(gb, 2) == 1;
-- 
1.7.9.5

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


Re: [libav-devel] [PATCH] vc1: Use codec ID from AVCodecContext while parsing frame header

2012-10-11 Thread Kostya Shishkov
On Thu, Oct 11, 2012 at 01:13:56PM -0400, Mashiat Sarker Shakkhar wrote:
 This fixes a segfault with samples that I have (both of them MPEG-TS). Looks 
 like
 avctx-codec is not being set during parsing.
 ---
  libavcodec/vc1.c |2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
 
 diff --git a/libavcodec/vc1.c b/libavcodec/vc1.c
 index c7edc25..a8dd38a 100644
 --- a/libavcodec/vc1.c
 +++ b/libavcodec/vc1.c
 @@ -576,7 +576,7 @@ int ff_vc1_parse_frame_header(VC1Context *v, 
 GetBitContext* gb)
  
  if (v-finterpflag)
  v-interpfrm = get_bits1(gb);
 -if (v-s.avctx-codec-id == AV_CODEC_ID_MSS2)
 +if (v-s.avctx-codec_id == AV_CODEC_ID_MSS2)
  v-respic   =
  v-rangered =
  v-multires = get_bits(gb, 2) == 1;
 -- 

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