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

libavcodec/bmp.c:122:13: 
120.     case 32:
121.         if(comp == BMP_BITFIELDS){
122.             rgb[0] = (rgb[0] >> 15) & 3;
                 ^
123.             rgb[1] = (rgb[1] >> 15) & 3;
124.             rgb[2] = (rgb[2] >> 15) & 3;

static int bmp_decode_frame(AVCodecContext *avctx,
                            void *data, int *data_size,
                            const uint8_t *buf, int buf_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;
    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)\n",
               buf_size, fsize);
        return -1;
    }
    buf += 2;
    buf += 2;
    hsize = bytestream_get_le32(&buf);
    if(fsize <= hsize){
        av_log(avctx, AV_LOG_ERROR, "not enough data (%d < %d)\n",
               fsize, hsize);
        return -1;
    }
    ihsize = bytestream_get_le32(&buf);
    if(ihsize + 14 > hsize){
        av_log(avctx, AV_LOG_ERROR, "invalid header size %d\n", hsize);
        return -1;
    }
    width = bytestream_get_le32(&buf);
    height = bytestream_get_le32(&buf);
    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 > 16)
        comp = bytestream_get_le32(&buf);
    else
        comp = BMP_RGB;
    if(comp != BMP_RGB && comp != BMP_BITFIELDS){
        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 = 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 = PIX_FMT_BGR24;
        break;
    case 24:
        avctx->pix_fmt = PIX_FMT_BGR24;
        break;
    case 16:
        if(comp == BMP_RGB)
            avctx->pix_fmt = PIX_FMT_RGB555;
        break;
    default:
        av_log(avctx, AV_LOG_ERROR, "depth %d not supported\n", depth);
        return -1;
    }
    if(avctx->pix_fmt == 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 = FF_I_TYPE;
    p->key_frame = 1;
    buf = buf0 + hsize;
    dsize = buf_size - hsize;
    n = (avctx->width * (depth / 8) + 3) & ~3;
    if(n * avctx->height > dsize){
        av_log(avctx, AV_LOG_ERROR, "not enough data (%d < %d)\n",
               dsize, n * avctx->height);
        return -1;
    }
    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];
    }
    switch(depth){
    case 24:
        for(i = 0; i < avctx->height; i++){
            memcpy(ptr, buf, avctx->width*(depth>>3));
            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++ = le2me_16(*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                            const uint8_t *buf, int buf_size)\n{\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    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)\\n",\n               buf_size, fsize);\n        return -1;\n    }\n    buf += 2;\n    buf += 2;\n    hsize = bytestream_get_le32(&buf);\n    if(fsize <= hsize){\n        av_log(avctx, AV_LOG_ERROR, "not enough data (%d < %d)\\n",\n               fsize, hsize);\n        return -1;\n    }\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    width = bytestream_get_le32(&buf);\n    height = bytestream_get_le32(&buf);\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 > 16)\n        comp = bytestream_get_le32(&buf);\n    else\n        comp = BMP_RGB;\n    if(comp != BMP_RGB && comp != BMP_BITFIELDS){\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 = 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 = PIX_FMT_BGR24;\n        break;\n    case 24:\n        avctx->pix_fmt = PIX_FMT_BGR24;\n        break;\n    case 16:\n        if(comp == BMP_RGB)\n            avctx->pix_fmt = PIX_FMT_RGB555;\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 == 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 = FF_I_TYPE;\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){\n        av_log(avctx, AV_LOG_ERROR, "not enough data (%d < %d)\\n",\n               dsize, n * avctx->height);\n        return -1;\n    }\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    switch(depth){\n    case 24:\n        for(i = 0; i < avctx->height; i++){\n            memcpy(ptr, buf, avctx->width*(depth>>3));\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++ = le2me_16(*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    *picture = s->picture;\n    *data_size = sizeof(AVPicture);\n    return buf_size;\n}']