diff options
author | Thomas Huth <thuth@redhat.com> | 2024-04-29 09:59:08 +0200 |
---|---|---|
committer | Peter Maydell <peter.maydell@linaro.org> | 2024-04-30 15:45:37 +0100 |
commit | afdc29b4a3a5a61c22342551bb38eea3731bfee1 (patch) | |
tree | 7120f355271c31556c667050a4f1076234282707 | |
parent | 4b00855f0ee2e2eee8fd2500ffef27c108be6dc3 (diff) | |
download | qemu-afdc29b4a3a5a61c22342551bb38eea3731bfee1.zip qemu-afdc29b4a3a5a61c22342551bb38eea3731bfee1.tar.gz qemu-afdc29b4a3a5a61c22342551bb38eea3731bfee1.tar.bz2 |
hw/char/stm32l4x5_usart: Fix memory corruption by adding correct class_size
"make check-qtest-aarch64" recently started failing on FreeBSD builds,
and valgrind on Linux also detected that there is something fishy with
the new stm32l4x5-usart: The code forgot to set the correct class_size
here, so the various class_init functions in this file wrote beyond
the allocated buffer when setting the subc->type field.
Fixes: 4fb37aea7e ("hw/char: Implement STM32L4x5 USART skeleton")
Signed-off-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-id: 20240429075908.36302-1-thuth@redhat.com
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
-rw-r--r-- | hw/char/stm32l4x5_usart.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/hw/char/stm32l4x5_usart.c b/hw/char/stm32l4x5_usart.c index 2627aab..02f6663 100644 --- a/hw/char/stm32l4x5_usart.c +++ b/hw/char/stm32l4x5_usart.c @@ -617,6 +617,7 @@ static const TypeInfo stm32l4x5_usart_types[] = { .parent = TYPE_SYS_BUS_DEVICE, .instance_size = sizeof(Stm32l4x5UsartBaseState), .instance_init = stm32l4x5_usart_base_init, + .class_size = sizeof(Stm32l4x5UsartBaseClass), .class_init = stm32l4x5_usart_base_class_init, .abstract = true, }, { |