-
Notifications
You must be signed in to change notification settings - Fork 46
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
* resolved multi account conflicts with cve PR * resolved conflicts * resolved conflicts * minor changes in cve attackpath to integrate the aws inspector cves * minor correction in alert slideover component --------- Co-authored-by: Tushar Goel <[email protected]>
- Loading branch information
1 parent
f50a890
commit 4dfc7da
Showing
34 changed files
with
1,892 additions
and
317 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
157 changes: 157 additions & 0 deletions
157
backend/rules/attackpath/PubliclyExposedServerlessAdminCriticalCVE.go
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,157 @@ | ||
| package attackpath | ||
|
|
||
| import ( | ||
| "fmt" | ||
|
|
||
| "github.com/Zeus-Labs/ZeusCloud/rules/types" | ||
| "github.com/neo4j/neo4j-go-driver/v4/neo4j" | ||
| ) | ||
|
|
||
| type PubliclyExposedServerlessAdminCriticalCVE struct{} | ||
|
|
||
| func (PubliclyExposedServerlessAdminCriticalCVE) UID() string { | ||
| return "attackpath/publicly_exposed_serverless_admin_permissions_critical_cve" | ||
| } | ||
|
|
||
| func (PubliclyExposedServerlessAdminCriticalCVE) Description() string { | ||
| return "Publicly exposed serverless function with critical CVE and effective admin permissions." | ||
| } | ||
|
|
||
| func (PubliclyExposedServerlessAdminCriticalCVE) Severity() types.Severity { | ||
| return types.Critical | ||
| } | ||
|
|
||
| func (PubliclyExposedServerlessAdminCriticalCVE) RiskCategories() types.RiskCategoryList { | ||
| return []types.RiskCategory{ | ||
| types.PubliclyExposed, | ||
| types.IamMisconfiguration, | ||
| types.Vulnerability, | ||
| } | ||
| } | ||
|
|
||
| func (PubliclyExposedServerlessAdminCriticalCVE) Execute(tx neo4j.Transaction) ([]types.Result, error) { | ||
| records, err := tx.Run( | ||
| `MATCH (a:AWSAccount{inscope: true})-[:RESOURCE]->(lambda:AWSLambda) | ||
| OPTIONAL MATCH | ||
| (:IpRange{range:'0.0.0.0/0'})-[:MEMBER_OF_IP_RULE]-> | ||
| (perm:IpPermissionInbound)-[:MEMBER_OF_EC2_SECURITY_GROUP]-> | ||
| (elbv2_group:EC2SecurityGroup)<-[:MEMBER_OF_EC2_SECURITY_GROUP]- | ||
| (elbv2:LoadBalancerV2{scheme: 'internet-facing'})—[:ELBV2_LISTENER]-> | ||
| (listener:ELBV2Listener), | ||
| (elbv2CVE:CVE {severity:"critical"})<-[HAS_VULNERABILITY]-(elbv2)-[:EXPOSE]->(lambda) | ||
| WHERE listener.port >= perm.fromport AND listener.port <= perm.toport | ||
| WITH a, lambda, collect(distinct elbv2.id) as public_elbv2_ids, collect(distinct elbv2CVE.template_id) as elbv2_cve_ids | ||
| OPTIONAL MATCH | ||
| (lambda)-[:STS_ASSUME_ROLE_ALLOW]->(role:AWSRole{is_admin: True}) | ||
| WITH a, lambda, public_elbv2_ids, elbv2_cve_ids, collect(role.arn) as admin_roles, collect(role.admin_reason) as admin_reasons | ||
| WITH a, lambda, public_elbv2_ids, elbv2_cve_ids, admin_roles, admin_reasons, | ||
| (size(public_elbv2_ids) > 0 AND size(elbv2_cve_ids) > 0) as publicly_exposed_critical_cve_lambda, | ||
| (size(admin_roles) > 0) as is_admin | ||
| RETURN lambda.id as resource_id, | ||
| 'AWSLambda' as resource_type, | ||
| a.id as account_id, | ||
| CASE | ||
| WHEN publicly_exposed_critical_cve_lambda AND is_admin THEN 'failed' | ||
| ELSE 'passed' | ||
| END as status, | ||
| CASE | ||
| WHEN publicly_exposed_critical_cve_lambda THEN ( | ||
| 'The function is publicly exposed. ' + | ||
| CASE | ||
| WHEN size(public_elbv2_ids) > 0 THEN ( | ||
| 'The function is publicly exposed through these ELBv2 load balancers: ' + substring(apoc.text.join(public_elbv2_ids, ', '), 0, 1000) + './n' + | ||
| CASE | ||
| WHEN size(elbv2_cve_ids) > 0 THEN 'The following critical CVEs are detected for the ELBv2 load balancers: ' + substring(apoc.text.join(elbv2_cve_ids, ', '), 0, 1000) + '.' | ||
| ELSE 'No critical CVEs are detected for the ELBv2 load balancers.' | ||
| END | ||
| ) | ||
| ELSE 'The function is not publicly exposed through any ELBv2 load balancers.' | ||
| END | ||
| ) | ||
| ELSE 'The function is neither directly publicly exposed, nor indirectly public exposed through an ELBv2 load balancer.' | ||
| END + '\n' + | ||
| CASE | ||
| WHEN is_admin THEN ( | ||
| 'The function is effectively an admin in the account because of: ' + admin_reasons[0] + '.' | ||
| ) | ||
| ELSE 'The function was not detected as effectively an admin in the account.' | ||
| END as context`, | ||
| nil, | ||
| ) | ||
| if err != nil { | ||
| return nil, err | ||
| } | ||
|
|
||
| var results []types.Result | ||
| for records.Next() { | ||
| record := records.Record() | ||
| resourceID, _ := record.Get("resource_id") | ||
| resourceIDStr, ok := resourceID.(string) | ||
| if !ok { | ||
| return nil, fmt.Errorf("resource_id %v should be of type string", resourceID) | ||
| } | ||
| resourceType, _ := record.Get("resource_type") | ||
| resourceTypeStr, ok := resourceType.(string) | ||
| if !ok { | ||
| return nil, fmt.Errorf("resource_type %v should be of type string", resourceType) | ||
| } | ||
| accountID, _ := record.Get("account_id") | ||
| accountIDStr, ok := accountID.(string) | ||
| if !ok { | ||
| return nil, fmt.Errorf("account_id %v should be of type string", accountID) | ||
| } | ||
| status, _ := record.Get("status") | ||
| statusStr, ok := status.(string) | ||
| if !ok { | ||
| return nil, fmt.Errorf("status %v should be of type string", status) | ||
| } | ||
| context, _ := record.Get("context") | ||
| contextStr, ok := context.(string) | ||
| if !ok { | ||
| return nil, fmt.Errorf("context %v should be of type string", context) | ||
| } | ||
| results = append(results, types.Result{ | ||
| ResourceID: resourceIDStr, | ||
| ResourceType: resourceTypeStr, | ||
| AccountID: accountIDStr, | ||
| Status: statusStr, | ||
| Context: contextStr, | ||
| }) | ||
| } | ||
| return results, nil | ||
| } | ||
|
|
||
| func (PubliclyExposedServerlessAdminCriticalCVE) ProduceRuleGraph(tx neo4j.Transaction, resourceId string) (neo4j.Result, error) { | ||
| params := map[string]interface{}{ | ||
| "InstanceId": resourceId, | ||
| } | ||
| records, err := tx.Run( | ||
| `MATCH (a:AWSAccount{inscope: true})-[:RESOURCE]->(lambda:AWSLambda{id: $InstanceId}) | ||
| OPTIONAL MATCH | ||
| (:IpRange{range:'0.0.0.0/0'})-[:MEMBER_OF_IP_RULE]-> | ||
| (perm:IpPermissionInbound)-[:MEMBER_OF_EC2_SECURITY_GROUP]-> | ||
| (elbv2_group:EC2SecurityGroup)<-[:MEMBER_OF_EC2_SECURITY_GROUP]- | ||
| (elbv2:LoadBalancerV2{scheme: 'internet-facing'})—[:ELBV2_LISTENER]-> | ||
| (listener:ELBV2Listener), | ||
| (lambda)<-[:EXPOSE]-(elbv2)-[:HAS_VULNERABILITY]->(:CVE {severity:"critical"}) | ||
| WHERE listener.port >= perm.fromport AND listener.port <= perm.toport | ||
| OPTIONAL MATCH | ||
| indirectPath=(iprange)-[:MEMBER_OF_IP_RULE]->(perm)-[:MEMBER_OF_EC2_SECURITY_GROUP]-> | ||
| (elbv2_group)<-[:MEMBER_OF_EC2_SECURITY_GROUP]-(elbv2)-[:EXPOSE]->(lambda) | ||
| OPTIONAL MATCH | ||
| elbv2CvePath = (elbv2)-[HAS_VULNERABILITY]->(:CVE {severity:"critical"}) | ||
| WITH a, lambda, collect(indirectPath) as indirectPaths, collect(elbv2CvePath) as elbv2CvePaths | ||
| OPTIONAL MATCH | ||
| adminRolePath= | ||
| (lambda)-[:STS_ASSUME_ROLE_ALLOW]->(role:AWSRole{is_admin: True}) | ||
| WITH lambda, indirectPaths, elbv2CvePaths, collect(adminRolePath) as adminRolePaths | ||
| WITH indirectPaths + adminRolePaths + elbv2CvePaths AS paths | ||
| RETURN paths`, | ||
| params, | ||
| ) | ||
| if err != nil { | ||
| return nil, err | ||
| } | ||
|
|
||
| return records, nil | ||
| } |
Oops, something went wrong.