X-Git-Url: https://rtime.felk.cvut.cz/gitweb/frescor/ffmpeg.git/blobdiff_plain/58a70780850cafd783b7cd63d18803f884981a88..e6b000b4e52e685bc6387faa806f36b5f56716e8:/libavformat/aviobuf.c diff --git a/libavformat/aviobuf.c b/libavformat/aviobuf.c index 0ad63299a..04e519916 100644 --- a/libavformat/aviobuf.c +++ b/libavformat/aviobuf.c @@ -2,76 +2,128 @@ * Buffered I/O for ffmpeg system * Copyright (c) 2000,2001 Fabrice Bellard * - * This library is free software; you can redistribute it and/or + * This file is part of FFmpeg. + * + * 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 of the License, or (at your option) any later version. + * version 2.1 of the License, or (at your option) any later version. * - * This library 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 this library; if not, write to the Free Software - * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA + * License along with FFmpeg; if not, write to the Free Software + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ + +#include "libavutil/crc.h" +#include "libavutil/intreadwrite.h" #include "avformat.h" #include "avio.h" #include +#include +#include #define IO_BUFFER_SIZE 32768 +static void fill_buffer(ByteIOContext *s); + int init_put_byte(ByteIOContext *s, unsigned char *buffer, int buffer_size, int write_flag, void *opaque, int (*read_packet)(void *opaque, uint8_t *buf, int buf_size), - void (*write_packet)(void *opaque, uint8_t *buf, int buf_size), - int (*seek)(void *opaque, offset_t offset, int whence)) + int (*write_packet)(void *opaque, uint8_t *buf, int buf_size), + int64_t (*seek)(void *opaque, int64_t offset, int whence)) { s->buffer = buffer; s->buffer_size = buffer_size; s->buf_ptr = buffer; - s->write_flag = write_flag; - if (!s->write_flag) - s->buf_end = buffer; - else - s->buf_end = buffer + buffer_size; s->opaque = opaque; + url_resetbuf(s, write_flag ? URL_WRONLY : URL_RDONLY); s->write_packet = write_packet; s->read_packet = read_packet; s->seek = seek; s->pos = 0; s->must_flush = 0; s->eof_reached = 0; + s->error = 0; s->is_streamed = 0; s->max_packet_size = 0; s->update_checksum= NULL; + if(!read_packet && !write_flag){ + s->pos = buffer_size; + s->buf_end = s->buffer + buffer_size; + } + s->read_pause = NULL; + s->read_seek = NULL; return 0; } - -#ifdef CONFIG_ENCODERS +ByteIOContext *av_alloc_put_byte( + unsigned char *buffer, + int buffer_size, + int write_flag, + void *opaque, + int (*read_packet)(void *opaque, uint8_t *buf, int buf_size), + int (*write_packet)(void *opaque, uint8_t *buf, int buf_size), + int64_t (*seek)(void *opaque, int64_t offset, int whence)) +{ + ByteIOContext *s = av_mallocz(sizeof(ByteIOContext)); + init_put_byte(s, buffer, buffer_size, write_flag, opaque, + read_packet, write_packet, seek); + return s; +} + +#define BLOCK_SIZE 512 + static void flush_buffer(ByteIOContext *s) { - if (s->buf_ptr > s->buffer) { - if (s->write_packet) - s->write_packet(s->opaque, s->buffer, s->buf_ptr - s->buffer); + size_t data_len = s->buf_ptr - s->buffer; + + /* Write all the data in the buffer to the disk together with any + * additional space at the end of the buffer up to the first + * multiple of BLOCK_SIZE. The last block is always written to the + * disk despite not being fully filled. If the last block is not + * fully filled, we keep it (moved to the begginig of the buffer) + * and any subsequent call to this function will rewrite the block + * together with the additional data added to the buffer. */ + size_t to_write = (data_len + BLOCK_SIZE - 1) & ~(BLOCK_SIZE - 1); + size_t whole_blocks = data_len & ~(BLOCK_SIZE - 1); + int64_t res = AVERROR(EPIPE); + av_log(NULL, AV_LOG_ERROR, "YYY dl=%zd, tw=%zd\n", data_len, to_write); + if (data_len > 0) { + if (s->write_packet && !s->error){ + int ret= s->write_packet(s->opaque, s->buffer, to_write); + if(ret < 0){ + s->error = ret; + } + } if(s->update_checksum){ + av_log(NULL, AV_LOG_ERROR, "Checksumming not implemented in O_DIRECT patch\n"); s->checksum= s->update_checksum(s->checksum, s->checksum_ptr, s->buf_ptr - s->checksum_ptr); s->checksum_ptr= s->buffer; } - s->pos += s->buf_ptr - s->buffer; + s->pos += whole_blocks; + memcpy(s->buffer, &s->buffer[whole_blocks], data_len - whole_blocks); + s->buf_ptr -= whole_blocks; + + if (s->seek) + res = s->seek(s->opaque, s->pos, SEEK_SET); + if (res < 0) + av_log(NULL, AV_LOG_ERROR, "seek error inside flush_buffer: %lld\n", res); + av_log(NULL, AV_LOG_ERROR, "ZZZ tw=%zu, pos=%zd\n", to_write, s->pos); } - s->buf_ptr = s->buffer; } void put_byte(ByteIOContext *s, int b) { *(s->buf_ptr)++ = b; - if (s->buf_ptr >= s->buf_end) + if (s->buf_ptr >= s->buf_end) flush_buffer(s); } @@ -86,7 +138,7 @@ void put_buffer(ByteIOContext *s, const unsigned char *buf, int size) memcpy(s->buf_ptr, buf, len); s->buf_ptr += len; - if (s->buf_ptr >= s->buf_end) + if (s->buf_ptr >= s->buf_end) flush_buffer(s); buf += len; @@ -99,80 +151,104 @@ void put_flush_packet(ByteIOContext *s) flush_buffer(s); s->must_flush = 0; } -#endif //CONFIG_ENCODERS -offset_t url_fseek(ByteIOContext *s, offset_t offset, int whence) +int64_t url_fseek(ByteIOContext *s, int64_t offset, int whence) { - offset_t offset1; + int64_t offset1; + int64_t pos; + + if(!s) + return AVERROR(EINVAL); + + pos = s->pos - (s->write_flag ? 0 : (s->buf_end - s->buffer)); if (whence != SEEK_CUR && whence != SEEK_SET) - return -EINVAL; - -#ifdef CONFIG_ENCODERS - if (s->write_flag) { - if (whence == SEEK_CUR) { - offset1 = s->pos + (s->buf_ptr - s->buffer); - if (offset == 0) - return offset1; - offset += offset1; - } - offset1 = offset - s->pos; - if (!s->must_flush && - offset1 >= 0 && offset1 < (s->buf_end - s->buffer)) { - /* can do the seek inside the buffer */ - s->buf_ptr = s->buffer + offset1; - } else { - if (!s->seek) - return -EPIPE; + return AVERROR(EINVAL); + + if (whence == SEEK_CUR) { + offset1 = pos + (s->buf_ptr - s->buffer); + if (offset == 0) + return offset1; + offset += offset1; + } + av_log(NULL, AV_LOG_ERROR, "SEEK to %llu\n", offset); + + offset1 = offset - pos; + if (!s->must_flush && + offset1 >= 0 && offset1 <= (s->buf_end - s->buffer)) { + /* can do the seek inside the buffer */ + s->buf_ptr = s->buffer + offset1; + } else if(s->is_streamed && !s->write_flag && + offset1 >= 0 && offset1 < (s->buf_end - s->buffer) + (1<<16)){ + while(s->pos < offset && !s->eof_reached) + fill_buffer(s); + if (s->eof_reached) + return AVERROR(EPIPE); + s->buf_ptr = s->buf_end + offset - s->pos; + } else { + int64_t res = AVERROR(EPIPE); + +#if CONFIG_MUXERS || CONFIG_NETWORK + if (s->write_flag) { flush_buffer(s); s->must_flush = 1; - s->buf_ptr = s->buffer; - s->seek(s->opaque, offset, SEEK_SET); - s->pos = offset; - } - } else -#endif //CONFIG_ENCODERS - { - if (whence == SEEK_CUR) { - offset1 = s->pos - (s->buf_end - s->buffer) + (s->buf_ptr - s->buffer); - if (offset == 0) - return offset1; - offset += offset1; } - offset1 = offset - (s->pos - (s->buf_end - s->buffer)); - if (offset1 >= 0 && offset1 <= (s->buf_end - s->buffer)) { - /* can do the seek inside the buffer */ - s->buf_ptr = s->buffer + offset1; - } else { - if (!s->seek) - return -EPIPE; - s->buf_ptr = s->buffer; +#endif /* CONFIG_MUXERS || CONFIG_NETWORK */ +#warning TODO read the whole block from the file to the buffer + if (!s->seek || (res = s->seek(s->opaque, offset, SEEK_SET)) < 0) + return res; + if (!s->write_flag) s->buf_end = s->buffer; - if (s->seek(s->opaque, offset, SEEK_SET) == (offset_t)-EPIPE) - return -EPIPE; - s->pos = offset; - } - s->eof_reached = 0; + s->buf_ptr = s->buffer; + s->pos = offset; } + s->eof_reached = 0; return offset; } -void url_fskip(ByteIOContext *s, offset_t offset) +void url_fskip(ByteIOContext *s, int64_t offset) { url_fseek(s, offset, SEEK_CUR); } -offset_t url_ftell(ByteIOContext *s) +int64_t url_ftell(ByteIOContext *s) { return url_fseek(s, 0, SEEK_CUR); } +int64_t url_fsize(ByteIOContext *s) +{ + int64_t size; + + if(!s) + return AVERROR(EINVAL); + + if (!s->seek) + return AVERROR(EPIPE); + size = s->seek(s->opaque, 0, AVSEEK_SIZE); + if(size<0){ + if ((size = s->seek(s->opaque, -1, SEEK_END)) < 0) + return size; + size++; + s->seek(s->opaque, s->pos, SEEK_SET); + } + return size; +} + int url_feof(ByteIOContext *s) { + if(!s) + return 0; return s->eof_reached; } -#ifdef CONFIG_ENCODERS +int url_ferror(ByteIOContext *s) +{ + if(!s) + return 0; + return s->error; +} + void put_le32(ByteIOContext *s, unsigned int val) { put_byte(s, val); @@ -189,17 +265,6 @@ void put_be32(ByteIOContext *s, unsigned int val) put_byte(s, val); } -/* IEEE format is assumed */ -void put_be64_double(ByteIOContext *s, double val) -{ - union { - double d; - uint64_t ull; - } u; - u.d = val; - put_be64(s, u.ull); -} - void put_strz(ByteIOContext *s, const char *str) { if (str) @@ -232,57 +297,85 @@ void put_be16(ByteIOContext *s, unsigned int val) put_byte(s, val); } +void put_le24(ByteIOContext *s, unsigned int val) +{ + put_le16(s, val & 0xffff); + put_byte(s, val >> 16); +} + +void put_be24(ByteIOContext *s, unsigned int val) +{ + put_be16(s, val >> 8); + put_byte(s, val); +} + void put_tag(ByteIOContext *s, const char *tag) { while (*tag) { put_byte(s, *tag++); } } -#endif //CONFIG_ENCODERS /* Input stream */ static void fill_buffer(ByteIOContext *s) { - int len; + uint8_t *dst= !s->max_packet_size && s->buf_end - s->buffer < s->buffer_size ? s->buf_ptr : s->buffer; + int len= s->buffer_size - (dst - s->buffer); + + assert(s->buf_ptr == s->buf_end); /* no need to do anything if EOF already reached */ if (s->eof_reached) return; - if(s->update_checksum){ - s->checksum= s->update_checksum(s->checksum, s->checksum_ptr, s->buf_end - s->checksum_ptr); + if(s->update_checksum && dst == s->buffer){ + if(s->buf_end > s->checksum_ptr) + s->checksum= s->update_checksum(s->checksum, s->checksum_ptr, s->buf_end - s->checksum_ptr); s->checksum_ptr= s->buffer; } - len = s->read_packet(s->opaque, s->buffer, s->buffer_size); + if(s->read_packet) + len = s->read_packet(s->opaque, dst, len); + else + len = 0; if (len <= 0) { /* do not modify buffer if EOF reached so that a seek back can be done without rereading data */ s->eof_reached = 1; + if(len<0) + s->error= len; } else { s->pos += len; - s->buf_ptr = s->buffer; - s->buf_end = s->buffer + len; + s->buf_ptr = dst; + s->buf_end = dst + len; } } -unsigned long get_checksum(ByteIOContext *s){ +unsigned long ff_crc04C11DB7_update(unsigned long checksum, const uint8_t *buf, + unsigned int len) +{ + return av_crc(av_crc_get_table(AV_CRC_32_IEEE), checksum, buf, len); +} + +unsigned long get_checksum(ByteIOContext *s) +{ s->checksum= s->update_checksum(s->checksum, s->checksum_ptr, s->buf_ptr - s->checksum_ptr); s->update_checksum= NULL; return s->checksum; } -void init_checksum(ByteIOContext *s, unsigned long (*update_checksum)(unsigned long c, const uint8_t *p, unsigned int len), unsigned long checksum){ +void init_checksum(ByteIOContext *s, + unsigned long (*update_checksum)(unsigned long c, const uint8_t *p, unsigned int len), + unsigned long checksum) +{ s->update_checksum= update_checksum; if(s->update_checksum){ - s->checksum= s->update_checksum(checksum, NULL, 0); + s->checksum= checksum; s->checksum_ptr= s->buf_ptr; } } -/* NOTE: return 0 if EOF, so you cannot use it if EOF handling is - necessary */ /* XXX: put an inline version */ int get_byte(ByteIOContext *s) { @@ -297,7 +390,6 @@ int get_byte(ByteIOContext *s) } } -/* NOTE: return URL_EOF (-1) if EOF */ int url_fgetc(ByteIOContext *s) { if (s->buf_ptr < s->buf_end) { @@ -321,10 +413,29 @@ int get_buffer(ByteIOContext *s, unsigned char *buf, int size) if (len > size) len = size; if (len == 0) { - fill_buffer(s); - len = s->buf_end - s->buf_ptr; - if (len == 0) - break; + if(size > s->buffer_size && !s->update_checksum){ + if(s->read_packet) + len = s->read_packet(s->opaque, buf, size); + if (len <= 0) { + /* do not modify buffer if EOF reached so that a seek back can + be done without rereading data */ + s->eof_reached = 1; + if(len<0) + s->error= len; + break; + } else { + s->pos += len; + size -= len; + buf += len; + s->buf_ptr = s->buffer; + s->buf_end = s->buffer/* + len*/; + } + }else{ + fill_buffer(s); + len = s->buf_end - s->buf_ptr; + if (len == 0) + break; + } } else { memcpy(buf, s->buf_ptr, len); buf += len; @@ -339,6 +450,9 @@ int get_partial_buffer(ByteIOContext *s, unsigned char *buf, int size) { int len; + if(size<0) + return -1; + len = s->buf_end - s->buf_ptr; if (len == 0) { fill_buffer(s); @@ -359,13 +473,19 @@ unsigned int get_le16(ByteIOContext *s) return val; } -unsigned int get_le32(ByteIOContext *s) +unsigned int get_le24(ByteIOContext *s) { unsigned int val; - val = get_byte(s); - val |= get_byte(s) << 8; + val = get_le16(s); val |= get_byte(s) << 16; - val |= get_byte(s) << 24; + return val; +} + +unsigned int get_le32(ByteIOContext *s) +{ + unsigned int val; + val = get_le16(s); + val |= get_le16(s) << 16; return val; } @@ -385,25 +505,19 @@ unsigned int get_be16(ByteIOContext *s) return val; } -unsigned int get_be32(ByteIOContext *s) +unsigned int get_be24(ByteIOContext *s) { unsigned int val; - val = get_byte(s) << 24; - val |= get_byte(s) << 16; - val |= get_byte(s) << 8; + val = get_be16(s) << 8; val |= get_byte(s); return val; } - -double get_be64_double(ByteIOContext *s) +unsigned int get_be32(ByteIOContext *s) { - union { - double d; - uint64_t ull; - } u; - - u.ull = get_be64(s); - return u.d; + unsigned int val; + val = get_be16(s) << 16; + val |= get_be16(s); + return val; } char *get_strz(ByteIOContext *s, char *buf, int maxlen) @@ -415,7 +529,7 @@ char *get_strz(ByteIOContext *s, char *buf, int maxlen) if (i < maxlen-1) buf[i++] = c; } - + buf[i] = 0; /* Ensure null terminated, but may be truncated */ return buf; @@ -429,80 +543,88 @@ uint64_t get_be64(ByteIOContext *s) return val; } -/* link with avio functions */ +uint64_t ff_get_v(ByteIOContext *bc){ + uint64_t val = 0; + int tmp; -#ifdef CONFIG_ENCODERS -static void url_write_packet(void *opaque, uint8_t *buf, int buf_size) -{ - URLContext *h = opaque; - url_write(h, buf, buf_size); -} -#else -#define url_write_packet NULL -#endif //CONFIG_ENCODERS - -static int url_read_packet(void *opaque, uint8_t *buf, int buf_size) -{ - URLContext *h = opaque; - return url_read(h, buf, buf_size); -} - -static int url_seek_packet(void *opaque, int64_t offset, int whence) -{ - URLContext *h = opaque; - return url_seek(h, offset, whence); - //return 0; + do{ + tmp = get_byte(bc); + val= (val<<7) + (tmp&127); + }while(tmp&128); + return val; } -int url_fdopen(ByteIOContext *s, URLContext *h) +int url_fdopen(ByteIOContext **s, URLContext *h) { uint8_t *buffer; int buffer_size, max_packet_size; - + max_packet_size = url_get_max_packet_size(h); if (max_packet_size) { buffer_size = max_packet_size; /* no need to bufferize more than one packet */ } else { buffer_size = IO_BUFFER_SIZE; } - buffer = av_malloc(buffer_size); + buffer_size = 2*buffer_size+BLOCK_SIZE; + buffer = memalign(sysconf(_SC_PAGESIZE), buffer_size); if (!buffer) - return -ENOMEM; + return AVERROR(ENOMEM); + + *s = av_mallocz(sizeof(ByteIOContext)); + if(!*s) { + av_free(buffer); + return AVERROR(ENOMEM); + } - if (init_put_byte(s, buffer, buffer_size, + if (init_put_byte(*s, buffer, buffer_size, (h->flags & URL_WRONLY || h->flags & URL_RDWR), h, - url_read_packet, url_write_packet, url_seek_packet) < 0) { + url_read, url_write, url_seek) < 0) { av_free(buffer); - return AVERROR_IO; + av_freep(s); + return AVERROR(EIO); + } + (*s)->is_streamed = h->is_streamed; + (*s)->max_packet_size = max_packet_size; + if(h->prot) { + (*s)->read_pause = (int (*)(void *, int))h->prot->url_read_pause; + (*s)->read_seek = (int64_t (*)(void *, int, int64_t, int))h->prot->url_read_seek; } - s->is_streamed = h->is_streamed; - s->max_packet_size = max_packet_size; return 0; } -/* XXX: must be called before any I/O */ int url_setbufsize(ByteIOContext *s, int buf_size) { uint8_t *buffer; buffer = av_malloc(buf_size); if (!buffer) - return -ENOMEM; + return AVERROR(ENOMEM); av_free(s->buffer); s->buffer = buffer; s->buffer_size = buf_size; s->buf_ptr = buffer; - if (!s->write_flag) - s->buf_end = buffer; - else - s->buf_end = buffer + buf_size; + url_resetbuf(s, s->write_flag ? URL_WRONLY : URL_RDONLY); + return 0; +} + +int url_resetbuf(ByteIOContext *s, int flags) +{ + URLContext *h = s->opaque; + if ((flags & URL_RDWR) || (h && h->flags != flags && !h->flags & URL_RDWR)) + return AVERROR(EINVAL); + + if (flags & URL_WRONLY) { + s->buf_end = s->buffer + s->buffer_size; + s->write_flag = 1; + } else { + s->buf_end = s->buffer; + s->write_flag = 0; + } return 0; } -/* NOTE: when opened as read/write, the buffers are only used for - reading */ -int url_fopen(ByteIOContext *s, const char *filename, int flags) +int url_fopen(ByteIOContext **s, const char *filename, int flags) { URLContext *h; int err; @@ -521,9 +643,9 @@ int url_fopen(ByteIOContext *s, const char *filename, int flags) int url_fclose(ByteIOContext *s) { URLContext *h = s->opaque; - + av_free(s->buffer); - memset(s, 0, sizeof(ByteIOContext)); + av_free(s); return url_close(h); } @@ -532,8 +654,7 @@ URLContext *url_fileno(ByteIOContext *s) return s->opaque; } -#ifdef CONFIG_ENCODERS -/* XXX: currently size is limited */ +#if CONFIG_MUXERS int url_fprintf(ByteIOContext *s, const char *fmt, ...) { va_list ap; @@ -546,10 +667,8 @@ int url_fprintf(ByteIOContext *s, const char *fmt, ...) put_buffer(s, buf, strlen(buf)); return ret; } -#endif //CONFIG_ENCODERS +#endif //CONFIG_MUXERS -/* note: unlike fgets, the EOL character is not returned and a whole - line is parsed. return NULL if first char read was EOF */ char *url_fgets(ByteIOContext *s, char *buf, int buf_size) { int c; @@ -571,29 +690,51 @@ char *url_fgets(ByteIOContext *s, char *buf, int buf_size) return buf; } -/* - * Return the maximum packet size associated to packetized buffered file - * handle. If the file is not packetized (stream like http or file on - * disk), then 0 is returned. - * - * @param h buffered file handle - * @return maximum packet size in bytes - */ int url_fget_max_packet_size(ByteIOContext *s) { return s->max_packet_size; } -#ifdef CONFIG_ENCODERS +int av_url_read_fpause(ByteIOContext *s, int pause) +{ + if (!s->read_pause) + return AVERROR(ENOSYS); + return s->read_pause(s->opaque, pause); +} + +int64_t av_url_read_fseek(ByteIOContext *s, int stream_index, + int64_t timestamp, int flags) +{ + URLContext *h = s->opaque; + int64_t ret; + if (!s->read_seek) + return AVERROR(ENOSYS); + ret = s->read_seek(h, stream_index, timestamp, flags); + if(ret >= 0) { + s->buf_ptr = s->buf_end; // Flush buffer + s->pos = s->seek(h, 0, SEEK_CUR); + } + return ret; +} + +/* url_open_dyn_buf and url_close_dyn_buf are used in rtp.c to send a response + * back to the server even if CONFIG_MUXERS is false. */ +#if CONFIG_MUXERS || CONFIG_NETWORK /* buffer handling */ -int url_open_buf(ByteIOContext *s, uint8_t *buf, int buf_size, int flags) +int url_open_buf(ByteIOContext **s, uint8_t *buf, int buf_size, int flags) { - return init_put_byte(s, buf, buf_size, - (flags & URL_WRONLY || flags & URL_RDWR), - NULL, NULL, NULL, NULL); + int ret; + *s = av_mallocz(sizeof(ByteIOContext)); + if(!*s) + return AVERROR(ENOMEM); + ret = init_put_byte(*s, buf, buf_size, + (flags & URL_WRONLY || flags & URL_RDWR), + NULL, NULL, NULL, NULL); + if(ret != 0) + av_freep(s); + return ret; } -/* return the written or read size */ int url_close_buf(ByteIOContext *s) { put_flush_packet(s); @@ -609,49 +750,52 @@ typedef struct DynBuffer { uint8_t io_buffer[1]; } DynBuffer; -static void dyn_buf_write(void *opaque, uint8_t *buf, int buf_size) +static int dyn_buf_write(void *opaque, uint8_t *buf, int buf_size) { DynBuffer *d = opaque; - int new_size, new_allocated_size; - + unsigned new_size, new_allocated_size; + /* reallocate buffer if needed */ new_size = d->pos + buf_size; new_allocated_size = d->allocated_size; + if(new_size < d->pos || new_size > INT_MAX/2) + return -1; while (new_size > new_allocated_size) { if (!new_allocated_size) new_allocated_size = new_size; else - new_allocated_size = (new_allocated_size * 3) / 2 + 1; + new_allocated_size += new_allocated_size / 2 + 1; } - + if (new_allocated_size > d->allocated_size) { d->buffer = av_realloc(d->buffer, new_allocated_size); if(d->buffer == NULL) - return ; + return AVERROR(ENOMEM); d->allocated_size = new_allocated_size; } memcpy(d->buffer + d->pos, buf, buf_size); d->pos = new_size; if (d->pos > d->size) d->size = d->pos; + return buf_size; } -static void dyn_packet_buf_write(void *opaque, uint8_t *buf, int buf_size) +static int dyn_packet_buf_write(void *opaque, uint8_t *buf, int buf_size) { unsigned char buf1[4]; + int ret; /* packetized write: output the header */ - buf1[0] = (buf_size >> 24); - buf1[1] = (buf_size >> 16); - buf1[2] = (buf_size >> 8); - buf1[3] = (buf_size); - dyn_buf_write(opaque, buf1, 4); + AV_WB32(buf1, buf_size); + ret= dyn_buf_write(opaque, buf1, 4); + if(ret < 0) + return ret; /* then the data */ - dyn_buf_write(opaque, buf, buf_size); + return dyn_buf_write(opaque, buf, buf_size); } -static int dyn_buf_seek(void *opaque, offset_t offset, int whence) +static int64_t dyn_buf_seek(void *opaque, int64_t offset, int whence) { DynBuffer *d = opaque; @@ -665,68 +809,48 @@ static int dyn_buf_seek(void *opaque, offset_t offset, int whence) return 0; } -static int url_open_dyn_buf_internal(ByteIOContext *s, int max_packet_size) +static int url_open_dyn_buf_internal(ByteIOContext **s, int max_packet_size) { DynBuffer *d; - int io_buffer_size, ret; - - if (max_packet_size) - io_buffer_size = max_packet_size; - else - io_buffer_size = 1024; - - d = av_malloc(sizeof(DynBuffer) + io_buffer_size); - if (!d) + int ret; + unsigned io_buffer_size = max_packet_size ? max_packet_size : 1024; + + if(sizeof(DynBuffer) + io_buffer_size < io_buffer_size) return -1; + d = av_mallocz(sizeof(DynBuffer) + io_buffer_size); + if (!d) + return AVERROR(ENOMEM); + *s = av_mallocz(sizeof(ByteIOContext)); + if(!*s) { + av_free(d); + return AVERROR(ENOMEM); + } d->io_buffer_size = io_buffer_size; - d->buffer = NULL; - d->pos = 0; - d->size = 0; - d->allocated_size = 0; - ret = init_put_byte(s, d->io_buffer, io_buffer_size, - 1, d, NULL, - max_packet_size ? dyn_packet_buf_write : dyn_buf_write, + ret = init_put_byte(*s, d->io_buffer, io_buffer_size, + 1, d, NULL, + max_packet_size ? dyn_packet_buf_write : dyn_buf_write, max_packet_size ? NULL : dyn_buf_seek); if (ret == 0) { - s->max_packet_size = max_packet_size; + (*s)->max_packet_size = max_packet_size; + } else { + av_free(d); + av_freep(s); } return ret; } -/* - * Open a write only memory stream. - * - * @param s new IO context - * @return zero if no error. - */ -int url_open_dyn_buf(ByteIOContext *s) +int url_open_dyn_buf(ByteIOContext **s) { return url_open_dyn_buf_internal(s, 0); } -/* - * Open a write only packetized memory stream with a maximum packet - * size of 'max_packet_size'. The stream is stored in a memory buffer - * with a big endian 4 byte header giving the packet size in bytes. - * - * @param s new IO context - * @param max_packet_size maximum packet size (must be > 0) - * @return zero if no error. - */ -int url_open_dyn_packet_buf(ByteIOContext *s, int max_packet_size) +int url_open_dyn_packet_buf(ByteIOContext **s, int max_packet_size) { if (max_packet_size <= 0) return -1; return url_open_dyn_buf_internal(s, max_packet_size); } -/* - * Return the written size and a pointer to the buffer. The buffer - * must be freed with av_free(). - * @param s IO context - * @param pointer to a byte buffer - * @return the length of the byte buffer - */ int url_close_dyn_buf(ByteIOContext *s, uint8_t **pbuffer) { DynBuffer *d = s->opaque; @@ -737,6 +861,7 @@ int url_close_dyn_buf(ByteIOContext *s, uint8_t **pbuffer) *pbuffer = d->buffer; size = d->size; av_free(d); + av_free(s); return size; } -#endif //CONFIG_ENCODERS +#endif /* CONFIG_MUXERS || CONFIG_NETWORK */