output/httpd: flush encoder after tag

Without the flush, ReadPage() may not return any data, or not all data. This may result in incomplete ddata the new "header" page, corrupting streams with some encoders such as Vorbis. Fixes #145

output/httpd: flush encoder after tag
014f8cd6 · Max Kellermann · aea37e46 · 014f8cd6 · 014f8cd6
Commit 014f8cd6 authored Nov 14, 2017 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 0 deletions

NEWS NEWS +1 -0

HttpdOutputPlugin.cxx src/output/plugins/httpd/HttpdOutputPlugin.cxx +1 -0

No files found.
--- a/NEWS
+++ b/NEWS
@@ -12,6 +12,7 @@ ver 0.20.12 (not yet released)
  - vorbis: default to quality 3
 * output
  - fix hanging playback with soxr resampler
+  - httpd: flush encoder after tag; fixes corrupt Vorbis stream

 ver 0.20.11 (2017/10/18)
 * storage

--- a/src/output/plugins/httpd/HttpdOutputPlugin.cxx
+++ b/src/output/plugins/httpd/HttpdOutputPlugin.cxx
@@ -468,6 +468,7 @@ HttpdOutput::SendTag(const Tag &tag)

 		try {
 			encoder->SendTag(tag);
+			encoder->Flush();
 		} catch (const std::runtime_error &) {
 			/* ignore */
 		}