command/fingerprint: fix mutex locking bug

GetChromaprintCommand::DecodeStream() is called without holding the mutex.

command/fingerprint: fix mutex locking bug
040573c6 · Max Kellermann · bc5d4f94 · 040573c6
Commit 040573c6 authored May 07, 2019 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 2 deletions

FingerprintCommands.cxx src/command/FingerprintCommands.cxx +1 -2

No files found.
--- a/src/command/FingerprintCommands.cxx
+++ b/src/command/FingerprintCommands.cxx
@@ -107,11 +107,10 @@ GetChromaprintCommand::DecodeStream(InputStream &input_stream,

 	/* rewind the stream, so each plugin gets a fresh start */
 	try {
-		input_stream.Rewind();
+		input_stream.LockRewind();
 	} catch (...) {
 	}

-	const ScopeUnlock unlock(mutex);
 	plugin.StreamDecode(*this, input_stream);
 }