bblfshctl: fix NPE in driver remove; fixes #214

Signed-off-by: Denys Smirnov <[email protected]>
bblfsh · Nov 23, 2018 · cd80a49 · cd80a49
1 parent f1c65d7
commit cd80a49
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/cmd/bblfshctl/cmd/driver_remove.go b/cmd/bblfshctl/cmd/driver_remove.go
@@ -48,12 +48,13 @@ func (c *DriverRemoveCommand) Execute(args []string) error {
 
 	for _, lang := range langs {
 		r, err := c.srv.RemoveDriver(ctx, &protocol.RemoveDriverRequest{Language: lang})
-		if err != nil || len(r.Errors) > 0 {
+		if err != nil {
+			return err
+		} else if len(r.Errors) != 0 {
 			for _, e := range r.Errors {
 				fmt.Fprintf(os.Stderr, "Error, %s\n", e)
 			}
-
-			return err
+			return fmt.Errorf("driver remove failed: %v", r.Errors)
 		}
 	}
 	return nil