libavcodec/smacker.c:311: error: Null Dereference
  pointer `smk->full_tbl` last assigned on line 310 could be null and is dereferenced at line 311, column 9.

libavcodec/smacker.c:279:1: start of procedure decode_header_trees()
277. }
278. 
279. static int decode_header_trees(SmackVContext *smk) {
     ^
280.     GetBitContext gb;
281.     int mmap_size, mclr_size, full_size, type_size;

libavcodec/smacker.c:283:5: 
281.     int mmap_size, mclr_size, full_size, type_size;
282. 
283.     mmap_size = AV_RL32(smk->avctx->extradata);
         ^
284.     mclr_size = AV_RL32(smk->avctx->extradata + 4);
285.     full_size = AV_RL32(smk->avctx->extradata + 8);

libavcodec/smacker.c:284:5: 
282. 
283.     mmap_size = AV_RL32(smk->avctx->extradata);
284.     mclr_size = AV_RL32(smk->avctx->extradata + 4);
         ^
285.     full_size = AV_RL32(smk->avctx->extradata + 8);
286.     type_size = AV_RL32(smk->avctx->extradata + 12);

libavcodec/smacker.c:285:5: 
283.     mmap_size = AV_RL32(smk->avctx->extradata);
284.     mclr_size = AV_RL32(smk->avctx->extradata + 4);
285.     full_size = AV_RL32(smk->avctx->extradata + 8);
         ^
286.     type_size = AV_RL32(smk->avctx->extradata + 12);
287. 

libavcodec/smacker.c:286:5: 
284.     mclr_size = AV_RL32(smk->avctx->extradata + 4);
285.     full_size = AV_RL32(smk->avctx->extradata + 8);
286.     type_size = AV_RL32(smk->avctx->extradata + 12);
         ^
287. 
288.     init_get_bits(&gb, smk->avctx->extradata + 16, (smk->avctx->extradata_size - 16) * 8);

libavcodec/smacker.c:288:5: 
286.     type_size = AV_RL32(smk->avctx->extradata + 12);
287. 
288.     init_get_bits(&gb, smk->avctx->extradata + 16, (smk->avctx->extradata_size - 16) * 8);
         ^
289. 
290.     if(!get_bits1(&gb)) {

libavcodec/get_bits.h:306:1: start of procedure init_get_bits()
  304.  * responsible for checking for the buffer end yourself (take advantage of the padding)!
  305.  */
  306. static inline void init_get_bits(GetBitContext *s,
       ^
  307.                    const uint8_t *buffer, int bit_size)
  308. {

libavcodec/get_bits.h:309:5: 
  307.                    const uint8_t *buffer, int bit_size)
  308. {
  309.     int buffer_size = (bit_size+7)>>3;
           ^
  310.     if (buffer_size < 0 || bit_size < 0) {
  311.         buffer_size = bit_size = 0;

libavcodec/get_bits.h:310:9: Taking false branch
  308. {
  309.     int buffer_size = (bit_size+7)>>3;
  310.     if (buffer_size < 0 || bit_size < 0) {
               ^
  311.         buffer_size = bit_size = 0;
  312.         buffer = NULL;

libavcodec/get_bits.h:310:28: Taking false branch
  308. {
  309.     int buffer_size = (bit_size+7)>>3;
  310.     if (buffer_size < 0 || bit_size < 0) {
                                  ^
  311.         buffer_size = bit_size = 0;
  312.         buffer = NULL;

libavcodec/get_bits.h:315:5: 
  313.     }
  314. 
  315.     s->buffer       = buffer;
           ^
  316.     s->size_in_bits = bit_size;
  317.     s->buffer_end   = buffer + buffer_size;

libavcodec/get_bits.h:316:5: 
  314. 
  315.     s->buffer       = buffer;
  316.     s->size_in_bits = bit_size;
           ^
  317.     s->buffer_end   = buffer + buffer_size;
  318.     s->index        = 0;

libavcodec/get_bits.h:317:5: 
  315.     s->buffer       = buffer;
  316.     s->size_in_bits = bit_size;
  317.     s->buffer_end   = buffer + buffer_size;
           ^
  318.     s->index        = 0;
  319. }

libavcodec/get_bits.h:318:5: 
  316.     s->size_in_bits = bit_size;
  317.     s->buffer_end   = buffer + buffer_size;
  318.     s->index        = 0;
           ^
  319. }
  320. 

libavcodec/get_bits.h:319:1: return from a call to init_get_bits
  317.     s->buffer_end   = buffer + buffer_size;
  318.     s->index        = 0;
  319. }
       ^
  320. 
  321. static inline void align_get_bits(GetBitContext *s)

libavcodec/smacker.c:290:9: 
288.     init_get_bits(&gb, smk->avctx->extradata + 16, (smk->avctx->extradata_size - 16) * 8);
289. 
290.     if(!get_bits1(&gb)) {
             ^
291.         av_log(smk->avctx, AV_LOG_INFO, "Skipping MMAP tree\n");
292.         smk->mmap_tbl = av_malloc(sizeof(int) * 2);

libavcodec/get_bits.h:230:1: start of procedure get_bits1()
  228. }
  229. 
  230. static inline unsigned int get_bits1(GetBitContext *s){
       ^
  231.     unsigned int index = s->index;
  232.     uint8_t result = s->buffer[index>>3];

libavcodec/get_bits.h:231:5: 
  229. 
  230. static inline unsigned int get_bits1(GetBitContext *s){
  231.     unsigned int index = s->index;
           ^
  232.     uint8_t result = s->buffer[index>>3];
  233. #ifdef ALT_BITSTREAM_READER_LE

libavcodec/get_bits.h:232:5: 
  230. static inline unsigned int get_bits1(GetBitContext *s){
  231.     unsigned int index = s->index;
  232.     uint8_t result = s->buffer[index>>3];
           ^
  233. #ifdef ALT_BITSTREAM_READER_LE
  234.     result >>= index & 7;

libavcodec/get_bits.h:234:5: 
  232.     uint8_t result = s->buffer[index>>3];
  233. #ifdef ALT_BITSTREAM_READER_LE
  234.     result >>= index & 7;
           ^
  235.     result &= 1;
  236. #else

libavcodec/get_bits.h:235:5: 
  233. #ifdef ALT_BITSTREAM_READER_LE
  234.     result >>= index & 7;
  235.     result &= 1;
           ^
  236. #else
  237.     result <<= index & 7;

libavcodec/get_bits.h:240:5: 
  238.     result >>= 8 - 1;
  239. #endif
  240.     index++;
           ^
  241.     s->index = index;
  242. 

libavcodec/get_bits.h:241:5: 
  239. #endif
  240.     index++;
  241.     s->index = index;
           ^
  242. 
  243.     return result;

libavcodec/get_bits.h:243:5: 
  241.     s->index = index;
  242. 
  243.     return result;
           ^
  244. }
  245. 

libavcodec/get_bits.h:244:1: return from a call to get_bits1
  242. 
  243.     return result;
  244. }
       ^
  245. 
  246. static inline unsigned int show_bits1(GetBitContext *s){

libavcodec/smacker.c:290:9: Taking true branch
288.     init_get_bits(&gb, smk->avctx->extradata + 16, (smk->avctx->extradata_size - 16) * 8);
289. 
290.     if(!get_bits1(&gb)) {
             ^
291.         av_log(smk->avctx, AV_LOG_INFO, "Skipping MMAP tree\n");
292.         smk->mmap_tbl = av_malloc(sizeof(int) * 2);

libavcodec/smacker.c:291:9: Skipping av_log(): empty list of specs
289. 
290.     if(!get_bits1(&gb)) {
291.         av_log(smk->avctx, AV_LOG_INFO, "Skipping MMAP tree\n");
             ^
292.         smk->mmap_tbl = av_malloc(sizeof(int) * 2);
293.         smk->mmap_tbl[0] = 0;

libavcodec/smacker.c:292:9: 
290.     if(!get_bits1(&gb)) {
291.         av_log(smk->avctx, AV_LOG_INFO, "Skipping MMAP tree\n");
292.         smk->mmap_tbl = av_malloc(sizeof(int) * 2);
             ^
293.         smk->mmap_tbl[0] = 0;
294.         smk->mmap_last[0] = smk->mmap_last[1] = smk->mmap_last[2] = 1;

libavutil/mem.c:64:1: start of procedure av_malloc()
  62.    linker will do it automatically. */
  63. 
  64. void *av_malloc(size_t size)
      ^
  65. {
  66.     void *ptr = NULL;

libavutil/mem.c:66:5: 
  64. void *av_malloc(size_t size)
  65. {
  66.     void *ptr = NULL;
          ^
  67. #if CONFIG_MEMALIGN_HACK
  68.     long diff;

libavutil/mem.c:72:8: Taking false branch
  70. 
  71.     /* let's disallow possible ambiguous cases */
  72.     if(size > (INT_MAX-32) )
             ^
  73.         return NULL;
  74. 

libavutil/mem.c:83:9: Taking false branch
  81.     ((char*)ptr)[-1]= diff;
  82. #elif HAVE_POSIX_MEMALIGN
  83.     if (posix_memalign(&ptr,32,size))
              ^
  84.         ptr = NULL;
  85. #elif HAVE_MEMALIGN

libavutil/mem.c:114:5: 
  112.     ptr = malloc(size);
  113. #endif
  114.     return ptr;
           ^
  115. }
  116. 

libavutil/mem.c:115:1: return from a call to av_malloc
  113. #endif
  114.     return ptr;
  115. }
       ^
  116. 
  117. void *av_realloc(void *ptr, size_t size)

libavcodec/smacker.c:293:9: 
291.         av_log(smk->avctx, AV_LOG_INFO, "Skipping MMAP tree\n");
292.         smk->mmap_tbl = av_malloc(sizeof(int) * 2);
293.         smk->mmap_tbl[0] = 0;
             ^
294.         smk->mmap_last[0] = smk->mmap_last[1] = smk->mmap_last[2] = 1;
295.     } else {

libavcodec/smacker.c:294:9: 
292.         smk->mmap_tbl = av_malloc(sizeof(int) * 2);
293.         smk->mmap_tbl[0] = 0;
294.         smk->mmap_last[0] = smk->mmap_last[1] = smk->mmap_last[2] = 1;
             ^
295.     } else {
296.         if (smacker_decode_header_tree(smk, &gb, &smk->mmap_tbl, smk->mmap_last, mmap_size))

libavcodec/smacker.c:299:9: 
297.             return -1;
298.     }
299.     if(!get_bits1(&gb)) {
             ^
300.         av_log(smk->avctx, AV_LOG_INFO, "Skipping MCLR tree\n");
301.         smk->mclr_tbl = av_malloc(sizeof(int) * 2);

libavcodec/get_bits.h:230:1: start of procedure get_bits1()
  228. }
  229. 
  230. static inline unsigned int get_bits1(GetBitContext *s){
       ^
  231.     unsigned int index = s->index;
  232.     uint8_t result = s->buffer[index>>3];

libavcodec/get_bits.h:231:5: 
  229. 
  230. static inline unsigned int get_bits1(GetBitContext *s){
  231.     unsigned int index = s->index;
           ^
  232.     uint8_t result = s->buffer[index>>3];
  233. #ifdef ALT_BITSTREAM_READER_LE

libavcodec/get_bits.h:232:5: 
  230. static inline unsigned int get_bits1(GetBitContext *s){
  231.     unsigned int index = s->index;
  232.     uint8_t result = s->buffer[index>>3];
           ^
  233. #ifdef ALT_BITSTREAM_READER_LE
  234.     result >>= index & 7;

libavcodec/get_bits.h:234:5: 
  232.     uint8_t result = s->buffer[index>>3];
  233. #ifdef ALT_BITSTREAM_READER_LE
  234.     result >>= index & 7;
           ^
  235.     result &= 1;
  236. #else

libavcodec/get_bits.h:235:5: 
  233. #ifdef ALT_BITSTREAM_READER_LE
  234.     result >>= index & 7;
  235.     result &= 1;
           ^
  236. #else
  237.     result <<= index & 7;

libavcodec/get_bits.h:240:5: 
  238.     result >>= 8 - 1;
  239. #endif
  240.     index++;
           ^
  241.     s->index = index;
  242. 

libavcodec/get_bits.h:241:5: 
  239. #endif
  240.     index++;
  241.     s->index = index;
           ^
  242. 
  243.     return result;

libavcodec/get_bits.h:243:5: 
  241.     s->index = index;
  242. 
  243.     return result;
           ^
  244. }
  245. 

libavcodec/get_bits.h:244:1: return from a call to get_bits1
  242. 
  243.     return result;
  244. }
       ^
  245. 
  246. static inline unsigned int show_bits1(GetBitContext *s){

libavcodec/smacker.c:299:9: Taking true branch
297.             return -1;
298.     }
299.     if(!get_bits1(&gb)) {
             ^
300.         av_log(smk->avctx, AV_LOG_INFO, "Skipping MCLR tree\n");
301.         smk->mclr_tbl = av_malloc(sizeof(int) * 2);

libavcodec/smacker.c:300:9: Skipping av_log(): empty list of specs
298.     }
299.     if(!get_bits1(&gb)) {
300.         av_log(smk->avctx, AV_LOG_INFO, "Skipping MCLR tree\n");
             ^
301.         smk->mclr_tbl = av_malloc(sizeof(int) * 2);
302.         smk->mclr_tbl[0] = 0;

libavcodec/smacker.c:301:9: 
299.     if(!get_bits1(&gb)) {
300.         av_log(smk->avctx, AV_LOG_INFO, "Skipping MCLR tree\n");
301.         smk->mclr_tbl = av_malloc(sizeof(int) * 2);
             ^
302.         smk->mclr_tbl[0] = 0;
303.         smk->mclr_last[0] = smk->mclr_last[1] = smk->mclr_last[2] = 1;

libavutil/mem.c:64:1: start of procedure av_malloc()
  62.    linker will do it automatically. */
  63. 
  64. void *av_malloc(size_t size)
      ^
  65. {
  66.     void *ptr = NULL;

libavutil/mem.c:66:5: 
  64. void *av_malloc(size_t size)
  65. {
  66.     void *ptr = NULL;
          ^
  67. #if CONFIG_MEMALIGN_HACK
  68.     long diff;

libavutil/mem.c:72:8: Taking false branch
  70. 
  71.     /* let's disallow possible ambiguous cases */
  72.     if(size > (INT_MAX-32) )
             ^
  73.         return NULL;
  74. 

libavutil/mem.c:83:9: Taking false branch
  81.     ((char*)ptr)[-1]= diff;
  82. #elif HAVE_POSIX_MEMALIGN
  83.     if (posix_memalign(&ptr,32,size))
              ^
  84.         ptr = NULL;
  85. #elif HAVE_MEMALIGN

libavutil/mem.c:114:5: 
  112.     ptr = malloc(size);
  113. #endif
  114.     return ptr;
           ^
  115. }
  116. 

libavutil/mem.c:115:1: return from a call to av_malloc
  113. #endif
  114.     return ptr;
  115. }
       ^
  116. 
  117. void *av_realloc(void *ptr, size_t size)

libavcodec/smacker.c:302:9: 
300.         av_log(smk->avctx, AV_LOG_INFO, "Skipping MCLR tree\n");
301.         smk->mclr_tbl = av_malloc(sizeof(int) * 2);
302.         smk->mclr_tbl[0] = 0;
             ^
303.         smk->mclr_last[0] = smk->mclr_last[1] = smk->mclr_last[2] = 1;
304.     } else {

libavcodec/smacker.c:303:9: 
301.         smk->mclr_tbl = av_malloc(sizeof(int) * 2);
302.         smk->mclr_tbl[0] = 0;
303.         smk->mclr_last[0] = smk->mclr_last[1] = smk->mclr_last[2] = 1;
             ^
304.     } else {
305.         if (smacker_decode_header_tree(smk, &gb, &smk->mclr_tbl, smk->mclr_last, mclr_size))

libavcodec/smacker.c:308:9: 
306.             return -1;
307.     }
308.     if(!get_bits1(&gb)) {
             ^
309.         av_log(smk->avctx, AV_LOG_INFO, "Skipping FULL tree\n");
310.         smk->full_tbl = av_malloc(sizeof(int) * 2);

libavcodec/get_bits.h:230:1: start of procedure get_bits1()
  228. }
  229. 
  230. static inline unsigned int get_bits1(GetBitContext *s){
       ^
  231.     unsigned int index = s->index;
  232.     uint8_t result = s->buffer[index>>3];

libavcodec/get_bits.h:231:5: 
  229. 
  230. static inline unsigned int get_bits1(GetBitContext *s){
  231.     unsigned int index = s->index;
           ^
  232.     uint8_t result = s->buffer[index>>3];
  233. #ifdef ALT_BITSTREAM_READER_LE

libavcodec/get_bits.h:232:5: 
  230. static inline unsigned int get_bits1(GetBitContext *s){
  231.     unsigned int index = s->index;
  232.     uint8_t result = s->buffer[index>>3];
           ^
  233. #ifdef ALT_BITSTREAM_READER_LE
  234.     result >>= index & 7;

libavcodec/get_bits.h:234:5: 
  232.     uint8_t result = s->buffer[index>>3];
  233. #ifdef ALT_BITSTREAM_READER_LE
  234.     result >>= index & 7;
           ^
  235.     result &= 1;
  236. #else

libavcodec/get_bits.h:235:5: 
  233. #ifdef ALT_BITSTREAM_READER_LE
  234.     result >>= index & 7;
  235.     result &= 1;
           ^
  236. #else
  237.     result <<= index & 7;

libavcodec/get_bits.h:240:5: 
  238.     result >>= 8 - 1;
  239. #endif
  240.     index++;
           ^
  241.     s->index = index;
  242. 

libavcodec/get_bits.h:241:5: 
  239. #endif
  240.     index++;
  241.     s->index = index;
           ^
  242. 
  243.     return result;

libavcodec/get_bits.h:243:5: 
  241.     s->index = index;
  242. 
  243.     return result;
           ^
  244. }
  245. 

libavcodec/get_bits.h:244:1: return from a call to get_bits1
  242. 
  243.     return result;
  244. }
       ^
  245. 
  246. static inline unsigned int show_bits1(GetBitContext *s){

libavcodec/smacker.c:308:9: Taking true branch
306.             return -1;
307.     }
308.     if(!get_bits1(&gb)) {
             ^
309.         av_log(smk->avctx, AV_LOG_INFO, "Skipping FULL tree\n");
310.         smk->full_tbl = av_malloc(sizeof(int) * 2);

libavcodec/smacker.c:309:9: Skipping av_log(): empty list of specs
307.     }
308.     if(!get_bits1(&gb)) {
309.         av_log(smk->avctx, AV_LOG_INFO, "Skipping FULL tree\n");
             ^
310.         smk->full_tbl = av_malloc(sizeof(int) * 2);
311.         smk->full_tbl[0] = 0;

libavcodec/smacker.c:310:9: 
308.     if(!get_bits1(&gb)) {
309.         av_log(smk->avctx, AV_LOG_INFO, "Skipping FULL tree\n");
310.         smk->full_tbl = av_malloc(sizeof(int) * 2);
             ^
311.         smk->full_tbl[0] = 0;
312.         smk->full_last[0] = smk->full_last[1] = smk->full_last[2] = 1;

libavutil/mem.c:64:1: start of procedure av_malloc()
  62.    linker will do it automatically. */
  63. 
  64. void *av_malloc(size_t size)
      ^
  65. {
  66.     void *ptr = NULL;

libavutil/mem.c:66:5: 
  64. void *av_malloc(size_t size)
  65. {
  66.     void *ptr = NULL;
          ^
  67. #if CONFIG_MEMALIGN_HACK
  68.     long diff;

libavutil/mem.c:72:8: Taking false branch
  70. 
  71.     /* let's disallow possible ambiguous cases */
  72.     if(size > (INT_MAX-32) )
             ^
  73.         return NULL;
  74. 

libavutil/mem.c:83:9: Taking true branch
  81.     ((char*)ptr)[-1]= diff;
  82. #elif HAVE_POSIX_MEMALIGN
  83.     if (posix_memalign(&ptr,32,size))
              ^
  84.         ptr = NULL;
  85. #elif HAVE_MEMALIGN

libavutil/mem.c:84:9: 
  82. #elif HAVE_POSIX_MEMALIGN
  83.     if (posix_memalign(&ptr,32,size))
  84.         ptr = NULL;
              ^
  85. #elif HAVE_MEMALIGN
  86.     ptr = memalign(32,size);

libavutil/mem.c:114:5: 
  112.     ptr = malloc(size);
  113. #endif
  114.     return ptr;
           ^
  115. }
  116. 

libavutil/mem.c:115:1: return from a call to av_malloc
  113. #endif
  114.     return ptr;
  115. }
       ^
  116. 
  117. void *av_realloc(void *ptr, size_t size)

libavcodec/smacker.c:311:9: 
309.         av_log(smk->avctx, AV_LOG_INFO, "Skipping FULL tree\n");
310.         smk->full_tbl = av_malloc(sizeof(int) * 2);
311.         smk->full_tbl[0] = 0;
             ^
312.         smk->full_last[0] = smk->full_last[1] = smk->full_last[2] = 1;
313.     } else {
