Merge pull request #733 from majestrate/fix-crash-2019-07-26

fix crash of clients and propagate lookup failures
5 years ago · 4501156aff
parent ce2ff2a93e 972d4f8672
commit 4501156aff
1 changed files with 6 additions and 3 deletions
--- a/llarp/router/router.cpp
+++ b/llarp/router/router.cpp
@ -728,10 +728,13 @@ namespace llarp
        [=](const RouterID &id, const RouterContact *const rc,
            const RCRequestResult result) {
          (void)id;
-          std::vector< RouterContact > routers;
-          if(result == RCRequestResult::Success && rc != nullptr)
+          if(resultHandler)
          {
-            routers.push_back(*rc);
+            std::vector< RouterContact > routers;
+            if(result == RCRequestResult::Success && rc != nullptr)
+            {
+              routers.push_back(*rc);
+            }
            resultHandler(routers);
          }
        });