[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH for-6.2 3/4] tests/qtest/ipmi-bt-test: Zero-initialize sockad
From: |
Eric Blake |
Subject: |
Re: [PATCH for-6.2 3/4] tests/qtest/ipmi-bt-test: Zero-initialize sockaddr struct |
Date: |
Fri, 13 Aug 2021 13:38:28 -0500 |
User-agent: |
NeoMutt/20210205-687-0ed190 |
On Fri, Aug 13, 2021 at 04:05:05PM +0100, Peter Maydell wrote:
> Zero-initialize the sockaddr_in struct that we're about to fill in
> and pass to bind(), to ensure we don't leave possible
> implementation-defined extension fields as uninitialized garbage.
>
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> ---
> tests/qtest/ipmi-bt-test.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Eric Blake <eblake@redhat.com>
>
> diff --git a/tests/qtest/ipmi-bt-test.c b/tests/qtest/ipmi-bt-test.c
> index 8492f02a9c3..19612e9405a 100644
> --- a/tests/qtest/ipmi-bt-test.c
> +++ b/tests/qtest/ipmi-bt-test.c
> @@ -378,7 +378,7 @@ static void test_enable_irq(void)
> */
> static void open_socket(void)
> {
> - struct sockaddr_in myaddr;
> + struct sockaddr_in myaddr = {};
> socklen_t addrlen;
>
> myaddr.sin_family = AF_INET;
> --
> 2.20.1
>
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
- [PATCH for-6.2 0/4] Zero sockaddr_in when initializing it, Peter Maydell, 2021/08/13
- [PATCH for-6.2 1/4] net: Zero sockaddr_in in parse_host_port(), Peter Maydell, 2021/08/13
- [PATCH for-6.2 2/4] gdbstub: Zero-initialize sockaddr structs, Peter Maydell, 2021/08/13
- [PATCH for-6.2 3/4] tests/qtest/ipmi-bt-test: Zero-initialize sockaddr struct, Peter Maydell, 2021/08/13
- [PATCH for-6.2 4/4] tests/tcg/multiarch/linux-test: Zero-initialize sockaddr structs, Peter Maydell, 2021/08/13
- Re: [PATCH for-6.2 0/4] Zero sockaddr_in when initializing it, Eric Blake, 2021/08/13
- Re: [PATCH for-6.2 0/4] Zero sockaddr_in when initializing it, Peter Maydell, 2021/08/26