commit 6860710cc3864382a898c847d722f950b5e01a6e
author Philippe Mathieu-Daudé <f4bug@amsat.org> Wed Jul 26 23:42:18 2017 -0300
committer Michael Tokarev <mjt@tls.msk.ru> Mon Jul 31 13:06:39 2017 +0300
tree 0983297b681c27070cf7aedb84a4af648260486c
parent bc658e4a2e81593f75a3ae34b112be77efbb3e0a

syscall: fix use of uninitialized values

linux-user/syscall.c:1627:35: warning: 1st function call argument is an uninitialized value
        target_saddr->sa_family = tswap16(addr->sa_family);
                                  ^~~~~~~~~~~~~~~~~~~~~~~~
linux-user/syscall.c:1629:25: warning: The left operand of '==' is a garbage value
    if (addr->sa_family == AF_NETLINK && len >= sizeof(struct sockaddr_nl)) {
        ~~~~~~~~~~~~~~~ ^

Reported-by: Clang Static Analyzer
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Laurent Vivier <laurent@vivier.eu>
Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>

linux-user/syscall.c

diff --git a/linux-user/syscall.c b/linux-user/syscall.c
index 71d45a9..81f52f7 100644
--- a/linux-user/syscall.c
+++ b/linux-user/syscall.c
@@ -1622,6 +1622,7 @@
     if (len == 0) {
         return 0;
     }
+    assert(addr);
 
     target_saddr = lock_user(VERIFY_WRITE, target_addr, len, 0);
     if (!target_saddr)