Apply suggestions from code review

Co-authored-by: Wei Zhou <[email protected]>
apache · Jan 27, 2025 · 22462bd · 22462bd
1 parent 4df8b55
commit 22462bd
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/systemvm/debian/opt/cloud/bin/cs/CsAddress.py b/systemvm/debian/opt/cloud/bin/cs/CsAddress.py
@@ -559,11 +559,11 @@ def fw_vpcrouter(self):
                     if item == "id":
                         continue
                     static_route = static_routes.get_bag()[item]
-                    if static_route['ip_address'] == self.address['public_ip'] and not static_route['revoke']:
+                    if 'ip_address' in static_route and static_route['ip_address'] == self.address['public_ip'] and not static_route['revoke']:
                         self.fw.append(["mangle", "",
                                         "-A PREROUTING -m state --state NEW -i %s -s %s ! -d %s/32 -j ACL_OUTBOUND_%s" %
                                         (self.dev, static_route['network'], static_route['ip_address'], self.dev)])
-                        self.fw.append(["filter", "", "-A FORWARD -d %s -o %s -j ACL_INBOUND_%s" %
+                        self.fw.append(["filter", "front", "-A FORWARD -d %s -o %s -j ACL_INBOUND_%s" %
                                 (static_route['network'], self.dev, self.dev)])
 
             if self.address["source_nat"]: