lib/chromaprint/DecoderClient: check `ready` in GetCommand()

Fixes access to uninitialized memory (`remaining_bytes` isn't initialized until Ready() is called) and fixes spurious `getfingerprint` failures.

lib/chromaprint/DecoderClient: check `ready` in GetCommand()
d942f874 · Max Kellermann · 01632d37 · d942f874
Commit d942f874 authored Mar 12, 2020 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

DecoderClient.hxx src/lib/chromaprint/DecoderClient.hxx +1 -1

No files found.
--- a/src/lib/chromaprint/DecoderClient.hxx
+++ b/src/lib/chromaprint/DecoderClient.hxx
@@ -71,7 +71,7 @@ public:
 		   bool seekable, SignedSongTime duration) noexcept override;

 	DecoderCommand GetCommand() noexcept override {
-		return !error && remaining_bytes > 0
+		return !error && (!ready || remaining_bytes > 0)
 			? DecoderCommand::NONE
 			: DecoderCommand::STOP;
 	}