diff options
| author | Jens Freimann <jfreimann@redhat.com> | 2017-11-06 15:05:46 +0100 |
|---|---|---|
| committer | Jason Wang <jasowang@redhat.com> | 2017-11-13 18:05:12 +0800 |
| commit | bb160b571fe469b03228d4502c75a18045978a74 (patch) | |
| tree | f42459bb94cd7f55a75a7cfce3d2e8ee2393000a /net | |
| parent | 5e89dc01133f8f5e621f6b66b356c6f37d31dafb (diff) | |
| download | qemu-bb160b571fe469b03228d4502c75a18045978a74.zip qemu-bb160b571fe469b03228d4502c75a18045978a74.tar.gz qemu-bb160b571fe469b03228d4502c75a18045978a74.tar.bz2 | |
net/socket: fix coverity issue
This fixes coverity issue CID1005339.
Make sure that saddr is not used uninitialized if the
mcast parameter is NULL.
Cc: qemu-stable@nongnu.org
Reported-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Jens Freimann <jfreimann@redhat.com>
Signed-off-by: Jason Wang <jasowang@redhat.com>
Diffstat (limited to 'net')
| -rw-r--r-- | net/socket.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/socket.c b/net/socket.c index 83a2a31..6917fbc 100644 --- a/net/socket.c +++ b/net/socket.c @@ -373,7 +373,7 @@ static NetSocketState *net_socket_fd_init_dgram(NetClientState *peer, net_socket_read_poll(s, true); /* mcast: save bound address as dst */ - if (is_connected) { + if (is_connected && mcast != NULL) { s->dgram_dst = saddr; snprintf(nc->info_str, sizeof(nc->info_str), "socket: fd=%d (cloned mcast=%s:%d)", |