diff options
| author | Joshua Colp <jcolp@digium.com> | 2017-04-19 08:38:59 -0500 |
|---|---|---|
| committer | Gerrit Code Review <gerrit2@gerrit.digium.api> | 2017-04-19 08:38:59 -0500 |
| commit | b84abac1449c56d6c9f781525a533691b326ad8c (patch) | |
| tree | dc94a05551bf83f987b108f13ced42d05ca3aea6 /formats | |
| parent | e756a9e063dd71cf49abec6b03c305b54e837f39 (diff) | |
| parent | 6c0ab9afa775d30fcc498072eee7f660202fa51e (diff) | |
Merge "format_wav: Read 16khz wav samples properly"
Diffstat (limited to 'formats')
| -rw-r--r-- | formats/format_wav.c | 24 |
1 files changed, 14 insertions, 10 deletions
diff --git a/formats/format_wav.c b/formats/format_wav.c index 049ead40e..8316c3530 100644 --- a/formats/format_wav.c +++ b/formats/format_wav.c @@ -80,9 +80,8 @@ struct wav_desc { /* format-specific parameters */ static int check_header_fmt(FILE *f, int hsize, int hz) { - short format, chans, bysam, bisam; - int bysec; - int freq; + unsigned short format, chans, bysam, bisam; + unsigned int freq, bysec; if (hsize < 16) { ast_log(LOG_WARNING, "Unexpected header size %d\n", hsize); return -1; @@ -92,7 +91,7 @@ static int check_header_fmt(FILE *f, int hsize, int hz) return -1; } if (ltohs(format) != 1) { - ast_log(LOG_WARNING, "Not a supported wav file format (%d). Only PCM encoded, 16 bit, mono, 8kHz files are supported with a lowercase '.wav' extension.\n", ltohs(format)); + ast_log(LOG_WARNING, "Not a supported wav file format (%d). Only PCM encoded, 16 bit, mono, 8kHz/16kHz files are supported with a lowercase '.wav' extension.\n", ltohs(format)); return -1; } if (fread(&chans, 1, 2, f) != 2) { @@ -107,10 +106,9 @@ static int check_header_fmt(FILE *f, int hsize, int hz) ast_log(LOG_WARNING, "Read failed (freq)\n"); return -1; } - if (((ltohl(freq) != 8000) && (ltohl(freq) != 16000)) || - ((ltohl(freq) == 8000) && (hz != 8000)) || - ((ltohl(freq) == 16000) && (hz != 16000))) { - ast_log(LOG_WARNING, "Unexpected frequency mismatch %d (expecting %d)\n", ltohl(freq),hz); + freq = ltohl(freq); + if ((freq != 8000 && freq != 16000) || freq != hz) { + ast_log(LOG_WARNING, "Unexpected frequency mismatch %d (expecting %d)\n", freq, hz); return -1; } /* Ignore the byte frequency */ @@ -323,9 +321,15 @@ static int wav_open(struct ast_filestream *s) /* We don't have any header to read or anything really, but if we did, it would go here. We also might want to check and be sure it's a valid file. */ - struct wav_desc *tmp = (struct wav_desc *)s->_private; - if ((tmp->maxlen = check_header(s->f, ast_format_get_sample_rate(s->fmt->format))) < 0) + struct wav_desc *tmp = s->_private; + unsigned int sample_rate = ast_format_get_sample_rate(s->fmt->format); + + tmp->maxlen = check_header(s->f, sample_rate); + if (tmp->maxlen < 0) { return -1; + } + + tmp->hz = sample_rate; return 0; } |