libavcodec/bmp.c:169: error: Uninitialized Value
  The value read from rgb[_] was never initialized.

libavcodec/bmp.c:169:42: 
167.             if (rgb[0] == 0xF800 && rgb[1] == 0x07E0 && rgb[2] == 0x001F)
168.                avctx->pix_fmt = AV_PIX_FMT_RGB565;
169.             else if (rgb[0] == 0x7C00 && rgb[1] == 0x03E0 && rgb[2] == 0x001F)
                                              ^
170.                avctx->pix_fmt = AV_PIX_FMT_RGB555;
171.             else if (rgb[0] == 0x0F00 && rgb[1] == 0x00F0 && rgb[2] == 0x000F)

static int bmp_decode_frame(AVCodecContext *avctx,
                            void *data, int *data_size,
                            AVPacket *avpkt)
{
    const uint8_t *buf = avpkt->data;
    int buf_size = avpkt->size;
    BMPContext *s = avctx->priv_data;
    AVFrame *picture = data;
    AVFrame *p = &s->picture;
    unsigned int fsize, hsize;
    int width, height;
    unsigned int depth;
    BiCompression comp;
    unsigned int ihsize;
    int i, j, n, linesize;
    uint32_t rgb[3];
    uint8_t *ptr;
    int dsize;
    const uint8_t *buf0 = buf;
    GetByteContext gb;
    if(buf_size < 14){
        av_log(avctx, AV_LOG_ERROR, "buf size too small (%d)\n", buf_size);
        return -1;
    }
    if(bytestream_get_byte(&buf) != 'B' ||
       bytestream_get_byte(&buf) != 'M') {
        av_log(avctx, AV_LOG_ERROR, "bad magic number\n");
        return -1;
    }
    fsize = bytestream_get_le32(&buf);
    if(buf_size < fsize){
        av_log(avctx, AV_LOG_ERROR, "not enough data (%d < %d), trying to decode anyway\n",
               buf_size, fsize);
        fsize = buf_size;
    }
    buf += 2;
    buf += 2;
    hsize = bytestream_get_le32(&buf);
    ihsize = bytestream_get_le32(&buf);
    if(ihsize + 14 > hsize){
        av_log(avctx, AV_LOG_ERROR, "invalid header size %d\n", hsize);
        return -1;
    }
    if(fsize == 14 || fsize == ihsize + 14)
        fsize = buf_size - 2;
    if(fsize <= hsize){
        av_log(avctx, AV_LOG_ERROR, "declared file size is less than header size (%d < %d)\n",
               fsize, hsize);
        return -1;
    }
    switch(ihsize){
    case  40:
    case  64:
    case 108:
    case 124:
        width = bytestream_get_le32(&buf);
        height = bytestream_get_le32(&buf);
        break;
    case  12:
        width  = bytestream_get_le16(&buf);
        height = bytestream_get_le16(&buf);
        break;
    default:
        av_log(avctx, AV_LOG_ERROR, "unsupported BMP file, patch welcome\n");
        return -1;
    }
    if(bytestream_get_le16(&buf) != 1){
        av_log(avctx, AV_LOG_ERROR, "invalid BMP header\n");
        return -1;
    }
    depth = bytestream_get_le16(&buf);
    if(ihsize == 40)
        comp = bytestream_get_le32(&buf);
    else
        comp = BMP_RGB;
    if(comp != BMP_RGB && comp != BMP_BITFIELDS && comp != BMP_RLE4 && comp != BMP_RLE8){
        av_log(avctx, AV_LOG_ERROR, "BMP coding %d not supported\n", comp);
        return -1;
    }
    if(comp == BMP_BITFIELDS){
        buf += 20;
        rgb[0] = bytestream_get_le32(&buf);
        rgb[1] = bytestream_get_le32(&buf);
        rgb[2] = bytestream_get_le32(&buf);
    }
    avctx->width = width;
    avctx->height = height > 0? height: -height;
    avctx->pix_fmt = AV_PIX_FMT_NONE;
    switch(depth){
    case 32:
        if(comp == BMP_BITFIELDS){
            rgb[0] = (rgb[0] >> 15) & 3;
            rgb[1] = (rgb[1] >> 15) & 3;
            rgb[2] = (rgb[2] >> 15) & 3;
            if(rgb[0] + rgb[1] + rgb[2] != 3 ||
               rgb[0] == rgb[1] || rgb[0] == rgb[2] || rgb[1] == rgb[2]){
                break;
            }
        } else {
            rgb[0] = 2;
            rgb[1] = 1;
            rgb[2] = 0;
        }
        avctx->pix_fmt = AV_PIX_FMT_BGR24;
        break;
    case 24:
        avctx->pix_fmt = AV_PIX_FMT_BGR24;
        break;
    case 16:
        if(comp == BMP_RGB)
            avctx->pix_fmt = AV_PIX_FMT_RGB555;
        else if (comp == BMP_BITFIELDS) {
            if (rgb[0] == 0xF800 && rgb[1] == 0x07E0 && rgb[2] == 0x001F)
               avctx->pix_fmt = AV_PIX_FMT_RGB565;
            else if (rgb[0] == 0x7C00 && rgb[1] == 0x03E0 && rgb[2] == 0x001F)
               avctx->pix_fmt = AV_PIX_FMT_RGB555;
            else if (rgb[0] == 0x0F00 && rgb[1] == 0x00F0 && rgb[2] == 0x000F)
               avctx->pix_fmt = AV_PIX_FMT_RGB444;
            else {
               av_log(avctx, AV_LOG_ERROR, "Unknown bitfields %0X %0X %0X\n", rgb[0], rgb[1], rgb[2]);
               return AVERROR(EINVAL);
            }
        }
        break;
    case 8:
        if(hsize - ihsize - 14 > 0)
            avctx->pix_fmt = AV_PIX_FMT_PAL8;
        else
            avctx->pix_fmt = AV_PIX_FMT_GRAY8;
        break;
    case 1:
    case 4:
        if(hsize - ihsize - 14 > 0){
            avctx->pix_fmt = AV_PIX_FMT_PAL8;
        }else{
            av_log(avctx, AV_LOG_ERROR, "Unknown palette for %d-colour BMP\n", 1<<depth);
            return -1;
        }
        break;
    default:
        av_log(avctx, AV_LOG_ERROR, "depth %d not supported\n", depth);
        return -1;
    }
    if(avctx->pix_fmt == AV_PIX_FMT_NONE){
        av_log(avctx, AV_LOG_ERROR, "unsupported pixel format\n");
        return -1;
    }
    if(p->data[0])
        avctx->release_buffer(avctx, p);
    p->reference = 0;
    if(avctx->get_buffer(avctx, p) < 0){
        av_log(avctx, AV_LOG_ERROR, "get_buffer() failed\n");
        return -1;
    }
    p->pict_type = AV_PICTURE_TYPE_I;
    p->key_frame = 1;
    buf = buf0 + hsize;
    dsize = buf_size - hsize;
    n = ((avctx->width * depth) / 8 + 3) & ~3;
    if(n * avctx->height > dsize && comp != BMP_RLE4 && comp != BMP_RLE8){
        av_log(avctx, AV_LOG_ERROR, "not enough data (%d < %d)\n",
               dsize, n * avctx->height);
        return -1;
    }
    if(comp == BMP_RLE4 || comp == BMP_RLE8)
        memset(p->data[0], 0, avctx->height * p->linesize[0]);
    if(height > 0){
        ptr = p->data[0] + (avctx->height - 1) * p->linesize[0];
        linesize = -p->linesize[0];
    } else {
        ptr = p->data[0];
        linesize = p->linesize[0];
    }
    if(avctx->pix_fmt == AV_PIX_FMT_PAL8){
        int colors = 1 << depth;
        memset(p->data[1], 0, 1024);
        if(ihsize >= 36){
            int t;
            buf = buf0 + 46;
            t = bytestream_get_le32(&buf);
            if(t < 0 || t > (1 << depth)){
                av_log(avctx, AV_LOG_ERROR, "Incorrect number of colors - %X for bitdepth %d\n", t, depth);
            }else if(t){
                colors = t;
            }
        }
        buf = buf0 + 14 + ihsize;
        if((hsize-ihsize-14) < (colors << 2)){
            for(i = 0; i < colors; i++)
                ((uint32_t*)p->data[1])[i] = bytestream_get_le24(&buf);
        }else{
            for(i = 0; i < colors; i++)
                ((uint32_t*)p->data[1])[i] = bytestream_get_le32(&buf);
        }
        buf = buf0 + hsize;
    }
    if(comp == BMP_RLE4 || comp == BMP_RLE8){
        if(height < 0){
            p->data[0] += p->linesize[0] * (avctx->height - 1);
            p->linesize[0] = -p->linesize[0];
        }
        bytestream2_init(&gb, buf, dsize);
        ff_msrle_decode(avctx, (AVPicture*)p, depth, &gb);
        if(height < 0){
            p->data[0] += p->linesize[0] * (avctx->height - 1);
            p->linesize[0] = -p->linesize[0];
        }
    }else{
        switch(depth){
        case 1:
            for (i = 0; i < avctx->height; i++) {
                int j;
                for (j = 0; j < n; j++) {
                    ptr[j*8+0] =  buf[j] >> 7;
                    ptr[j*8+1] = (buf[j] >> 6) & 1;
                    ptr[j*8+2] = (buf[j] >> 5) & 1;
                    ptr[j*8+3] = (buf[j] >> 4) & 1;
                    ptr[j*8+4] = (buf[j] >> 3) & 1;
                    ptr[j*8+5] = (buf[j] >> 2) & 1;
                    ptr[j*8+6] = (buf[j] >> 1) & 1;
                    ptr[j*8+7] =  buf[j]       & 1;
                }
                buf += n;
                ptr += linesize;
            }
            break;
        case 8:
        case 24:
            for(i = 0; i < avctx->height; i++){
                memcpy(ptr, buf, n);
                buf += n;
                ptr += linesize;
            }
            break;
        case 4:
            for(i = 0; i < avctx->height; i++){
                int j;
                for(j = 0; j < n; j++){
                    ptr[j*2+0] = (buf[j] >> 4) & 0xF;
                    ptr[j*2+1] = buf[j] & 0xF;
                }
                buf += n;
                ptr += linesize;
            }
            break;
        case 16:
            for(i = 0; i < avctx->height; i++){
                const uint16_t *src = (const uint16_t *) buf;
                uint16_t *dst = (uint16_t *) ptr;
                for(j = 0; j < avctx->width; j++)
                    *dst++ = av_le2ne16(*src++);
                buf += n;
                ptr += linesize;
            }
            break;
        case 32:
            for(i = 0; i < avctx->height; i++){
                const uint8_t *src = buf;
                uint8_t *dst = ptr;
                for(j = 0; j < avctx->width; j++){
                    dst[0] = src[rgb[2]];
                    dst[1] = src[rgb[1]];
                    dst[2] = src[rgb[0]];
                    dst += 3;
                    src += 4;
                }
                buf += n;
                ptr += linesize;
            }
            break;
        default:
            av_log(avctx, AV_LOG_ERROR, "BMP decoder is broken\n");
            return -1;
        }
    }
    *picture = s->picture;
    *data_size = sizeof(AVPicture);
    return buf_size;
}
['static int bmp_decode_frame(AVCodecContext *avctx,\n                            void *data, int *data_size,\n                            AVPacket *avpkt)\n{\n    const uint8_t *buf = avpkt->data;\n    int buf_size = avpkt->size;\n    BMPContext *s = avctx->priv_data;\n    AVFrame *picture = data;\n    AVFrame *p = &s->picture;\n    unsigned int fsize, hsize;\n    int width, height;\n    unsigned int depth;\n    BiCompression comp;\n    unsigned int ihsize;\n    int i, j, n, linesize;\n    uint32_t rgb[3];\n    uint8_t *ptr;\n    int dsize;\n    const uint8_t *buf0 = buf;\n    GetByteContext gb;\n    if(buf_size < 14){\n        av_log(avctx, AV_LOG_ERROR, "buf size too small (%d)\\n", buf_size);\n        return -1;\n    }\n    if(bytestream_get_byte(&buf) != \'B\' ||\n       bytestream_get_byte(&buf) != \'M\') {\n        av_log(avctx, AV_LOG_ERROR, "bad magic number\\n");\n        return -1;\n    }\n    fsize = bytestream_get_le32(&buf);\n    if(buf_size < fsize){\n        av_log(avctx, AV_LOG_ERROR, "not enough data (%d < %d), trying to decode anyway\\n",\n               buf_size, fsize);\n        fsize = buf_size;\n    }\n    buf += 2;\n    buf += 2;\n    hsize = bytestream_get_le32(&buf);\n    ihsize = bytestream_get_le32(&buf);\n    if(ihsize + 14 > hsize){\n        av_log(avctx, AV_LOG_ERROR, "invalid header size %d\\n", hsize);\n        return -1;\n    }\n    if(fsize == 14 || fsize == ihsize + 14)\n        fsize = buf_size - 2;\n    if(fsize <= hsize){\n        av_log(avctx, AV_LOG_ERROR, "declared file size is less than header size (%d < %d)\\n",\n               fsize, hsize);\n        return -1;\n    }\n    switch(ihsize){\n    case  40:\n    case  64:\n    case 108:\n    case 124:\n        width = bytestream_get_le32(&buf);\n        height = bytestream_get_le32(&buf);\n        break;\n    case  12:\n        width  = bytestream_get_le16(&buf);\n        height = bytestream_get_le16(&buf);\n        break;\n    default:\n        av_log(avctx, AV_LOG_ERROR, "unsupported BMP file, patch welcome\\n");\n        return -1;\n    }\n    if(bytestream_get_le16(&buf) != 1){\n        av_log(avctx, AV_LOG_ERROR, "invalid BMP header\\n");\n        return -1;\n    }\n    depth = bytestream_get_le16(&buf);\n    if(ihsize == 40)\n        comp = bytestream_get_le32(&buf);\n    else\n        comp = BMP_RGB;\n    if(comp != BMP_RGB && comp != BMP_BITFIELDS && comp != BMP_RLE4 && comp != BMP_RLE8){\n        av_log(avctx, AV_LOG_ERROR, "BMP coding %d not supported\\n", comp);\n        return -1;\n    }\n    if(comp == BMP_BITFIELDS){\n        buf += 20;\n        rgb[0] = bytestream_get_le32(&buf);\n        rgb[1] = bytestream_get_le32(&buf);\n        rgb[2] = bytestream_get_le32(&buf);\n    }\n    avctx->width = width;\n    avctx->height = height > 0? height: -height;\n    avctx->pix_fmt = AV_PIX_FMT_NONE;\n    switch(depth){\n    case 32:\n        if(comp == BMP_BITFIELDS){\n            rgb[0] = (rgb[0] >> 15) & 3;\n            rgb[1] = (rgb[1] >> 15) & 3;\n            rgb[2] = (rgb[2] >> 15) & 3;\n            if(rgb[0] + rgb[1] + rgb[2] != 3 ||\n               rgb[0] == rgb[1] || rgb[0] == rgb[2] || rgb[1] == rgb[2]){\n                break;\n            }\n        } else {\n            rgb[0] = 2;\n            rgb[1] = 1;\n            rgb[2] = 0;\n        }\n        avctx->pix_fmt = AV_PIX_FMT_BGR24;\n        break;\n    case 24:\n        avctx->pix_fmt = AV_PIX_FMT_BGR24;\n        break;\n    case 16:\n        if(comp == BMP_RGB)\n            avctx->pix_fmt = AV_PIX_FMT_RGB555;\n        else if (comp == BMP_BITFIELDS) {\n            if (rgb[0] == 0xF800 && rgb[1] == 0x07E0 && rgb[2] == 0x001F)\n               avctx->pix_fmt = AV_PIX_FMT_RGB565;\n            else if (rgb[0] == 0x7C00 && rgb[1] == 0x03E0 && rgb[2] == 0x001F)\n               avctx->pix_fmt = AV_PIX_FMT_RGB555;\n            else if (rgb[0] == 0x0F00 && rgb[1] == 0x00F0 && rgb[2] == 0x000F)\n               avctx->pix_fmt = AV_PIX_FMT_RGB444;\n            else {\n               av_log(avctx, AV_LOG_ERROR, "Unknown bitfields %0X %0X %0X\\n", rgb[0], rgb[1], rgb[2]);\n               return AVERROR(EINVAL);\n            }\n        }\n        break;\n    case 8:\n        if(hsize - ihsize - 14 > 0)\n            avctx->pix_fmt = AV_PIX_FMT_PAL8;\n        else\n            avctx->pix_fmt = AV_PIX_FMT_GRAY8;\n        break;\n    case 1:\n    case 4:\n        if(hsize - ihsize - 14 > 0){\n            avctx->pix_fmt = AV_PIX_FMT_PAL8;\n        }else{\n            av_log(avctx, AV_LOG_ERROR, "Unknown palette for %d-colour BMP\\n", 1<<depth);\n            return -1;\n        }\n        break;\n    default:\n        av_log(avctx, AV_LOG_ERROR, "depth %d not supported\\n", depth);\n        return -1;\n    }\n    if(avctx->pix_fmt == AV_PIX_FMT_NONE){\n        av_log(avctx, AV_LOG_ERROR, "unsupported pixel format\\n");\n        return -1;\n    }\n    if(p->data[0])\n        avctx->release_buffer(avctx, p);\n    p->reference = 0;\n    if(avctx->get_buffer(avctx, p) < 0){\n        av_log(avctx, AV_LOG_ERROR, "get_buffer() failed\\n");\n        return -1;\n    }\n    p->pict_type = AV_PICTURE_TYPE_I;\n    p->key_frame = 1;\n    buf = buf0 + hsize;\n    dsize = buf_size - hsize;\n    n = ((avctx->width * depth) / 8 + 3) & ~3;\n    if(n * avctx->height > dsize && comp != BMP_RLE4 && comp != BMP_RLE8){\n        av_log(avctx, AV_LOG_ERROR, "not enough data (%d < %d)\\n",\n               dsize, n * avctx->height);\n        return -1;\n    }\n    if(comp == BMP_RLE4 || comp == BMP_RLE8)\n        memset(p->data[0], 0, avctx->height * p->linesize[0]);\n    if(height > 0){\n        ptr = p->data[0] + (avctx->height - 1) * p->linesize[0];\n        linesize = -p->linesize[0];\n    } else {\n        ptr = p->data[0];\n        linesize = p->linesize[0];\n    }\n    if(avctx->pix_fmt == AV_PIX_FMT_PAL8){\n        int colors = 1 << depth;\n        memset(p->data[1], 0, 1024);\n        if(ihsize >= 36){\n            int t;\n            buf = buf0 + 46;\n            t = bytestream_get_le32(&buf);\n            if(t < 0 || t > (1 << depth)){\n                av_log(avctx, AV_LOG_ERROR, "Incorrect number of colors - %X for bitdepth %d\\n", t, depth);\n            }else if(t){\n                colors = t;\n            }\n        }\n        buf = buf0 + 14 + ihsize;\n        if((hsize-ihsize-14) < (colors << 2)){\n            for(i = 0; i < colors; i++)\n                ((uint32_t*)p->data[1])[i] = bytestream_get_le24(&buf);\n        }else{\n            for(i = 0; i < colors; i++)\n                ((uint32_t*)p->data[1])[i] = bytestream_get_le32(&buf);\n        }\n        buf = buf0 + hsize;\n    }\n    if(comp == BMP_RLE4 || comp == BMP_RLE8){\n        if(height < 0){\n            p->data[0] += p->linesize[0] * (avctx->height - 1);\n            p->linesize[0] = -p->linesize[0];\n        }\n        bytestream2_init(&gb, buf, dsize);\n        ff_msrle_decode(avctx, (AVPicture*)p, depth, &gb);\n        if(height < 0){\n            p->data[0] += p->linesize[0] * (avctx->height - 1);\n            p->linesize[0] = -p->linesize[0];\n        }\n    }else{\n        switch(depth){\n        case 1:\n            for (i = 0; i < avctx->height; i++) {\n                int j;\n                for (j = 0; j < n; j++) {\n                    ptr[j*8+0] =  buf[j] >> 7;\n                    ptr[j*8+1] = (buf[j] >> 6) & 1;\n                    ptr[j*8+2] = (buf[j] >> 5) & 1;\n                    ptr[j*8+3] = (buf[j] >> 4) & 1;\n                    ptr[j*8+4] = (buf[j] >> 3) & 1;\n                    ptr[j*8+5] = (buf[j] >> 2) & 1;\n                    ptr[j*8+6] = (buf[j] >> 1) & 1;\n                    ptr[j*8+7] =  buf[j]       & 1;\n                }\n                buf += n;\n                ptr += linesize;\n            }\n            break;\n        case 8:\n        case 24:\n            for(i = 0; i < avctx->height; i++){\n                memcpy(ptr, buf, n);\n                buf += n;\n                ptr += linesize;\n            }\n            break;\n        case 4:\n            for(i = 0; i < avctx->height; i++){\n                int j;\n                for(j = 0; j < n; j++){\n                    ptr[j*2+0] = (buf[j] >> 4) & 0xF;\n                    ptr[j*2+1] = buf[j] & 0xF;\n                }\n                buf += n;\n                ptr += linesize;\n            }\n            break;\n        case 16:\n            for(i = 0; i < avctx->height; i++){\n                const uint16_t *src = (const uint16_t *) buf;\n                uint16_t *dst = (uint16_t *) ptr;\n                for(j = 0; j < avctx->width; j++)\n                    *dst++ = av_le2ne16(*src++);\n                buf += n;\n                ptr += linesize;\n            }\n            break;\n        case 32:\n            for(i = 0; i < avctx->height; i++){\n                const uint8_t *src = buf;\n                uint8_t *dst = ptr;\n                for(j = 0; j < avctx->width; j++){\n                    dst[0] = src[rgb[2]];\n                    dst[1] = src[rgb[1]];\n                    dst[2] = src[rgb[0]];\n                    dst += 3;\n                    src += 4;\n                }\n                buf += n;\n                ptr += linesize;\n            }\n            break;\n        default:\n            av_log(avctx, AV_LOG_ERROR, "BMP decoder is broken\\n");\n            return -1;\n        }\n    }\n    *picture = s->picture;\n    *data_size = sizeof(AVPicture);\n    return buf_size;\n}']