From: Antonio Larrosa <larrosa@kde.org>
Date: Mon, 6 Mar 2017 12:51:22 +0100
Subject: Always check the number of coefficients

When building the library with NDEBUG, asserts are eliminated
so it's better to always check that the number of coefficients
is inside the array range.

This fixes the 00191-audiofile-indexoob issue in #41
---
 libaudiofile/WAVE.cpp | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/libaudiofile/WAVE.cpp b/libaudiofile/WAVE.cpp
index 9dd8511..0fc48e8 100644
--- a/libaudiofile/WAVE.cpp
+++ b/libaudiofile/WAVE.cpp
@@ -281,6 +281,12 @@ status WAVEFile::parseFormat(const Tag &id, uint32_t size)
 
 			/* numCoefficients should be at least 7. */
 			assert(numCoefficients >= 7 && numCoefficients <= 255);
+			if (numCoefficients < 7 || numCoefficients > 255)
+			{
+				_af_error(AF_BAD_HEADER,
+						"Bad number of coefficients");
+				return AF_FAIL;
+			}
 
 			m_msadpcmNumCoefficients = numCoefficients;
 
 class='tabs'><tr><td>
<a href='/guix/about/'>about</a><a href='/guix/'>summary</a><a href='/guix/refs/?id=f739b5f408e63cb21e0fbcb54e7d85923069de3a'>refs</a><a class='active' href='/guix/log/tests/keys'>log</a><a href='/guix/tree/tests/keys?id=f739b5f408e63cb21e0fbcb54e7d85923069de3a'>tree</a><a href='/guix/commit/tests/keys?id=f739b5f408e63cb21e0fbcb54e7d85923069de3a'>commit</a><a href='/guix/diff/tests/keys?id=f739b5f408e63cb21e0fbcb54e7d85923069de3a'>diff</a></td><td class='form'><form class='right' method='get' action='/guix/log/tests/keys'>
<input type='hidden' name='id' value='f739b5f408e63cb21e0fbcb54e7d85923069de3a'/><select name='qt'>
<option value='grep'>log msg</option>
<option value='author'>author</option>
<option value='committer'>committer</option>
<option value='range'>range</option>
</select>
<input class='txt' type='search' size='10' name='q' value=''/>
<input type='submit' value='search'/>
</form>
</td></tr></table>
<div class='path'>path: <a href='/guix/log/?id=f739b5f408e63cb21e0fbcb54e7d85923069de3a'>root</a>/<a href='/guix/log/tests?id=f739b5f408e63cb21e0fbcb54e7d85923069de3a'>tests</a>/<a href='/guix/log/tests/keys?id=f739b5f408e63cb21e0fbcb54e7d85923069de3a'>keys</a></div><div class='content'><table class='list nowrap'><tr class='nohover'><th class='left'>Age</th><th class='left'>Commit message (<a href='/guix/log/tests/keys?id=f739b5f408e63cb21e0fbcb54e7d85923069de3a&amp;showmsg=1'>Expand</a>)</th><th class='left'>Author</th></tr>
<tr><td><span title='2022-05-18 23:20:21 +0200'>2022-05-18</span></td><td><a href='/guix/commit/tests/keys?id=3ae7632ca0a1edca9d8c3c766efb0dcc8aa5da37'>tests: Ensure test OpenPGP keys never expire.</a><span class='msg-avail'>...<span class='msg-tooltip'>All these keys had expiration dates.  'tests/keys/ed25519.pub' expired
on 2022-04-24.

Fixes &lt;https://issues.guix.gnu.org/55506&gt;.

* tests/keys/ed25519.pub, tests/keys/ed25519-2.pub,
tests/keys/ed25519-3.pub: Remove expiration date.
</span></span></td><td>Ludovic Courtès</td></tr>
<tr><td><span title='2021-12-22 10:53:19 +0100'>2021-12-22</span></td><td><a href='/guix/commit/tests/keys?id=9ebc9ca070a21ba0aeb3f61202a8071bb250ccc8'>tests: Move keys into ./tests/keys/ and add a third ed25519 key.</a><span class='msg-avail'>...<span class='msg-tooltip'>The third key will be used in an upcoming commit.

Rename public keys to .pub.

* guix/tests/gnupg.scm (%ed25519-3-public-key-file): New variable.
(%ed25519-3-secret-key-file): New variable.
(%ed25519-2-public-key-file): Renamed from %ed25519bis-public-key-file.
(%ed25519-2-secret-key-file): Renamed from %ed25519bis-secret-key-file.
* tests/keys/ed25519-3.key: New file.
* tests/keys/ed25519-3.sec: New file.

Signed-off-by: Mathieu Othacehe &lt;othacehe@gnu.org&gt;
</span></span></td><td>Attila Lendvai</td></tr>