diff options
author | Jean-Marc Valin <jmvalin@jmvalin.ca> | 2018-05-15 22:36:33 +0300 |
---|---|---|
committer | Jean-Marc Valin <jmvalin@jmvalin.ca> | 2018-05-15 22:36:33 +0300 |
commit | 652c4559f593d3aad78bd5c85a216eeae7859429 (patch) | |
tree | c8d4b6b4780badfddf343cea7305d270213c8e3d /celt/celt_encoder.c | |
parent | 42f43db7e470dec8c7a40c86180d6a07241c3577 (diff) |
Aborting on NaN in CELT
NaNs should be filtered at the Opus layer, so if there are any in the CELT
encoder, then it's likely something went horribly wrong (e.g. corrupted state).
In that case, better abort than have something bad happen.
Diffstat (limited to 'celt/celt_encoder.c')
-rw-r--r-- | celt/celt_encoder.c | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/celt/celt_encoder.c b/celt/celt_encoder.c index 2cc0ae0c..ad0ebf0a 100644 --- a/celt/celt_encoder.c +++ b/celt/celt_encoder.c @@ -362,6 +362,12 @@ static int transient_analysis(const opus_val32 * OPUS_RESTRICT in, int len, int /* Compute harmonic mean discarding the unreliable boundaries The data is smooth, so we only take 1/4th of the samples */ unmask=0; + /* We should never see NaNs here. If we find any, then something really bad happened and we better abort + before it does any damage later on. If these asserts are disabled (no hardening), then the table + lookup a few lines below (id = ...) is likely to crash dur to an out-of-bounds read. DO NOT FIX + that crash on NaN since it could result in a worse issue later on. */ + celt_assert(!celt_isnan(tmp[0])); + celt_assert(!celt_isnan(norm)); for (i=12;i<len2-5;i+=4) { int id; @@ -1716,6 +1722,9 @@ int celt_encode_with_ec(CELTEncoder * OPUS_RESTRICT st, const opus_val16 * pcm, } compute_mdcts(mode, shortBlocks, in, freq, C, CC, LM, st->upsample, st->arch); + /* This should catch any NaN in the CELT input. Since we're not supposed to see any (they're filtered + at the Opus layer), just abort. */ + celt_assert(!celt_isnan(freq[0]) && (C==1 || !celt_isnan(freq[C*N]))); if (CC==2&&C==1) tf_chan = 0; compute_band_energies(mode, freq, bandE, effEnd, C, LM, st->arch); |