Merge commit 'cdf58f0599c39852ee3beafe5f64af7d57d4215b'
authorMichael Niedermayer <michaelni@gmx.at>
Mon, 19 May 2014 16:50:40 +0000 (18:50 +0200)
committerMichael Niedermayer <michaelni@gmx.at>
Mon, 19 May 2014 16:51:34 +0000 (18:51 +0200)
* commit 'cdf58f0599c39852ee3beafe5f64af7d57d4215b':
  avpacket: fix copying side data in av_packet_copy_props()

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

diff --combined libavcodec/avpacket.c
@@@ -2,20 -2,20 +2,20 @@@
   * AVPacket functions for libavcodec
   * Copyright (c) 2000, 2001, 2002 Fabrice Bellard
   *
 - * This file is part of Libav.
 + * This file is part of FFmpeg.
   *
 - * Libav is free software; you can redistribute it and/or
 + * FFmpeg is free software; you can redistribute it and/or
   * modify it under the terms of the GNU Lesser General Public
   * License as published by the Free Software Foundation; either
   * version 2.1 of the License, or (at your option) any later version.
   *
 - * Libav is distributed in the hope that it will be useful,
 + * FFmpeg is distributed in the hope that it will be useful,
   * but WITHOUT ANY WARRANTY; without even the implied warranty of
   * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
   * Lesser General Public License for more details.
   *
   * You should have received a copy of the GNU Lesser General Public
 - * License along with Libav; if not, write to the Free Software
 + * License along with FFmpeg; if not, write to the Free Software
   * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
   */
  
@@@ -26,9 -26,6 +26,9 @@@
  #include "libavutil/internal.h"
  #include "libavutil/mem.h"
  #include "avcodec.h"
 +#include "bytestream.h"
 +#include "internal.h"
 +
  #if FF_API_DESTRUCT_PACKET
  
  void av_destruct_packet(AVPacket *pkt)
