From fde6d79fb8363c4a329a184ef0b107156602b225 Mon Sep 17 00:00:00 2001
From: Wim Taymans <wtaymans@redhat.com>
Date: Thu, 27 Sep 2018 10:48:45 +0200
Subject: [PATCH] ModuleState: handle compress/decompress init failure

When the unit initcompress or initdecompress function fails,
m_fileModule is NULL. Return AF_FAIL in that case instead of
causing NULL pointer dereferences later.

Fixes #49
---
 libaudiofile/modules/ModuleState.cpp | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/libaudiofile/modules/ModuleState.cpp b/libaudiofile/modules/ModuleState.cpp
index 0c29d7a..070fd9b 100644
--- a/libaudiofile/modules/ModuleState.cpp
+++ b/libaudiofile/modules/ModuleState.cpp
@@ -75,6 +75,9 @@ status ModuleState::initFileModule(AFfilehandle file, Track *track)
 		m_fileModule = unit->initcompress(track, file->m_fh, file->m_seekok,
 			file->m_fileFormat == AF_FILE_RAWDATA, &chunkFrames);
 
+	if (!m_fileModule)
+		return AF_FAIL;
+
 	if (unit->needsRebuffer)
 	{
 		assert(unit->nativeSampleFormat == AF_SAMPFMT_TWOSCOMP);
a href='/guix/'>summary</a><a href='/guix/refs/?id=7432b922efa0e9b633a05bd52644ee00ebd58d9f'>refs</a><a class='active' href='/guix/log/tests/ui.scm'>log</a><a href='/guix/tree/tests/ui.scm?id=7432b922efa0e9b633a05bd52644ee00ebd58d9f'>tree</a><a href='/guix/commit/tests/ui.scm?id=7432b922efa0e9b633a05bd52644ee00ebd58d9f'>commit</a><a href='/guix/diff/tests/ui.scm?id=7432b922efa0e9b633a05bd52644ee00ebd58d9f'>diff</a></td><td class='form'><form class='right' method='get' action='/guix/log/tests/ui.scm'>
<input type='hidden' name='id' value='7432b922efa0e9b633a05bd52644ee00ebd58d9f'/><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=7432b922efa0e9b633a05bd52644ee00ebd58d9f'>root</a>/<a href='/guix/log/tests?id=7432b922efa0e9b633a05bd52644ee00ebd58d9f'>tests</a>/<a href='/guix/log/tests/ui.scm?id=7432b922efa0e9b633a05bd52644ee00ebd58d9f'>ui.scm</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/ui.scm?id=7432b922efa0e9b633a05bd52644ee00ebd58d9f&amp;showmsg=1'>Expand</a>)</th><th class='left'>Author</th></tr>
<tr><td><span title='2022-12-13 17:46:39 +0100'>2022-12-13</span></td><td><a href='/guix/commit/tests/ui.scm?id=bbcd06e56c06376e640a7ac81a7109e7135a20f2'>ui: Take package upstream name into account when searching.</a><span class='msg-avail'>...<span class='msg-tooltip'>* guix/ui.scm (%package-metrics): Add PACKAGE-UPSTREAM-NAME*.
* tests/ui.scm ("package-relevance and upstream name"): New test.

Co-authored-by: Ludovic Courtès &lt;ludo@gnu.org&gt;
</span></span></td><td>Lars-Dominik Braun</td></tr>
<tr><td><span title='2022-06-16 23:57:41 +0200'>2022-06-16</span></td><td><a href='/guix/commit/tests/ui.scm?id=c8803d89fe531a1ff0de8ccdbc77c86249d5e778'>ui: Improve pager selection logic when less is not installed.</a><span class='msg-avail'>...<span class='msg-tooltip'>* guix/ui.scm (find-available-pager): New procedure.
(call-with-paginated-output-port): Use it.
* guix/utils.scm (call-with-environment-variables): Allow clearing of
specified environment variables.
* tests/ui.scm (make-empty-file, assert-equals-find-available-pager):
New procedures.
("find-available-pager, GUIX_PAGER takes precedence")
("find-available-pager, PAGER takes precedence")
("find-available-pager, 'less' takes precedence")
("find-available-pager, 'more' takes precedence")
("find-available-pager, no pager"): New tests.

Co-authored-by: Ludovic Courtès &lt;ludo@gnu.org&gt;
</span></span></td><td>Taiju HIGASHI</td></tr>