summaryrefslogtreecommitdiff
path: root/libavformat/wavdec.c
diff options
context:
space:
mode:
authorMark Harris <mark.hsj@gmail.com>2016-12-15 16:44:57 -0800
committerMichael Niedermayer <michael@niedermayer.cc>2016-12-16 03:00:03 +0100
commita5cf600ccb7f20b65d552fef5c5d010513fcf74c (patch)
tree5adb2376f08c91087b7da031d9787a05914ce746 /libavformat/wavdec.c
parentd4c1cc2b876af1e8f1a8ca258bf092b3baa31245 (diff)
avformat/wavdec: Eliminate goto for clang -O0 DCE
Clang is not able to eliminate the reference to ff_spdif_probe() when there is a goto target in the same block and optimization is disabled. This fixes the following build failure on OS X: ./configure --disable-everything --disable-doc \ --enable-decoder=pcm_s16le --enable-demuxer=wav \ --enable-protocol=file --disable-optimizations --cc=clang make ... Undefined symbols for architecture x86_64: "_ff_spdif_probe", referenced from: _set_spdif in libavformat.a(wavdec.o) ld: symbol(s) not found for architecture x86_64 Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
Diffstat (limited to 'libavformat/wavdec.c')
-rw-r--r--libavformat/wavdec.c38
1 files changed, 17 insertions, 21 deletions
diff --git a/libavformat/wavdec.c b/libavformat/wavdec.c
index 7176cd6f2d..ae42a6167f 100644
--- a/libavformat/wavdec.c
+++ b/libavformat/wavdec.c
@@ -64,34 +64,30 @@ static void set_spdif(AVFormatContext *s, WAVDemuxContext *wav)
{
if (CONFIG_SPDIF_DEMUXER && s->streams[0]->codecpar->codec_tag == 1) {
enum AVCodecID codec;
- uint8_t *buf = NULL;
int len = 1<<16;
int ret = ffio_ensure_seekback(s->pb, len);
- int64_t pos = avio_tell(s->pb);
- if (ret < 0)
- goto end;
-
- buf = av_malloc(len);
- if (!buf) {
- ret = AVERROR(ENOMEM);
- goto end;
+ if (ret >= 0) {
+ uint8_t *buf = av_malloc(len);
+ if (!buf) {
+ ret = AVERROR(ENOMEM);
+ } else {
+ int64_t pos = avio_tell(s->pb);
+ len = ret = avio_read(s->pb, buf, len);
+ if (len >= 0) {
+ ret = ff_spdif_probe(buf, len, &codec);
+ if (ret > AVPROBE_SCORE_EXTENSION) {
+ s->streams[0]->codecpar->codec_id = codec;
+ wav->spdif = 1;
+ }
+ }
+ avio_seek(s->pb, pos, SEEK_SET);
+ av_free(buf);
+ }
}
- len = ret = avio_read(s->pb, buf, len);
- if (ret < 0)
- goto end;
-
- ret = ff_spdif_probe(buf, len, &codec);
- if (ret > AVPROBE_SCORE_EXTENSION) {
- s->streams[0]->codecpar->codec_id = codec;
- wav->spdif = 1;
- }
-end:
- avio_seek(s->pb, pos, SEEK_SET);
if (ret < 0)
av_log(s, AV_LOG_WARNING, "Cannot check for SPDIF\n");
- av_free(buf);
}
}