output/alsa: fix out_frame_size formula, multiply with channels

The hard-coded "3 bytes" was wrong because it ignored the number of channels.

output/alsa: fix out_frame_size formula, multiply with channels
a0e4b6e2 · Max Kellermann · 0de39b64 · a0e4b6e2
Commit a0e4b6e2 authored Apr 23, 2012 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 1 deletion

alsa_output_plugin.c src/output/alsa_output_plugin.c +3 -1

No files found.
--- a/src/output/alsa_output_plugin.c
+++ b/src/output/alsa_output_plugin.c
@@ -674,7 +674,9 @@ alsa_open(struct audio_output *ao, struct audio_format *audio_format, GError **e
 	}

 	ad->in_frame_size = audio_format_frame_size(audio_format);
-	ad->out_frame_size = ad->export.pack24 ? 3 : ad->in_frame_size;
+	ad->out_frame_size = ad->export.pack24
+		? (size_t)(audio_format->channels * 3)
+		: ad->in_frame_size;

 	return true;
 }