input/wav: Correctly check supported sample size.

[libsigrok.git] / src / input / wav.c

diff --git a/src/input/wav.c b/src/input/wav.c
index c5abf235d70864ff67da82990ef9af3a855f28e6..fe12a2f812647d4bc0d4b173ee65232d7d266983 100644 (file)
--- a/src/input/wav.c
+++ b/src/input/wav.c
@@ -63,7 +63,7 @@ static int parse_wav_header(GString *buf, struct context *inc)
        samplerate = RL32(buf->str + 24);
 
        samplesize = RL16(buf->str + 32);
-       if (samplesize != 1 && samplesize != 2 && samplesize != 4) {
+       if (samplesize != 8 && samplesize != 16 && samplesize != 32) {
                sr_err("Only 8, 16 or 32 bits per sample supported.");
                return SR_ERR_DATA;
        }
@@ -102,6 +102,10 @@ static int parse_wav_header(GString *buf, struct context *inc)
                        sr_err("Only PCM and floating point samples are supported.");
                        return SR_ERR_DATA;
                }
+               if (fmt_code == WAVE_FORMAT_IEEE_FLOAT && unitsize != 4) {
+                       sr_err("only 32-bit floats supported.");
+                       return SR_ERR_DATA;
+               }
        } else {
                sr_err("Only PCM and floating point samples are supported.");
                return SR_ERR_DATA;
@@ -244,8 +248,9 @@ static void send_chunk(const struct sr_input *in, int offset, int num_samples)
                } else {
                        /* BINARY32 float */
 #ifdef WORDS_BIGENDIAN
+                       int i;
                        for (i = 0; i < inc->unitsize; i++)
-                               d[i] = s[inc->unitsize - i];
+                               d[i] = s[inc->unitsize - 1 - i];
 #else
                        memcpy(d, s, inc->unitsize);
 #endif