fix(core): Avoid capturing security group ingress rules with empty ci…

…dr's. (#1206)
spinnaker · Mar 6, 2017 · 4b80f7a · 4b80f7a
1 parent 10c2b89
commit 4b80f7a
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/...netflix/spinnaker/orca/clouddriver/tasks/providers/gce/GoogleSecurityGroupUpserter.groovy b/...netflix/spinnaker/orca/clouddriver/tasks/providers/gce/GoogleSecurityGroupUpserter.groovy
@@ -67,7 +67,7 @@ class GoogleSecurityGroupUpserter implements SecurityGroupUpserter, CloudProvide
 
       // What is currently cached?
       Set<String> existingSourceRanges = existingSecurityGroup.inboundRules.findResults { inboundRule ->
-        inboundRule.range ? inboundRule.range.ip + inboundRule.range.cidr : null
+        inboundRule.range?.ip ? inboundRule.range.ip + inboundRule.range.cidr : null
       }.unique() as Set
 
       boolean rangesMatch = existingSourceRanges == targetSecurityGroupSourceRanges