epics/epics-base
0
0
Fork 0
Code Issues Pull Requests Projects Wiki Activity

fix: CodeQL check failure.

Browse Source 
Possibly wrong buffer size in string copy (Critical)
This commit is contained in:
Jerzy Jamroz
2025-02-07 09:41:25 +01:00
committed by Andrew Johnson
parent ecd76e5b5b
commit d7a02342e5
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
modules/libcom/src/iocsh/atInit.c
View File

@@ -52,11 +52,13 @@ static void atInitHook(initHookState state)
static struct cmditem* newItem(char* cmd)
{
struct cmditem* item = mallocMustSucceed(sizeof(struct cmditem) + strlen(cmd) + 1,
size_t cmd_len = strlen(cmd) + 1;
struct cmditem* item = mallocMustSucceed(sizeof(struct cmditem) + cmd_len,
ERL_ERROR " atInit: "
"failed to allocate memory for cmditem\n");
item->cmd = (char*)(item + 1);
strncpy(item->cmd, cmd, strlen(cmd) + 1);
strncpy(item->cmd, cmd, cmd_len);
ellAdd(&s_cmdlist, &item->node);