This is an automated email from Gerrit.

Cody Schafer ([email protected]) just uploaded a new patch set to Gerrit, 
which you can find at http://openocd.zylin.com/4584

-- gerrit

commit 3ec0b3d4030b1b00771160b20a2bb42faf05024d
Author: Cody Schafer <[email protected]>
Date:   Mon Jul 2 15:56:53 2018 -0400

    target/image: make i/j unsigned to avoid runtime error
    
        src/target/image.c:1055:15: runtime error: left shift of 128 by 24 
places cannot be represented in type 'int'
    
    Change-Id: I322fd391cf3f242beffc8a274824763c8c5e69a4
    Signed-off-by: Cody Schafer <[email protected]>

diff --git a/src/target/image.c b/src/target/image.c
index 9f56bea..0d98c57 100644
--- a/src/target/image.c
+++ b/src/target/image.c
@@ -1048,8 +1048,7 @@ int image_calculate_checksum(uint8_t *buffer, uint32_t 
nbytes, uint32_t *checksu
        static bool first_init;
        if (!first_init) {
                /* Initialize the CRC table and the decoding table.  */
-               int i, j;
-               unsigned int c;
+               unsigned int i, j, c;
                for (i = 0; i < 256; i++) {
                        /* as per gdb */
                        for (c = i << 24, j = 8; j > 0; --j)

-- 

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
OpenOCD-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to