From f03175b4af289f333c9b62c350ed0811c9a84b5e Mon Sep 17 00:00:00 2001 From: Mitchell Hashimoto Date: Fri, 15 Mar 2013 15:15:54 -0700 Subject: [PATCH] Don't "return" if FP less than port 1024 [GH-1421] --- CHANGELOG.md | 2 ++ plugins/providers/virtualbox/action/forward_ports.rb | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 2af337298..04537e578 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -14,6 +14,8 @@ BUG FIXES: - Human friendly error when "metadata.json" is missing in a box. - Don't use the full path to the manifest file with the Puppet provisioner because it exposes a bug with Puppet path lookup on VMware. + - Fix bug in VirtualBox provider where port forwarding just didn't work if + you attempted to forward to a port under 1024. [GH-1421] ## 1.1.0 (March 14, 2013) diff --git a/plugins/providers/virtualbox/action/forward_ports.rb b/plugins/providers/virtualbox/action/forward_ports.rb index 1b3a62f6b..55f30dd31 100644 --- a/plugins/providers/virtualbox/action/forward_ports.rb +++ b/plugins/providers/virtualbox/action/forward_ports.rb @@ -21,7 +21,7 @@ module VagrantPlugins env[:forwarded_ports].each do |fp| if fp.host_port <= 1024 env[:ui].warn I18n.t("vagrant.actions.vm.forward_ports.privileged_ports") - return + break end end