virgl: Array cmdline on stack should initialized to 0
authorYonggang Luo <luoyonggang@gmail.com>
Wed, 1 Feb 2023 07:11:00 +0000 (15:11 +0800)
committerMarge Bot <emma+marge@anholt.net>
Tue, 6 Jun 2023 13:28:19 +0000 (13:28 +0000)
Because strncpy may not append 0 if source string is longer than num(63)
Fixes: b3b82fe8ea1 ("virgl/vtest: add vtest driver")

Signed-off-by: Yonggang Luo <luoyonggang@gmail.com>
Reviewed-by: Gert Wollny <gert.wollny@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/21034>

src/gallium/winsys/virgl/vtest/virgl_vtest_socket.c

index fb6b27c..45f4d88 100644 (file)
@@ -119,7 +119,7 @@ static int virgl_vtest_send_init(struct virgl_vtest_winsys *vws)
 {
    uint32_t buf[VTEST_HDR_SIZE];
    const char *nstr = "virtest";
-   char cmdline[64];
+   char cmdline[64] = { 0 };
    int ret;
 
    ret = os_get_process_name(cmdline, 63);