Merge commit '1ecdf8912b9ced51b3267cdcdce5e394d0a3bf8e'
authorMichael Niedermayer <michaelni@gmx.at>
Thu, 14 Mar 2013 11:41:34 +0000 (12:41 +0100)
committerMichael Niedermayer <michaelni@gmx.at>
Thu, 14 Mar 2013 11:41:34 +0000 (12:41 +0100)
* commit '1ecdf8912b9ced51b3267cdcdce5e394d0a3bf8e':
  avformat: av_log_ask_for_sample() ---> avpriv_request_sample()

Conflicts:
libavformat/mxfdec.c

Merged-by: Michael Niedermayer <michaelni@gmx.at>
18 files changed:
1  2 
libavformat/anm.c
libavformat/au.c
libavformat/bethsoftvid.c
libavformat/filmstripdec.c
libavformat/mov.c
libavformat/mpegts.c
libavformat/mtv.c
libavformat/mxfdec.c
libavformat/omadec.c
libavformat/rsodec.c
libavformat/rtpdec_latm.c
libavformat/rtpdec_qt.c
libavformat/smjpegdec.c
libavformat/spdifdec.c
libavformat/spdifenc.c
libavformat/westwood_aud.c
libavformat/xmv.c
libavformat/xwma.c

Simple merge
Simple merge
Simple merge
Simple merge
Simple merge
Simple merge
Simple merge
@@@ -411,18 -396,15 +411,17 @@@ static int mxf_read_primer_pack(void *a
      int item_len = avio_rb32(pb);
  
      if (item_len != 18) {
-         av_log_ask_for_sample(pb, "unsupported primer pack item length %d\n",
-                               item_len);
+         avpriv_request_sample(pb, "Primer pack item length %d", item_len);
          return AVERROR_PATCHWELCOME;
      }
 -    if (item_num > UINT_MAX / item_len)
 +    if (item_num > 65536) {
 +        av_log(mxf->fc, AV_LOG_ERROR, "item_num %d is too large\n", item_num);
          return AVERROR_INVALIDDATA;
 -    mxf->local_tags_count = item_num;
 -    mxf->local_tags = av_malloc(item_num*item_len);
 +    }
 +    mxf->local_tags = av_calloc(item_num, item_len);
      if (!mxf->local_tags)
          return AVERROR(ENOMEM);
 +    mxf->local_tags_count = item_num;
      avio_read(pb, mxf->local_tags, item_num*item_len);
      return 0;
  }
@@@ -2125,11 -2023,14 +2124,13 @@@ static int mxf_read_packet_old(AVFormat
              next_ofs = mxf_set_current_edit_unit(mxf, klv.offset);
  
              if (next_ofs >= 0 && next_klv > next_ofs) {
 -                /* if this check is hit then it's possible OPAtom was treated
 -                 * as OP1a truncate the packet since it's probably very large
 -                 * (>2 GiB is common) */
 +                /* if this check is hit then it's possible OPAtom was treated as OP1a
 +                 * truncate the packet since it's probably very large (>2 GiB is common) */
-                 av_log_ask_for_sample(s,
-                     "KLV for edit unit %i extends into next edit unit - OPAtom misinterpreted as OP1a?\n",
-                     mxf->current_edit_unit);
+                 avpriv_request_sample(s,
+                                       "OPAtom misinterpreted as OP1a?"
+                                       "KLV for edit unit %i extending into "
+                                       "next edit unit",
+                                       mxf->current_edit_unit);
                  klv.length = next_ofs - avio_tell(s->pb);
              }
  
Simple merge
Simple merge
Simple merge
Simple merge
Simple merge
Simple merge
Simple merge
Simple merge
Simple merge
Simple merge