[OpenOCD-commit] Main OpenOCD repository branch, master, updated. v0.12.0-902-g16c6497a8

[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  16c6497a89600ab8e8b354e2fc2c0ceb9ae74330 (commit)
      from  d892a4d763c283ca775213d4148b76d5b0fde520 (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 16c6497a89600ab8e8b354e2fc2c0ceb9ae74330
Author: Evgeniy Naydanov <evg...@sy...>
Date:   Fri Feb 14 19:31:38 2025 +0300

    rtos/linux: fix name overwrite in `linux_thread_extra_info()`
    
    commit 908ee4dc9641bd3df2eb00264575501867da539d ("build: remove clang
    unused variable assignment warnings") introduced an error:
    ```
    -            tmp_str_ptr += sprintf(tmp_str_ptr, "%s", name);
    +            sprintf(tmp_str_ptr, "%s", name);
                 sprintf(tmp_str_ptr, "%s", temp->name);
    ```
    This results in `name` being overwritten by `temp->name`.
    Fix this, adding OOM handling along the way.
    
    Change-Id: Id41f73247c3f7e6194d7c92187ad3163a9ea6c89
    Signed-off-by: Evgeniy Naydanov <evg...@sy...>
    Reviewed-on: https://review.openocd.org/c/openocd/+/8761
    Tested-by: jenkins
    Reviewed-by: Antonio Borneo <bor...@gm...>

diff --git a/src/rtos/linux.c b/src/rtos/linux.c
index 91d9a39f2..5efdc9f60 100644
--- a/src/rtos/linux.c
+++ b/src/rtos/linux.c
@@ -1120,23 +1120,13 @@ static int linux_thread_extra_info(struct target *target,
 
 	while (temp) {
 		if (temp->threadid == threadid) {
-			char *pid = " PID: ";
-			char *pid_current = "*PID: ";
-			char *name = "Name: ";
-			int str_size = strlen(pid) + strlen(name);
-			char *tmp_str = calloc(1, str_size + 50);
-			char *tmp_str_ptr = tmp_str;
-
-			/*  discriminate current task */
-			if (temp->status == 3)
-				tmp_str_ptr += sprintf(tmp_str_ptr, "%s",
-						pid_current);
-			else
-				tmp_str_ptr += sprintf(tmp_str_ptr, "%s", pid);
-
-			tmp_str_ptr += sprintf(tmp_str_ptr, "%d, ", (int)temp->pid);
-			sprintf(tmp_str_ptr, "%s", name);
-			sprintf(tmp_str_ptr, "%s", temp->name);
+			char *tmp_str = alloc_printf("%cPID: %" PRIu32 ", Name: %s",
+					temp->status == 3 ? '*' : ' ',
+					temp->pid, temp->name);
+			if (!tmp_str) {
+				LOG_ERROR("Out of memory");
+				return ERROR_FAIL;
+			}
 			char *hex_str = calloc(1, strlen(tmp_str) * 2 + 1);
 			size_t pkt_len = hexify(hex_str, (const uint8_t *)tmp_str,
 				strlen(tmp_str), strlen(tmp_str) * 2 + 1);

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

Summary of changes:
 src/rtos/linux.c | 24 +++++++-----------------
 1 file changed, 7 insertions(+), 17 deletions(-)


hooks/post-receive
-- 
Main OpenOCD repository