Merge branch 'main' into schedule

kyma-project · Jan 28, 2025 · 53950dc · 53950dc
2 parents ecf2a77 + 187ba20
commit 53950dc
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 9 deletions.
diff --git a/.checkmarx/config.yml b/.checkmarx/config.yml
diff --git a/pkg/kcp/provider/aws/mock/routeTablesStore.go b/pkg/kcp/provider/aws/mock/routeTablesStore.go
@@ -108,12 +108,17 @@ func (s *routeTablesStore) DeleteRoute(ctx context.Context, routeTableId, destin
 }
 
 func (s *routeTablesStore) GetRouteCount(ctx context.Context, vpcId, vpcPeeringConnectionId, destinationCidrBlock string) int {
-	tables, _ := s.DescribeRouteTables(ctx, vpcId)
+	tables, err := s.DescribeRouteTables(ctx, vpcId)
+
+	if err != nil {
+		return -1
+	}
+
 	cnt := 0
 	for _, t := range tables {
 		for _, r := range t.Routes {
-			if *r.VpcPeeringConnectionId == vpcPeeringConnectionId &&
-				*r.DestinationCidrBlock == destinationCidrBlock {
+			if ptr.Deref(r.VpcPeeringConnectionId, "") == vpcPeeringConnectionId &&
+				ptr.Deref(r.DestinationCidrBlock, "") == destinationCidrBlock {
 				cnt++
 			}
 		}