Merge commit 'a801453423420915425f902bf9f9cbe982a55d87'
authorMichael Niedermayer <michaelni@gmx.at>
Sat, 8 Mar 2014 23:41:47 +0000 (00:41 +0100)
committerMichael Niedermayer <michaelni@gmx.at>
Sat, 8 Mar 2014 23:41:47 +0000 (00:41 +0100)
* commit 'a801453423420915425f902bf9f9cbe982a55d87':
  g2meet: K&R formatting cosmetics

Conflicts:
libavcodec/g2meet.c

Merged-by: Michael Niedermayer <michaelni@gmx.at>
1  2 
libavcodec/g2meet.c

@@@ -379,9 -379,8 +379,10 @@@ static int kempf_decode_tile(G2MContex
          src += 3;
      }
      npal = *src++ + 1;
-     memcpy(pal, src, npal * 3); src += npal * 3;
 +    if (src_end - src < npal * 3)
 +        return AVERROR_INVALIDDATA;
+     memcpy(pal, src, npal * 3);
+     src += npal * 3;
      if (sub_type != 2) {
          for (i = 0; i < npal; i++) {
              if (!memcmp(pal + i * 3, transp, 3)) {
  
      if (src_end - src < 2)
          return 0;
-     zsize = (src[0] << 8) | src[1]; src += 2;
+     zsize = (src[0] << 8) | src[1];
+     src  += 2;
  
 -    if (src_end - src < zsize)
 +    if (src_end - src < zsize + (sub_type != 2))
          return AVERROR_INVALIDDATA;
  
      ret = uncompress(c->kempf_buf, &dlen, src, zsize);
@@@ -491,14 -489,14 +493,14 @@@ static int g2m_load_cursor(AVCodecConte
      uint32_t cursor_hot_x, cursor_hot_y;
      int cursor_fmt, err;
  
-     cur_size      = bytestream2_get_be32(gb);
-     cursor_w      = bytestream2_get_byte(gb);
-     cursor_h      = bytestream2_get_byte(gb);
-     cursor_hot_x  = bytestream2_get_byte(gb);
-     cursor_hot_y  = bytestream2_get_byte(gb);
-     cursor_fmt    = bytestream2_get_byte(gb);
+     cur_size     = bytestream2_get_be32(gb);
+     cursor_w     = bytestream2_get_byte(gb);
+     cursor_h     = bytestream2_get_byte(gb);
+     cursor_hot_x = bytestream2_get_byte(gb);
+     cursor_hot_y = bytestream2_get_byte(gb);
+     cursor_fmt   = bytestream2_get_byte(gb);
  
 -    cursor_stride = FFALIGN(cursor_w, 32) * 4;
 +    cursor_stride = FFALIGN(cursor_w, cursor_fmt==1 ? 32 : 1) * 4;
  
      if (cursor_w < 1 || cursor_w > 256 ||
          cursor_h < 1 || cursor_h > 256) {
@@@ -733,14 -735,13 +741,14 @@@ static int g2m_decode_frame(AVCodecCont
              }
              c->tiles_x = (c->width  + c->tile_width  - 1) / c->tile_width;
              c->tiles_y = (c->height + c->tile_height - 1) / c->tile_height;
-             c->bpp = bytestream2_get_byte(&bc);
+             c->bpp     = bytestream2_get_byte(&bc);
              if (c->bpp == 32) {
                  if (bytestream2_get_bytes_left(&bc) < 16 ||
-                     (chunk_size - 21) < 16 ) {
+                     (chunk_size - 21) < 16) {
                      av_log(avctx, AV_LOG_ERROR,
                             "Display info: missing bitmasks!\n");
 -                    return AVERROR_INVALIDDATA;
 +                    ret = AVERROR_INVALIDDATA;
 +                    goto header_fail;
                  }
                  r_mask = bytestream2_get_be32(&bc);
                  g_mask = bytestream2_get_be32(&bc);