aboutsummaryrefslogtreecommitdiff
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
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>
-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);
}