simplified nil check

Author: gargipanatula

pkg/providers/v1/aws_loadbalancer.go

@@ -1510,21 +1510,17 @@ func (c *Cloud) ensureSSLNegotiationPolicy(ctx context.Context, loadBalancer *el
 			fmt.Sprintf(SSLNegotiationPolicyNameFormat, policyName),
 		},
 	})
-	policyNotFoundError := false
 	if err != nil {
+		// If DescribeLoadBalancerPolicies returns a PolicyNotFoundException, we must proceed and create the policy.
 		var notFoundErr *elbtypes.PolicyNotFoundException
 		if !errors.As(err, &notFoundErr) {
 			return fmt.Errorf("error describing security policies on load balancer: %q", err)
 		}
-		policyNotFoundError = true
 	}
 
-	// If DescribeLoadBalancerPolicies returns a PolicyNotFoundException, we must proceed and create the policy.
-	//
-	// The result of DescribeLoadBalancerPolicies will be nil, so we should only check
-	// result.PolicyDescriptions if DescribeLoadBalancerPolicies did not yield an error.
-	if (!policyNotFoundError) && (result != nil && len(result.PolicyDescriptions) > 0) {
-		// if policynotfound, would return false
+	// If DescribeLoadBalancerPolicies yielded a PolicyNotFoundException, result will be nil,
+	// so we must check before dereferencing
+	if result != nil && len(result.PolicyDescriptions) > 0 {
 		return nil
 	}