Router: remove useless check from ClientManager. Use main netDb if the runner performing a lookup has a null destHash.

Author: eyedeekay

router/java/src/net/i2p/router/client/ClientManager.java

@@ -785,11 +785,6 @@ public FloodfillNetworkDatabaseFacade getClientFloodfillNetworkDatabaseFacade(Ha
             if (_log.shouldLog(Log.DEBUG))
                 _log.debug("Getting subDb for desthash: " + destHash);
             ClientConnectionRunner runner = getRunner(destHash);
-            if (runner == null){
-                if (_log.shouldLog(Log.WARN))
-                _log.warn("ClientManager got a null runner in getClientFloodfillNetworkDatabaseFacade for " + destHash);
-                return null;
-            }
             if (_log.shouldLog(Log.DEBUG))
                 _log.debug("ClientManager got a runner in getClientFloodfillNetworkDatabaseFacade for " + destHash);
             return runner.getFloodfillNetworkDatabaseFacade();

router/java/src/net/i2p/router/client/LookupDestJob.java

@@ -274,6 +274,11 @@ else if (_hash != null)
      * @return
      */
     private FloodfillNetworkDatabaseFacade getRequiredFloodfillNetworkDatabaseFacade() {
+        if (_runner.getDestHash() == null) {
+            if (_log.shouldLog(Log.DEBUG))
+                _log.debug("destHash is null in runner in lookup for " + _hash + " using the main DB instead");
+            return getContext().netDb();
+        }
         FloodfillNetworkDatabaseFacade fndf = _runner.getFloodfillNetworkDatabaseFacade();
         if (fndf == null){
             fndf = getContext().netDb();