This is an automated email from Gerrit.

"Antonio Borneo <borneo.anto...@gmail.com>" just uploaded a new patch set to 
Gerrit, which you can find at https://review.openocd.org/c/openocd/+/7598

-- gerrit

commit 00a8fea31fc527021da9613eb5b0b97bd67925ae
Author: Antonio Borneo <borneo.anto...@gmail.com>
Date:   Sun Apr 9 00:33:38 2023 +0200

    rtos: with pointers, use NULL instead of 0
    
    Don't compare pointers with 0, use NULL when needed.
    Don't assign pointer to 0, use NULL.
    
    Detected through 'sparse' tool.
    
    Change-Id: Ifa81ba961c0d490cc74880b4a46b620e6358f779
    Signed-off-by: Antonio Borneo <borneo.anto...@gmail.com>

diff --git a/src/rtos/chibios.c b/src/rtos/chibios.c
index 68fe8a14c9..20378274ec 100644
--- a/src/rtos/chibios.c
+++ b/src/rtos/chibios.c
@@ -80,12 +80,12 @@ struct chibios_params {
 static struct chibios_params chibios_params_list[] = {
        {
        "cortex_m",                                                     /* 
target_name */
-       0,
+       NULL,
        NULL,                                                                   
/* stacking_info */
        },
        {
        "hla_target",                                                   /* 
target_name */
-       0,
+       NULL,
        NULL,                                                                   
/* stacking_info */
        }
 };
@@ -198,7 +198,7 @@ static int chibios_update_memory_signature(struct rtos 
*rtos)
 errfree:
        /* Error reading the ChibiOS memory structure */
        free(signature);
-       param->signature = 0;
+       param->signature = NULL;
        return -1;
 }
 
@@ -468,7 +468,7 @@ static int chibios_get_thread_reg_list(struct rtos *rtos, 
int64_t thread_id,
                return -1;
 
        /* Update stacking if it can only be determined from runtime 
information */
-       if ((param->stacking_info == 0) &&
+       if (!param->stacking_info &&
                (chibios_update_stacking(rtos) != ERROR_OK)) {
                LOG_ERROR("Failed to determine exact stacking for the target 
type %s", rtos->target->type->name);
                return -1;

-- 

Reply via email to