brillcodec: initialized AVPacket variable. 28/29228/1
authorKitae Kim <kt920.kim@samsung.com>
Tue, 21 Oct 2014 01:42:14 +0000 (10:42 +0900)
committerKitae Kim <kt920.kim@samsung.com>
Thu, 23 Oct 2014 04:10:10 +0000 (13:10 +0900)
there is a case when AVPacket variable can be used without initialization.

Change-Id: I973cb1f39ae2c7c5fc40cefe99751587a33373d5
Signed-off-by: Kitae Kim <kt920.kim@samsung.com>
tizen/src/hw/pci/maru_brillcodec.c

index cb6585d..fce5020 100644 (file)
@@ -1247,7 +1247,6 @@ static bool codec_decode_video(MaruBrillCodecState *s, int ctx_id, void *data_bu
     avpkt.data = inbuf;
     avpkt.size = inbuf_size;
 
-
     avctx = CONTEXT(s, ctx_id).avctx;
     picture = CONTEXT(s, ctx_id).frame;
     if (!avctx) {
@@ -1635,6 +1634,10 @@ static bool codec_encode_audio(MaruBrillCodecState *s, int ctx_id, void *data_bu
         // return false;
     }
 
+    av_init_packet(&avpkt);
+    avpkt.data = NULL;
+    avpkt.size = 0;
+
     avctx = CONTEXT(s, ctx_id).avctx;
     if (!avctx) {
         ERR("encode_audio. %d of Context is NULL\n", ctx_id);
@@ -1671,10 +1674,6 @@ static bool codec_encode_audio(MaruBrillCodecState *s, int ctx_id, void *data_bu
                         resample_sample_fmt);
 
                 if (resample_frame) {
-                    av_init_packet(&avpkt);
-                    avpkt.data = NULL;
-                    avpkt.size = 0;
-
                     ret = avcodec_encode_audio2(avctx, &avpkt, (const AVFrame *)resample_frame, &got_pkt);
                     TRACE("encode audio. ret %d got_pkt %d avpkt.size %d frame_number %d\n",
                         ret, got_pkt, avpkt.size, avctx->frame_number);