@@@ -189,55 -186,43 +189,55 @@@ do 
          dst = data;                                                     \
      } while (0)
  
 -int av_dup_packet(AVPacket *pkt)
 +/* Makes duplicates of data, side_data, but does not copy any other fields */
 +static int copy_packet_data(AVPacket *pkt, const AVPacket *src, int dup)
  {
 -    AVPacket tmp_pkt;
 -
 -FF_DISABLE_DEPRECATION_WARNINGS
 -    if (!pkt->buf && pkt->data
 -#if FF_API_DESTRUCT_PACKET
 -        && !pkt->destruct
 -#endif
 -        ) {
 -FF_ENABLE_DEPRECATION_WARNINGS
 -        tmp_pkt = *pkt;
 -
 -        pkt->data      = NULL;
 -        pkt->side_data = NULL;
 -        DUP_DATA(pkt->data, tmp_pkt.data, pkt->size, 1, ALLOC_BUF);
 +    pkt->data      = NULL;
 +    pkt->side_data = NULL;
 +    if (pkt->buf) {
 +        AVBufferRef *ref = av_buffer_ref(src->buf);
 +        if (!ref)
 +            return AVERROR(ENOMEM);
 +        pkt->buf  = ref;
 +        pkt->data = ref->data;
 +    } else {
 +        DUP_DATA(pkt->data, src->data, pkt->size, 1, ALLOC_BUF);
 +    }
  #if FF_API_DESTRUCT_PACKET
  FF_DISABLE_DEPRECATION_WARNINGS
 -        pkt->destruct = dummy_destruct_packet;
 +    pkt->destruct = dummy_destruct_packet;
  FF_ENABLE_DEPRECATION_WARNINGS
  #endif
 +    if (pkt->side_data_elems && dup)
 +        pkt->side_data = src->side_data;
 +    if (pkt->side_data_elems && !dup) {
 +        return av_copy_packet_side_data(pkt, src);
 +    }
 +    return 0;
  
 -        if (pkt->side_data_elems) {
 -            int i;
 +failed_alloc:
 +    av_free_packet(pkt);
 +    return AVERROR(ENOMEM);
 +}
  
 -            DUP_DATA(pkt->side_data, tmp_pkt.side_data,
 -                     pkt->side_data_elems * sizeof(*pkt->side_data), 0, ALLOC_MALLOC);
 +int av_copy_packet_side_data(AVPacket *pkt, const AVPacket *src)
 +{
 +    if (src->side_data_elems) {
 +        int i;
 +        DUP_DATA(pkt->side_data, src->side_data,
 +                src->side_data_elems * sizeof(*src->side_data), 0, ALLOC_MALLOC);
 +        if (src != pkt) {
              memset(pkt->side_data, 0,
 -                   pkt->side_data_elems * sizeof(*pkt->side_data));
 -            for (i = 0; i < pkt->side_data_elems; i++) {
 -                DUP_DATA(pkt->side_data[i].data, tmp_pkt.side_data[i].data,
 -                         tmp_pkt.side_data[i].size, 1, ALLOC_MALLOC);
 -                pkt->side_data[i].size = tmp_pkt.side_data[i].size;
 -                pkt->side_data[i].type = tmp_pkt.side_data[i].type;
 -            }
 +                   src->side_data_elems * sizeof(*src->side_data));
 +        }
 +        for (i = 0; i < src->side_data_elems; i++) {
 +            DUP_DATA(pkt->side_data[i].data, src->side_data[i].data,
 +                    src->side_data[i].size, 1, ALLOC_MALLOC);
 +            pkt->side_data[i].size = src->side_data[i].size;
 +            pkt->side_data[i].type = src->side_data[i].type;
          }
      }
 +    pkt->side_data_elems = src->side_data_elems;
      return 0;
  
  failed_alloc:
      return AVERROR(ENOMEM);
  }
  
 +int av_dup_packet(AVPacket *pkt)
 +{
 +    AVPacket tmp_pkt;
 +
 +FF_DISABLE_DEPRECATION_WARNINGS
 +    if (!pkt->buf && pkt->data
 +#if FF_API_DESTRUCT_PACKET
 +        && !pkt->destruct
 +#endif
 +        ) {
 +FF_ENABLE_DEPRECATION_WARNINGS
 +        tmp_pkt = *pkt;
 +        return copy_packet_data(pkt, &tmp_pkt, 1);
 +    }
 +    return 0;
 +}
 +
 +int av_copy_packet(AVPacket *dst, const AVPacket *src)
 +{
 +    *dst = *src;
 +    return copy_packet_data(dst, src, 0);
 +}
 +
  void av_packet_free_side_data(AVPacket *pkt)
  {
      int i;
@@@ -311,7 -273,7 +311,7 @@@ uint8_t *av_packet_new_side_data(AVPack
      if (!pkt->side_data)
          return NULL;
  
 -    pkt->side_data[elems].data = av_malloc(size + FF_INPUT_BUFFER_PADDING_SIZE);
 +    pkt->side_data[elems].data = av_mallocz(size + FF_INPUT_BUFFER_PADDING_SIZE);
      if (!pkt->side_data[elems].data)
          return NULL;
      pkt->side_data[elems].size = size;
@@@ -336,150 -298,6 +336,150 @@@ uint8_t *av_packet_get_side_data(AVPack
      return NULL;
  }
  
 +#define FF_MERGE_MARKER 0x8c4d9d108e25e9feULL
 +
 +int av_packet_merge_side_data(AVPacket *pkt){
 +    if(pkt->side_data_elems){
 +        AVBufferRef *buf;
 +        int i;
 +        uint8_t *p;
 +        uint64_t size= pkt->size + 8LL + FF_INPUT_BUFFER_PADDING_SIZE;
 +        AVPacket old= *pkt;
 +        for (i=0; i<old.side_data_elems; i++) {
 +            size += old.side_data[i].size + 5LL;
 +        }
 +        if (size > INT_MAX)
 +            return AVERROR(EINVAL);
 +        buf = av_buffer_alloc(size);
 +        if (!buf)
 +            return AVERROR(ENOMEM);
 +        pkt->buf = buf;
 +        pkt->data = p = buf->data;
 +#if FF_API_DESTRUCT_PACKET
 +FF_DISABLE_DEPRECATION_WARNINGS
 +        pkt->destruct = dummy_destruct_packet;
 +FF_ENABLE_DEPRECATION_WARNINGS
 +#endif
 +        pkt->size = size - FF_INPUT_BUFFER_PADDING_SIZE;
 +        bytestream_put_buffer(&p, old.data, old.size);
 +        for (i=old.side_data_elems-1; i>=0; i--) {
 +            bytestream_put_buffer(&p, old.side_data[i].data, old.side_data[i].size);
 +            bytestream_put_be32(&p, old.side_data[i].size);
 +            *p++ = old.side_data[i].type | ((i==old.side_data_elems-1)*128);
 +        }
 +        bytestream_put_be64(&p, FF_MERGE_MARKER);
 +        av_assert0(p-pkt->data == pkt->size);
 +        memset(p, 0, FF_INPUT_BUFFER_PADDING_SIZE);
 +        av_free_packet(&old);
 +        pkt->side_data_elems = 0;
 +        pkt->side_data = NULL;
 +        return 1;
 +    }
 +    return 0;
 +}
 +
 +int av_packet_split_side_data(AVPacket *pkt){
 +    if (!pkt->side_data_elems && pkt->size >12 && AV_RB64(pkt->data + pkt->size - 8) == FF_MERGE_MARKER){
 +        int i;
 +        unsigned int size;
 +        uint8_t *p;
 +
 +        p = pkt->data + pkt->size - 8 - 5;
 +        for (i=1; ; i++){
 +            size = AV_RB32(p);
 +            if (size>INT_MAX || p - pkt->data < size)
 +                return 0;
 +            if (p[4]&128)
 +                break;
 +            p-= size+5;
 +        }
 +
 +        pkt->side_data = av_malloc_array(i, sizeof(*pkt->side_data));
 +        if (!pkt->side_data)
 +            return AVERROR(ENOMEM);
 +
 +        p= pkt->data + pkt->size - 8 - 5;
 +        for (i=0; ; i++){
 +            size= AV_RB32(p);
 +            av_assert0(size<=INT_MAX && p - pkt->data >= size);
 +            pkt->side_data[i].data = av_mallocz(size + FF_INPUT_BUFFER_PADDING_SIZE);
 +            pkt->side_data[i].size = size;
 +            pkt->side_data[i].type = p[4]&127;
 +            if (!pkt->side_data[i].data)
 +                return AVERROR(ENOMEM);
 +            memcpy(pkt->side_data[i].data, p-size, size);
 +            pkt->size -= size + 5;
 +            if(p[4]&128)
 +                break;
 +            p-= size+5;
 +        }
 +        pkt->size -= 8;
 +        pkt->side_data_elems = i+1;
 +        return 1;
 +    }
 +    return 0;
 +}
 +
 +uint8_t *av_packet_pack_dictionary(AVDictionary *dict, int *size)
 +{
 +    AVDictionaryEntry *t = NULL;
 +    uint8_t *data = NULL;
 +    *size = 0;
 +
 +    if (!dict)
 +        return NULL;
 +
 +    while ((t = av_dict_get(dict, "", t, AV_DICT_IGNORE_SUFFIX))) {
 +        const size_t keylen   = strlen(t->key);
 +        const size_t valuelen = strlen(t->value);
 +        const size_t new_size = *size + keylen + 1 + valuelen + 1;
 +        uint8_t *const new_data = av_realloc(data, new_size);
 +
 +        if (!new_data)
 +            goto fail;
 +        data = new_data;
 +        if (new_size > INT_MAX)
 +            goto fail;
 +
 +        memcpy(data + *size, t->key, keylen + 1);
 +        memcpy(data + *size + keylen + 1, t->value, valuelen + 1);
 +
 +        *size = new_size;
 +    }
 +
 +    return data;
 +
 +fail:
 +    av_freep(&data);
 +    *size = 0;
 +    return NULL;
 +}
 +
 +int av_packet_unpack_dictionary(const uint8_t *data, int size, AVDictionary **dict)
 +{
 +    const uint8_t *end = data + size;
 +    int ret = 0;
 +
 +    if (!dict || !data || !size)
 +        return ret;
 +    if (size && end[-1])
 +        return AVERROR_INVALIDDATA;
 +    while (data < end) {
 +        const uint8_t *key = data;
 +        const uint8_t *val = data + strlen(key) + 1;
 +
 +        if (val >= end)
 +            return AVERROR_INVALIDDATA;
 +
 +        ret = av_dict_set(dict, key, val, 0);
 +        if (ret < 0)
 +            break;
 +        data = val + strlen(val) + 1;
 +    }
 +
 +    return ret;
 +}
 +
  int av_packet_shrink_side_data(AVPacket *pkt, enum AVPacketSideDataType type,
                                 int size)
  {
@@@ -507,7 -325,6 +507,6 @@@ int av_packet_copy_props(AVPacket *dst
      dst->convergence_duration = src->convergence_duration;
      dst->flags                = src->flags;
      dst->stream_index         = src->stream_index;
-     dst->side_data_elems      = src->side_data_elems;
  
      for (i = 0; i < src->side_data_elems; i++) {
           enum AVPacketSideDataType type = src->side_data[i].type;
@@@ -534,7 -351,7 +533,7 @@@ void av_packet_unref(AVPacket *pkt
      pkt->size = 0;
  }
  
 -int av_packet_ref(AVPacket *dst, AVPacket *src)
 +int av_packet_ref(AVPacket *dst, const AVPacket *src)
  {
      int ret;