[OpenOCD-commit] Main OpenOCD repository branch, master, updated. v0.12.0-rc1-5-g8683526af

[OpenOCD-Gerrit <ope...@us...>]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Main OpenOCD repository".

The branch, master has been updated
       via  8683526af7a9ca23115d26deb5b4835cd6be05d8 (commit)
      from  cde944fd8b9cd5eb67617aa3b68dc13eb49b1d10 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 8683526af7a9ca23115d26deb5b4835cd6be05d8
Author: Antonio Borneo <bor...@gm...>
Date:   Mon Sep 19 11:36:06 2022 +0200

    target/dsp563xx: fix scan-build warning
    
    Scan-build triggers a warning:
            Unix API: Allocator sizeof operand mismatch
            dsp563xx.c:2143 Result of 'calloc' is converted to a pointer
                    of type 'uint8_t', which is incompatible with sizeof
                     operand type 'uint32_t'
    
    It's a false positive because calloc() is properly used in this
    case, as the uint8_t array is used in blocks of 4 elements to read
    or write uint32_t values.
    
    Either
            calloc(sizeof(uint32_t), count);
    and
            malloc(count * sizeof(uint32_t));
    keep triggering the same warning.
    
    Drop the warning by using the constant '4' as size of uint32_t, as
    already used few lines below.
    
    Change-Id: I5bb1ece177774eefdc5d9cd049338f8f2be87cd7
    Signed-off-by: Antonio Borneo <bor...@gm...>
    Reviewed-on: https://review.openocd.org/c/openocd/+/7203
    Tested-by: jenkins
    Reviewed-by: Andreas Fritiofson <and...@gm...>

diff --git a/src/target/dsp563xx.c b/src/target/dsp563xx.c
index 3702cfb4c..36ee85371 100644
--- a/src/target/dsp563xx.c
+++ b/src/target/dsp563xx.c
@@ -2140,7 +2140,7 @@ COMMAND_HANDLER(dsp563xx_mem_command)
 			COMMAND_PARSE_NUMBER(u32, CMD_ARGV[1], count);
 	}
 
-	buffer = calloc(count, sizeof(uint32_t));
+	buffer = calloc(count, 4);
 
 	if (read_mem == 1) {
 		err = dsp563xx_read_memory(target, mem_type, address, sizeof(uint32_t),

-----------------------------------------------------------------------

Summary of changes:
 src/target/dsp563xx.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


hooks/post-receive
-- 
Main OpenOCD repository