From: Michael Labiuk <michael.lab...@virtuozzo.com> Using g_autofree char* and g_strdup_printf(...) instead of ugly snprintf on stack array.
Signed-off-by: Michael Labiuk <michael.lab...@virtuozzo.com> Message-Id: <20220929223547.1429580-8-michael.lab...@virtuozzo.com> Reviewed-by: Thomas Huth <th...@redhat.com> Signed-off-by: Thomas Huth <th...@redhat.com> --- tests/qtest/drive_del-test.c | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/tests/qtest/drive_del-test.c b/tests/qtest/drive_del-test.c index 44b9578801..106c613f4f 100644 --- a/tests/qtest/drive_del-test.c +++ b/tests/qtest/drive_del-test.c @@ -123,12 +123,10 @@ static const char *qvirtio_get_dev_type(void) static void device_add(QTestState *qts) { - QDict *response; - char driver[32]; - snprintf(driver, sizeof(driver), "virtio-blk-%s", - qvirtio_get_dev_type()); - - response = qtest_qmp(qts, "{'execute': 'device_add'," + g_autofree char *driver = g_strdup_printf("virtio-blk-%s", + qvirtio_get_dev_type()); + QDict *response = + qtest_qmp(qts, "{'execute': 'device_add'," " 'arguments': {" " 'driver': %s," " 'drive': 'drive0'," -- 2.31.1