aboutsummaryrefslogtreecommitdiff
path: root/tests/qtest
diff options
context:
space:
mode:
authorChristian Schoenebeck <qemu_oss@crudebyte.com>2020-02-08 09:24:19 +0100
committerGreg Kurz <groug@kaod.org>2020-02-08 09:24:19 +0100
commit2e2293c238d4daa791ab7ff60b326a322892df3a (patch)
tree789ab56f63b377415ef6c4b2044d6bd18b38785a /tests/qtest
parent42ccca1bd9456568f996d5646b2001faac96944b (diff)
downloadqemu-2e2293c238d4daa791ab7ff60b326a322892df3a.zip
qemu-2e2293c238d4daa791ab7ff60b326a322892df3a.tar.gz
qemu-2e2293c238d4daa791ab7ff60b326a322892df3a.tar.bz2
tests/virtio-9p: add terminating null in v9fs_string_read()
The 9p protocol sends strings in general without null termination over the wire. However for future use of this functions it is beneficial for the delivered string to be null terminated though for being able to use the string with standard C functions which often rely on strings being null terminated. Signed-off-by: Christian Schoenebeck <qemu_oss@crudebyte.com> Reviewed-by: Greg Kurz <groug@kaod.org> Message-Id: <52c84e2ce3bcafc2a38eed13b8c8e23bc1a8ecb9.1579567019.git.qemu_oss@crudebyte.com> Signed-off-by: Greg Kurz <groug@kaod.org>
Diffstat (limited to 'tests/qtest')
-rw-r--r--tests/qtest/virtio-9p-test.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/tests/qtest/virtio-9p-test.c b/tests/qtest/virtio-9p-test.c
index e7b58e3..06263ed 100644
--- a/tests/qtest/virtio-9p-test.c
+++ b/tests/qtest/virtio-9p-test.c
@@ -130,8 +130,9 @@ static void v9fs_string_read(P9Req *req, uint16_t *len, char **string)
*len = local_len;
}
if (string) {
- *string = g_malloc(local_len);
+ *string = g_malloc(local_len + 1);
v9fs_memread(req, *string, local_len);
+ (*string)[local_len] = 0;
} else {
v9fs_memskip(req, local_len);
}