pick_address: don't bother checking struct ifaddrs which have a null ifa_addr

I assume that's the localhost interface or similar.

Signed-off-by: Greg Farnum <gregory.farnum@dreamhost.com>

diff --git a/src/common/pick_address.cc b/src/common/pick_address.cc
index 20f300ae78a9d1433a48532d5e83eb3e7a361363..a9788331761dcc1bb1baea58c9e8729e21dc6eb5 100644 (file)
--- a/src/common/pick_address.cc
+++ b/src/common/pick_address.cc
@@ -96,13 +96,15 @@ bool have_local_addr(CephContext *cct, const list<entity_addr_t>& ls, entity_add
 
   bool found = false;
   for (struct ifaddrs *addrs = ifa; addrs != NULL; addrs = addrs->ifa_next) {
-    entity_addr_t a;
-    a.set_sockaddr(addrs->ifa_addr);
-    for (list<entity_addr_t>::const_iterator p = ls.begin(); p != ls.end(); ++p) {
-      if (a.is_same_host(*p)) {
-       *match = *p;
-       found = true;
-       goto out;
+    if (addrs->ifa_addr) {
+      entity_addr_t a;
+      a.set_sockaddr(addrs->ifa_addr);
+      for (list<entity_addr_t>::const_iterator p = ls.begin(); p != ls.end(); ++p) {
+        if (a.is_same_host(*p)) {
+          *match = *p;
+          found = true;
+          goto out;
+        }
       }
     }
   }