Land rapid7#4902, @nstarke's db_connect warning message

Author: wvu

lib/msf/ui/console/command_dispatcher/db.rb

@@ -1687,12 +1687,12 @@ def cmd_db_status(*args)
     return if not db_check_driver
 
     if framework.db.connection_established?
-      cdb = ""
-      ::ActiveRecord::Base.connection_pool.with_connection { |conn|
-        if conn.respond_to? :current_database
+      cdb = ''
+      ::ActiveRecord::Base.connection_pool.with_connection do |conn|
+        if conn.respond_to?(:current_database)
           cdb = conn.current_database
         end
-      }
+      end
       print_status("#{framework.db.driver} connected to #{cdb}")
     else
       print_status("#{framework.db.driver} selected, no connection")
@@ -1706,6 +1706,17 @@ def cmd_db_connect_help
 
   def cmd_db_connect(*args)
     return if not db_check_driver
+    if args[0] != '-h' && framework.db.connection_established?
+      cdb = ''
+      ::ActiveRecord::Base.connection_pool.with_connection do |conn|
+        if conn.respond_to?(:current_database)
+          cdb = conn.current_database
+        end
+      end
+      print_error("#{framework.db.driver} already connected to #{cdb}")
+      print_error('Run db_disconnect first if you wish to connect to a different database')
+      return
+    end
     if (args[0] == "-y")
       if (args[1] and not ::File.exists? ::File.expand_path(args[1]))
         print_error("File not found")