Skip to content

[Non-BE][Renderer Non-BE][Split Non-BE] Resolve potential out-of-bounds access in TD renderer ITD synthesis.

norvellrequested to merge
1296-decoder-memorysanitizer-use-of-uninitialized-value-in-limiter_process-scheduled-pipeline-c into main

Reason why this change is needed

  • TD renderer ITD resampling may read out-of-bounds. Observed for JBM+headtracking cases, but could happen in other cases too.

Description of the change

  • One line is removed, which assumes there is space to include one more filter multiplication:
        tmp += ( *p_forward ) * ( *p_sinc_forward ); /* Integer index always rounded down --> 4 steps backward, 5 steps forward */

Affected operating points

  • TD renderer operation when the ITD is changing (relative object position changes). Impact is expected to be small, since this is the tail of the filter. Resolves potential out-of-bounds access.

Related to #1296 (closed)

Merge request reports