diff --git a/builder/osc/common/state.go b/builder/osc/common/state.go
index fc6060743..a2e0e53a7 100644
--- a/builder/osc/common/state.go
+++ b/builder/osc/common/state.go
@@ -78,7 +78,6 @@ func waitForState(errCh chan<- error, target string, refresh stateRefreshFunc) {
 		return false, nil
 	})
 	errCh <- err
-	return
 }
 
 func waitUntilOscVmStateFunc(conn *osc.APIClient, id string) stateRefreshFunc {
diff --git a/builder/osc/common/step_security_group.go b/builder/osc/common/step_security_group.go
index ae591ff12..7c4bceba1 100644
--- a/builder/osc/common/step_security_group.go
+++ b/builder/osc/common/step_security_group.go
@@ -130,7 +130,7 @@ func (s *StepSecurityGroup) Run(_ context.Context, state multistep.StateBag) mul
 
 	ui.Say(fmt.Sprintf("Authorizing access to port %d from %s in the temporary security group...", port, s.TemporarySGSourceCidr))
 
-	conn.SecurityGroupRuleApi.CreateSecurityGroupRule(context.Background(), &osc.CreateSecurityGroupRuleOpts{
+	_, _, err = conn.SecurityGroupRuleApi.CreateSecurityGroupRule(context.Background(), &osc.CreateSecurityGroupRuleOpts{
 		CreateSecurityGroupRuleRequest: optional.NewInterface(createSGRReq),
 	})