Merge remote-tracking branch 'qatar/master'
authorMichael Niedermayer <michaelni@gmx.at>
Tue, 4 Oct 2011 01:21:14 +0000 (03:21 +0200)
committerMichael Niedermayer <michaelni@gmx.at>
Tue, 4 Oct 2011 01:24:59 +0000 (03:24 +0200)
* qatar/master:
  id3v2: remove pointless casts
  id3v2: read TXXX frames with two calls to decode_str() instead of one.
  id3v2: don't discard the whole tag when encountering empty frames.
  libvpx: fix build with older libvpx versions.
  ARM: check for inline asm 'y' operand modifier support

Conflicts:
libavcodec/libvpxenc.c
libavformat/id3v2.c

Merged-by: Michael Niedermayer <michaelni@gmx.at>
1  2 
configure
libavformat/id3v2.c

diff --cc configure
Simple merge
@@@ -410,18 -397,20 +406,24 @@@ static void ff_id3v2_parse(AVFormatCont
              tag[3] = 0;
              tlen = avio_rb24(s->pb);
          }
-         if (tlen > (1<<28) || !tlen)
 -        if (tlen < 0 || tlen > len - taghdrlen) {
 -            av_log(s, AV_LOG_WARNING, "Invalid size in frame %s, skipping the rest of tag.\n", tag);
++        if (tlen > (1<<28))
              break;
 -        }
          len -= taghdrlen + tlen;
 +
 +        if (len < 0)
 +            break;
 +
          next = avio_tell(s->pb) + tlen;
  
+         if (!tlen) {
+             if (tag[0])
+                 av_log(s, AV_LOG_DEBUG, "Invalid empty frame %s, skipping.\n", tag);
+             continue;
+         }
          if (tflags & ID3v2_FLAG_DATALEN) {
 +            if (tlen < 4)
 +                break;
              avio_rb32(s->pb);
              tlen -= 4;
          }