From bb43b5cac4002547ac787bea61a534eb48c725a3 Mon Sep 17 00:00:00 2001
From: Megan Marsh <megan@hashicorp.com>
Date: Tue, 14 Jul 2020 11:34:40 -0700
Subject: [PATCH] improve validation and error handling around synced_folder

---
 builder/vagrant/builder.go    |  5 +++++
 builder/vagrant/driver_2_2.go | 10 +++++++++-
 2 files changed, 14 insertions(+), 1 deletion(-)

diff --git a/builder/vagrant/builder.go b/builder/vagrant/builder.go
index c5110ede6..5692383bc 100644
--- a/builder/vagrant/builder.go
+++ b/builder/vagrant/builder.go
@@ -242,6 +242,11 @@ func (b *Builder) Prepare(raws ...interface{}) ([]string, []string, error) {
 		}
 	}
 
+	if _, err := os.Stat(b.config.SyncedFolder); err != nil {
+		errs = packer.MultiErrorAppend(errs,
+			fmt.Errorf("synced_folder \"%s\" does not exist on the Packer host.", b.config.SyncedFolder))
+	}
+
 	if errs != nil && len(errs.Errors) > 0 {
 		return nil, warnings, errs
 	}
diff --git a/builder/vagrant/driver_2_2.go b/builder/vagrant/driver_2_2.go
index 9e9679ff1..c1df9ea98 100644
--- a/builder/vagrant/driver_2_2.go
+++ b/builder/vagrant/driver_2_2.go
@@ -151,13 +151,21 @@ func (d *Vagrant_2_2_Driver) SSHConfig(id string) (*VagrantSSHConfig, error) {
 	if id != "" {
 		args = append(args, id)
 	}
-	stdout, _, err := d.vagrantCmd(args...)
 	sshConf := &VagrantSSHConfig{}
 
+	stdout, stderr, err := d.vagrantCmd(args...)
+	if stderr != "" {
+		err := fmt.Errorf("ssh-config command returned error: %s", stderr)
+		return sshConf, err
+	}
 	lines := strings.Split(stdout, "\n")
 	sshConf.Hostname = parseSSHConfig(lines, "HostName ")
 	sshConf.User = parseSSHConfig(lines, "User ")
 	sshConf.Port = parseSSHConfig(lines, "Port ")
+	if sshConf.Port == "" {
+		err := fmt.Errorf("error: SSH Port was not properly retrieved from SSHConfig.")
+		return sshConf, err
+	}
 	sshConf.UserKnownHostsFile = parseSSHConfig(lines, "UserKnownHostsFile ")
 	sshConf.IdentityFile = parseSSHConfig(lines, "IdentityFile ")
 	sshConf.LogLevel = parseSSHConfig(lines, "LogLevel ")