Commit 0bc8e39c authored by multrus's avatar multrus
Browse files

[cleanup] accept FIX_BASOP_2626_RC_UNI_DEC_READ_BITS_HANG

parent aa60804e

lib_com/options.h

+0 −1
Original line number Diff line number Diff line
@@ -86,7 +86,6 @@ 
#define FIX_1990_SANITIZER_IN_REVERB_LOAD               /* Nokia: Fix issue part of issue 1990 by introducing missing free of structure - keep until #2059 is addressed */

#define TMP_1342_WORKAROUND_DEC_FLUSH_BROKEN_IN_SR      /* FhG: Temporary workaround for incorrect implementation of decoder flush with split rendering */

#define NONBE_1122_KEEP_EVS_MODE_UNCHANGED              /* FhG: Disables fix for issue 1122 in EVS mode to keep BE tests green. This switch should be removed once the 1122 fix is added to EVS via a CR.  */

#define FIX_BASOP_2626_RC_UNI_DEC_READ_BITS_HANG        /* FhG: BASOP #2626: rc_uni_dec_read_bits: replace loop counter tmp with UWord64; with UWord32 the division-substitute loop (tmp <= low) never terminates after the bit-error sentinel sets rc_low=0xFFFFFFFF -> decoder hang on corrupted bitstreams */



/* #################### End BE switches ################################## */

lib_dec/ivas_range_uni_dec_fx.c

+0 −4
Original line number Diff line number Diff line
@@ -380,11 +380,7 @@ UWord16 rc_uni_dec_read_bits( 
    UWord32 val;

    UWord32 low;   /* local copy (2 to 6 uses) */

    UWord32 range; /* local copy (4 to 7 uses) */

#ifdef FIX_BASOP_2626_RC_UNI_DEC_READ_BITS_HANG

    UWord64 tmp;

#else

    UWord32 tmp;

#endif

    low = rc_st_dec->rc_low;     /*Q0*/

    range = rc_st_dec->rc_range; /*Q0*/

    move32();