ref: b1d51b2d3f0118a598d8813c59e02fd596e12646
parent: 3105fa273c125faeda6b7168c74854a7f71a90c4
author: Andrew Larkin <[email protected]>
date: Thu Jul 6 07:00:36 EDT 2017
Fix uninitialized variables in decide_dtx_mode() Signed-off-by: Felicia Lim <[email protected]>
--- a/src/opus_encoder.c
+++ b/src/opus_encoder.c
@@ -909,21 +909,20 @@
int arch
)
{
- int is_noise;
opus_val32 noise_energy;
- int is_sufficiently_quiet;
if (!is_silence)
{
- is_noise = activity_probability < DTX_ACTIVITY_THRESHOLD;
- if (is_noise)
+ if (activity_probability < DTX_ACTIVITY_THRESHOLD) /* is noise */
{
noise_energy = compute_frame_energy(pcm, frame_size, channels, arch);
- is_sufficiently_quiet = peak_signal_energy >= (PSEUDO_SNR_THRESHOLD * noise_energy);
+
+ /* but is sufficiently quiet */
+ is_silence = peak_signal_energy >= (PSEUDO_SNR_THRESHOLD * noise_energy);
}
}
- if (is_silence || (is_noise && is_sufficiently_quiet))
+ if (is_silence)
{
/* The number of consecutive DTX frames should be within the allowed bounds */
(*nb_no_activity_frames)++;