commit 75c970f64ed55630b645a9f98261e445749c7063
author Manoj Gupta <manojgupta@google.com> Tue Nov 22 21:15:59 2016 -0800
committer George Burgess IV <gbiv@google.com> Wed Nov 30 13:00:33 2016 -0800
tree 06be4015096f8cd1346db18368a96c398878808b
parent ff1587fc4ee0434129f6c2431eabeb291fdb0936

Fix clang static analyzer warnings.

system/netd/server/NetlinkHandler.cpp:218:12: warning: Dereference of
null pointer (loaded from variable 'gateway')

Test: Warning no longer appears
Change-Id: Idaa08940c990f7d572e855e77982ffd57a032dd4

server/NetlinkHandler.cpp

diff --git a/server/NetlinkHandler.cpp b/server/NetlinkHandler.cpp
index 19de240..e9a11ba 100644
--- a/server/NetlinkHandler.cpp
+++ b/server/NetlinkHandler.cpp
@@ -215,9 +215,9 @@
            "Route %s %s%s%s%s%s",
            (action == NetlinkEvent::Action::kRouteUpdated) ? kUpdated : kRemoved,
            route,
-           *gateway ? " via " : "",
+           (gateway && *gateway) ? " via " : "",
            gateway,
-           *iface ? " dev " : "",
+           (iface && *iface) ? " dev " : "",
            iface);
 }