Commit 15ed7ca4 authored by Michael Niedermayer's avatar Michael Niedermayer

Merge commit 'fd9badd3'

* commit 'fd9badd3':
  xwma: Do not leak on failure path

Conflicts:
	libavformat/xwma.c

See: 375a0c03Merged-by: 's avatarMichael Niedermayer <michaelni@gmx.at>
parents 5af0a701 fd9badd3
...@@ -44,7 +44,7 @@ static int xwma_probe(AVProbeData *p) ...@@ -44,7 +44,7 @@ static int xwma_probe(AVProbeData *p)
static int xwma_read_header(AVFormatContext *s) static int xwma_read_header(AVFormatContext *s)
{ {
int64_t size; int64_t size;
int ret; int ret = 0;
uint32_t dpds_table_size = 0; uint32_t dpds_table_size = 0;
uint32_t *dpds_table = NULL; uint32_t *dpds_table = NULL;
unsigned int tag; unsigned int tag;
...@@ -132,7 +132,7 @@ static int xwma_read_header(AVFormatContext *s) ...@@ -132,7 +132,7 @@ static int xwma_read_header(AVFormatContext *s)
for (;;) { for (;;) {
if (pb->eof_reached) { if (pb->eof_reached) {
ret = AVERROR_EOF; ret = AVERROR_EOF;
goto end; goto fail;
} }
/* read next chunk tag */ /* read next chunk tag */
tag = avio_rl32(pb); tag = avio_rl32(pb);
...@@ -155,7 +155,7 @@ static int xwma_read_header(AVFormatContext *s) ...@@ -155,7 +155,7 @@ static int xwma_read_header(AVFormatContext *s)
if (dpds_table) { if (dpds_table) {
av_log(s, AV_LOG_ERROR, "two dpds chunks present\n"); av_log(s, AV_LOG_ERROR, "two dpds chunks present\n");
ret = AVERROR_INVALIDDATA; ret = AVERROR_INVALIDDATA;
goto end; goto fail;
} }
/* Compute the number of entries in the dpds chunk. */ /* Compute the number of entries in the dpds chunk. */
...@@ -189,7 +189,7 @@ static int xwma_read_header(AVFormatContext *s) ...@@ -189,7 +189,7 @@ static int xwma_read_header(AVFormatContext *s)
/* Determine overall data length */ /* Determine overall data length */
if (size < 0) { if (size < 0) {
ret = AVERROR_INVALIDDATA; ret = AVERROR_INVALIDDATA;
goto end; goto fail;
} }
if (!size) { if (!size) {
xwma->data_end = INT64_MAX; xwma->data_end = INT64_MAX;
...@@ -210,7 +210,7 @@ static int xwma_read_header(AVFormatContext *s) ...@@ -210,7 +210,7 @@ static int xwma_read_header(AVFormatContext *s)
"Invalid bits_per_coded_sample %d for %d channels\n", "Invalid bits_per_coded_sample %d for %d channels\n",
st->codec->bits_per_coded_sample, st->codec->channels); st->codec->bits_per_coded_sample, st->codec->channels);
ret = AVERROR_INVALIDDATA; ret = AVERROR_INVALIDDATA;
goto end; goto fail;
} }
st->duration = total_decoded_bytes / bytes_per_sample; st->duration = total_decoded_bytes / bytes_per_sample;
...@@ -245,7 +245,7 @@ static int xwma_read_header(AVFormatContext *s) ...@@ -245,7 +245,7 @@ static int xwma_read_header(AVFormatContext *s)
st->duration = (size<<3) * st->codec->sample_rate / st->codec->bit_rate; st->duration = (size<<3) * st->codec->sample_rate / st->codec->bit_rate;
} }
end: fail:
av_free(dpds_table); av_free(dpds_table);
return ret; return ret;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment