From 152af3a29793681feb3170f9b3801eb52992bab5 Mon Sep 17 00:00:00 2001 From: Luke Date: Sun, 28 Feb 2021 22:46:19 +0000 Subject: [PATCH] LibAudio: Move format and BPS checks before VERIFYs in WAV loader It was accidentally checking the format/bits per sample too late, which would crash with the assertion. --- Userland/Libraries/LibAudio/WavLoader.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Userland/Libraries/LibAudio/WavLoader.cpp b/Userland/Libraries/LibAudio/WavLoader.cpp index 28c2fb8277..3250e0e2b4 100644 --- a/Userland/Libraries/LibAudio/WavLoader.cpp +++ b/Userland/Libraries/LibAudio/WavLoader.cpp @@ -192,8 +192,8 @@ bool WavLoaderPlugin::parse_header() u16 audio_format = read_u16(); CHECK_OK("Audio format"); // incomplete read check ok = ok && audio_format == 1; // WAVE_FORMAT_PCM + CHECK_OK("Audio format"); // value check VERIFY(audio_format == 1); - CHECK_OK("Audio format"); // value check m_num_channels = read_u16(); ok = ok && (m_num_channels == 1 || m_num_channels == 2); @@ -211,8 +211,8 @@ bool WavLoaderPlugin::parse_header() m_bits_per_sample = read_u16(); CHECK_OK("Bits per sample"); // incomplete read check ok = ok && (m_bits_per_sample == 8 || m_bits_per_sample == 16 || m_bits_per_sample == 24); - VERIFY(m_bits_per_sample == 8 || m_bits_per_sample == 16 || m_bits_per_sample == 24); CHECK_OK("Bits per sample"); // value check + VERIFY(m_bits_per_sample == 8 || m_bits_per_sample == 16 || m_bits_per_sample == 24); // Read chunks until we find DATA bool found_data = false;