packer/rpc: Panic in error case

a73e71c3 · Mitchell Hashimoto · 67eaa07c · a73e71c3
Commit a73e71c3 authored Jun 15, 2013 by Mitchell Hashimoto
Show whitespace changes
Inline Side-by-side

Showing with 5 additions and 2 deletions

packer/rpc/environment.go packer/rpc/environment.go +5 -2

No files found.
--- a/packer/rpc/environment.go
+++ b/packer/rpc/environment.go
@@ -93,8 +93,11 @@ func (e *Environment) Ui() packer.Ui {
 	var reply string
 	e.client.Call("Environment.Ui", new(interface{}), &reply)

-	// TODO: error handling
-	client, _ := rpc.Dial("tcp", reply)
+	client, err := rpc.Dial("tcp", reply)
+	if err != nil {
+		panic(err)
+	}
+
 	return &Ui{client}
 }