<div dir="ltr"><div>Hello,</div><div>in this commit in celt_encoder.c</div><a href="https://git.xiph.org/?p=opus.git;a=commitdiff;h=652c4559f593d3aad78bd5c85a216eeae7859429">https://git.xiph.org/?p=opus.git;a=commitdiff;h=652c4559f593d3aad78bd5c85a216eeae7859429</a><br><div><br></div><div>I see the note:</div><div>

<div class="gmail-diff gmail-add" style="font-family:monospace;font-size:small;white-space:pre;color:rgb(0,136,0);background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial">+      /* We should never see NaNs here. If we find any, then something really bad happened and we better abort</div><div class="gmail-diff gmail-add" style="font-family:monospace;font-size:small;white-space:pre;color:rgb(0,136,0);background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial">+         before it does any damage later on. If these asserts are disabled (no hardening), then the table</div><div class="gmail-diff gmail-add" style="font-family:monospace;font-size:small;white-space:pre;color:rgb(0,136,0);background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial">+         lookup a few lines below (id = ...) is likely to crash dur to an out-of-bounds read. DO NOT FIX</div><div class="gmail-diff gmail-add" style="font-family:monospace;font-size:small;white-space:pre;color:rgb(0,136,0);background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial">+         that crash on NaN since it could result in a worse issue later on. */</div>

<br></div><div>I think I'm exactly in that situation. Opus codec crashes occasionally for me with Access violation in the transient_analysis function on the line where it's computing the id from the floating point.</div><div><br></div><div> id = (int)MAX32(0,MIN32(127,floor(64*norm*(tmp[i]+EPSILON))));<br></div><div><br></div><div>Could you please provide more info about what could be the crash reason there?</div><div><br></div><div>Regards,</div><div>Alex</div></div>