vcsim: esxcli FirewallInfo fixes (#661)

- Add missing error check when calling NewExecutor

- Add RetrieveManagedMethodExecuter to type map

govc/host/esxcli/firewall_info.go

@@ -29,6 +29,9 @@ type FirewallInfo struct {
 // This helper can be useful in particular to determine if the firewall is enabled or disabled.
 func GetFirewallInfo(s *object.HostSystem) (*FirewallInfo, error) {
 	x, err := NewExecutor(s.Client(), s)
+	if err != nil {
+		return nil, err
+	}
 
 	res, err := x.Run([]string{"network", "firewall", "get"})
 	if err != nil {

vim25/types/internal.go

@@ -237,6 +237,10 @@ type RetrieveManagedMethodExecuter struct {
 	This ManagedObjectReference `xml:"_this"`
 }
 
+func init() {
+	t["RetrieveManagedMethodExecuter"] = reflect.TypeOf((*RetrieveManagedMethodExecuter)(nil)).Elem()
+}
+
 type RetrieveManagedMethodExecuterResponse struct {
 	Returnval *ReflectManagedMethodExecuter `xml:"urn:vim25 returnval"`
 }