Skip to content

Commit 8109c2e

Browse files
taoyuhongwenlingz
authored andcommitted
DM: restrict vmname size to 32 bytes
Length of vmname is restricted to MAX_VM_OS_NAME_LEN. Keep it coincide with VM name array size in vm_config. Tracked-On: #2851 Signed-off-by: Tao Yuhong <yuhong.tao@intel.com> Acked-by: Yin Fengwei <fengwei.yin@intel.com> Acked-by: Eddie Dong <eddie.dong@intel.com>
1 parent b1586cc commit 8109c2e

File tree

2 files changed

+5
-1
lines changed

2 files changed

+5
-1
lines changed

devicemodel/core/main.c

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -924,6 +924,10 @@ dm_run(int argc, char *argv[])
924924
}
925925

926926
vmname = argv[0];
927+
if (strnlen(vmname, MAX_VM_OS_NAME_LEN) >= MAX_VM_OS_NAME_LEN) {
928+
fprintf(stderr, "vmname size exceed %u\n",MAX_VM_OS_NAME_LEN);
929+
exit(1);
930+
}
927931

928932
for (;;) {
929933
ctx = vm_create(vmname, (unsigned long)vhm_req_buf);

devicemodel/hw/platform/ioc.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ typedef void* (*ioc_work)(void *arg);
100100
* IOC mediator and virtual UART communication channel path,
101101
* comes from DM command line parameters.
102102
*/
103-
static char virtual_uart_path[32];
103+
static char virtual_uart_path[32 + MAX_VM_OS_NAME_LEN];
104104

105105
/*
106106
* To activate CBC signal channel(/dev/cbc-signals).

0 commit comments

Comments
 (0)