net: Zero sockaddr_in in parse_host_port()
authorPeter Maydell <peter.maydell@linaro.org>
Fri, 13 Aug 2021 15:05:03 +0000 (16:05 +0100)
committerPeter Maydell <peter.maydell@linaro.org>
Thu, 26 Aug 2021 16:02:00 +0000 (17:02 +0100)
commit59292384621e93f707f862b6936694e56a6daed0
treeb7a91c8e469b70fc71695c20999cadf7043a971a
parent8efdb7ba1b2acce9fb63ccc2e7982e19fdf5be86
net: Zero sockaddr_in in parse_host_port()

We don't currently zero-initialize the 'struct sockaddr_in' that
parse_host_port() fills in, so any fields we don't explicitly
initialize might be left as random garbage.  POSIX states that
implementations may define extensions in sockaddr_in, and that those
extensions must not trigger if zero-initialized.  So not zero
initializing might result in inadvertently triggering an impdef
extension.

memset() the sockaddr_in before we start to fill it in.

Fixes: Coverity CID 1005338
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-id: 20210813150506.7768-2-peter.maydell@linaro.org
net/net.c