]> git.alsa-project.org Git - alsa-utils.git/commitdiff
aplay: fix S24_LE wav header
authorPeter Suti <peter.suti@streamunlimited.com>
Wed, 20 Mar 2024 08:40:21 +0000 (09:40 +0100)
committerJaroslav Kysela <perex@perex.cz>
Wed, 17 Apr 2024 10:05:59 +0000 (12:05 +0200)
S24_LE is 32 bits in width storing 24 bits of data and 8 bits of padding
So wav header needs to be 32 bits not 24

Closes: https://github.com/alsa-project/alsa-utils/pull/260
Signed-off-by: Peter Suti <peter.suti@streamunlimited.com>
Signed-off-by: Jaroslav Kysela <perex@perex.cz>
aplay/aplay.c

index 6c5a07eff7d437ac51685205fa1c05ad1949da26..5bead5bb28ca315907f3757f89892c4832005fb7 100644 (file)
@@ -2719,11 +2719,11 @@ static void begin_wave(int fd, size_t cnt)
        case SND_PCM_FORMAT_S16_LE:
                bits = 16;
                break;
+       case SND_PCM_FORMAT_S24_LE: /* S24_LE is 24 bits stored in 32 bit width with 8 bit padding */
        case SND_PCM_FORMAT_S32_LE:
-        case SND_PCM_FORMAT_FLOAT_LE:
+       case SND_PCM_FORMAT_FLOAT_LE:
                bits = 32;
                break;
-       case SND_PCM_FORMAT_S24_LE:
        case SND_PCM_FORMAT_S24_3LE:
                bits = 24;
                break;