aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMichal Nazarewicz <mina86@mina86.com>2015-03-23 19:14:54 +0100
committerMax Kellermann <max@duempel.org>2015-03-25 18:02:54 +0100
commit0d7ee2b014db4f5072ac17839d0c7f3535c60675 (patch)
tree6c25292077d5b4d2915f3f94aa6fa8942ad3f833
parent2f5fd91bd8cb11ce064b4002c1ed47dd7fb0d6f6 (diff)
downloadmpd-0d7ee2b014db4f5072ac17839d0c7f3535c60675.tar.gz
mpd-0d7ee2b014db4f5072ac17839d0c7f3535c60675.tar.xz
mpd-0d7ee2b014db4f5072ac17839d0c7f3535c60675.zip
OpusEncoderPlugin: initialise granulepos so we end up with sane values
Not initialising granulepos leads to it having arbitrary values in the encoded stream including possibly negative values which are not valid and confuse opusdec. Explicitly initialise opus_encoder::granulepos to avoid that problem.
-rw-r--r--NEWS2
-rw-r--r--src/encoder/plugins/OpusEncoderPlugin.cxx2
2 files changed, 3 insertions, 1 deletions
diff --git a/NEWS b/NEWS
index 713550034..73f1b403a 100644
--- a/NEWS
+++ b/NEWS
@@ -1,4 +1,6 @@
ver 0.19.10 (not yet released)
+* encoder
+ - opus: fix bogus granulepos
ver 0.19.9 (2015/02/06)
* decoder
diff --git a/src/encoder/plugins/OpusEncoderPlugin.cxx b/src/encoder/plugins/OpusEncoderPlugin.cxx
index 27b614b86..2b52228cb 100644
--- a/src/encoder/plugins/OpusEncoderPlugin.cxx
+++ b/src/encoder/plugins/OpusEncoderPlugin.cxx
@@ -66,7 +66,7 @@ struct opus_encoder {
ogg_int64_t granulepos;
- opus_encoder():encoder(opus_encoder_plugin) {}
+ opus_encoder():encoder(opus_encoder_plugin), granulepos(0) {}
};
static constexpr Domain opus_encoder_domain("opus_encoder");