[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: |
Corey Minyard |
Subject: |
Re: [PATCH for-6.2 3/4] tests/qtest/ipmi-bt-test: Zero-initialize sockaddr struct |
Date: |
Sat, 14 Aug 2021 10:46:18 -0500 |
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>
Looks good to me. Thanks, Peter. I looked through all the other
patches, too, and reviewed those.
Reviewed-by: Corey Minyard <cminyard@mvista.com>
> ---
> tests/qtest/ipmi-bt-test.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> 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
>
- [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