diff options
| author | Marc-André Lureau <marcandre.lureau@redhat.com> | 2022-04-20 17:25:57 +0400 |
|---|---|---|
| committer | Marc-André Lureau <marcandre.lureau@redhat.com> | 2022-04-21 17:03:51 +0400 |
| commit | aaea18d5d697d94ba63122a644690b9a9e7db007 (patch) | |
| tree | b54cc20cbea4cb31649b0d55f792420b7e8ced24 /hw/arm | |
| parent | e5c41835250ebb3f0297606bc7cc48cfa5474e8f (diff) | |
| download | qemu-aaea18d5d697d94ba63122a644690b9a9e7db007.zip qemu-aaea18d5d697d94ba63122a644690b9a9e7db007.tar.gz qemu-aaea18d5d697d94ba63122a644690b9a9e7db007.tar.bz2 | |
arm/allwinner-a10: replace snprintf() with g_strdup_printf()
Also fixes a GCC 12.0.1 false-positive:
../hw/arm/allwinner-a10.c: In function ‘aw_a10_realize’:
../hw/arm/allwinner-a10.c:135:35: error: ‘%d’ directive writing between 1 and 11 bytes into a region of size 8 [-Werror=format-overflow=]
135 | sprintf(bus, "usb-bus.%d", i);
| ^~
Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Message-Id: <20220420132624.2439741-15-marcandre.lureau@redhat.com>
Diffstat (limited to 'hw/arm')
| -rw-r--r-- | hw/arm/allwinner-a10.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/hw/arm/allwinner-a10.c b/hw/arm/allwinner-a10.c index 05e8472..7908228 100644 --- a/hw/arm/allwinner-a10.c +++ b/hw/arm/allwinner-a10.c @@ -130,9 +130,7 @@ static void aw_a10_realize(DeviceState *dev, Error **errp) int i; for (i = 0; i < AW_A10_NUM_USB; i++) { - char bus[16]; - - sprintf(bus, "usb-bus.%d", i); + g_autofree char *bus = g_strdup_printf("usb-bus.%d", i); object_property_set_bool(OBJECT(&s->ehci[i]), "companion-enable", true, &error_fatal); |