> +
> +      return getSecurityGroupById(id);
> +   }
> +
> +   @Override
> +   public SecurityGroup removeIpPermission(IpProtocol protocol, int 
> startPort, int endPort,
> +                                           Multimap<String, String> 
> tenantIdGroupNamePairs,
> +                                           Iterable<String> ipRanges,
> +                                           Iterable<String> groupIds, 
> SecurityGroup group) {
> +      IpPermission.Builder permBuilder = IpPermission.builder();
> +      permBuilder.ipProtocol(protocol);
> +      permBuilder.fromPort(startPort);
> +      permBuilder.toPort(endPort);
> +      permBuilder.tenantIdGroupNamePairs(tenantIdGroupNamePairs);
> +      permBuilder.cidrBlocks(ipRanges);
> +      permBuilder.groupIds(groupIds);

Any `null` checks needed here?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/152/files#r7377251

Reply via email to