]> granicus.if.org Git - openjpeg/commitdiff
suppress valgrind error - avoid accessing uninitialized memory in mq encoder
authorAaron Boxer <boxerab@gmail.com>
Sat, 30 Jan 2016 15:05:46 +0000 (10:05 -0500)
committerAaron Boxer <boxerab@gmail.com>
Sat, 30 Jan 2016 15:05:46 +0000 (10:05 -0500)
src/lib/openjp2/mqc.c

index e6e4cc8795c0332545dd2e0c65beeb0981b8af98..4e409a7ce23e4427eb1da92cab29ff2d4aed312f 100644 (file)
@@ -203,14 +203,20 @@ static opj_mqc_state_t mqc_states[47 * 2] = {
 */
 
 static void opj_mqc_byteout(opj_mqc_t *mqc) {
-       OPJ_BYTE bp_in_bounds = (mqc->bp >= mqc->start);
-       if (bp_in_bounds & (*mqc->bp == 0xff)) {
+       /* avoid accessing uninitialized memory*/
+       if (mqc->bp == mqc->start-1) {
+               mqc->bp++;
+               *mqc->bp = (OPJ_BYTE)(mqc->c >> 19);
+               mqc->c &= 0x7ffff;
+               mqc->ct = 8;
+       }
+       else if (*mqc->bp == 0xff) {
                mqc->bp++;
                *mqc->bp = (OPJ_BYTE)(mqc->c >> 20);
                mqc->c &= 0xfffff;
                mqc->ct = 7;
        } else {
-               if ((bp_in_bounds ^ 1) | ((mqc->c & 0x8000000) == 0)) { 
+               if ((mqc->c & 0x8000000) == 0) {        
                        mqc->bp++;
                        *mqc->bp = (OPJ_BYTE)(mqc->c >> 19);
                        mqc->c &= 0x7ffff;