<div style="line-height:1.7;color:#000000;font-size:14px;font-family:Arial"><div>Hi, </div><div><br></div><div>I downloaded the source code of opus-1.1.3.tar.gz, and statically analyzed the code by a static analysis tool. Many potential false alarms are reported. I checked manually and selected a potential error which is described as follow. Could you please check it?</div><div><br></div><div>In file ¡®src/opus_encoder.c¡¯, function ¡®opus_encode_native¡¯, the piece of code is:</div><div><br></div><div>¡­</div><div>Line 1802: if (frame_size == 2*st->Fs/25)  /* 80 ms -> 2x 40 ms */</div><div>Line 1803:   enc_frame_size = st->Fs/25;</div><div>Line 1804: if (frame_size == 3*st->Fs/25)  /* 120 ms -> 2x 60 ms */</div><div>Line 1805:   enc_frame_size = 3*st->Fs/50;</div><div>Line 1806: else                            /* 100 ms -> 5x 20 ms */</div><div>Line 1807:   enc_frame_size = st->Fs/50; </div><div>¡­</div><div><br></div><div>In the above code, Line 1802 and 1803 may be not useful. Because the result of variable ¡®enc_frame_size¡¯ is not read and used. After executing Line 1805 or Line 1807, the value of ¡®enc_frame_size¡¯ is overwritten. </div><div><br></div><div>Looking forward to your reply.</div><div><br></div><div>Best,</div><div>Zhendong</div></div>