Skip to content

Commit efe8564

Browse files
committed
Added CWE informatino for Semgrep, Defectdojo, and some pixee codemods
1 parent c54c9d7 commit efe8564

File tree

8 files changed

+23
-5
lines changed

8 files changed

+23
-5
lines changed

src/core_codemods/defectdojo/semgrep/avoid_insecure_deserialization.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
LibcstTransformerPipeline,
77
)
88
from codemodder.codemods.utils_mixin import NameResolutionMixin
9+
from codemodder.codetf import Reference
910
from core_codemods.defectdojo.api import DefectDojoCodemod, DefectDojoDetector
1011
from core_codemods.harden_pickle_load import HardenPickleLoad
1112
from core_codemods.harden_pyyaml import CodemodProtocol, HardenPyyamlCallMixin
@@ -56,7 +57,9 @@ def leave_Call(
5657
)
5758
],
5859
),
59-
references=[],
60+
references=[
61+
Reference(url="https://cwe.mitre.org/data/definitions/502.html"),
62+
],
6063
),
6164
transformer=LibcstTransformerPipeline(
6265
AvoidInsecureDeserializationTransformer, HardenPickleLoad

src/core_codemods/defectdojo/semgrep/django_secure_set_cookie.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
LibcstTransformerPipeline,
77
)
88
from codemodder.codemods.utils_mixin import NameResolutionMixin
9+
from codemodder.codetf import Reference
910
from core_codemods.defectdojo.api import DefectDojoCodemod, DefectDojoDetector
1011
from core_codemods.secure_cookie_mixin import SecureCookieMixin
1112

@@ -50,7 +51,9 @@ def leave_Call(self, original_node: cst.Call, updated_node: cst.Call) -> cst.Cal
5051
)
5152
],
5253
),
53-
references=[],
54+
references=[
55+
Reference(url="https://cwe.mitre.org/data/definitions/614.html"),
56+
],
5457
),
5558
transformer=LibcstTransformerPipeline(DjangoSecureSetCookieTransformer),
5659
detector=DefectDojoDetector(),

src/core_codemods/harden_pyyaml.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -126,6 +126,7 @@ def _update_bases(
126126
Reference(
127127
url="https://github.com/yaml/pyyaml/wiki/PyYAML-yaml.load(input)-Deprecation"
128128
),
129+
Reference(url="https://cwe.mitre.org/data/definitions/502.html"),
129130
],
130131
),
131132
detector=SemgrepRuleDetector(

src/core_codemods/semgrep/semgrep_nan_injection.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
LibcstTransformerPipeline,
1616
)
1717
from codemodder.codemods.semgrep import SemgrepSarifFileDetector
18+
from codemodder.codetf import Reference
1819
from core_codemods.semgrep.api import SemgrepCodemod, semgrep_url_from_id
1920

2021

@@ -124,7 +125,9 @@ def visit_Call(self, node: cst.Call) -> None:
124125
)
125126
],
126127
),
127-
references=[],
128+
references=[
129+
Reference(url="https://cwe.mitre.org/data/definitions/704.html"),
130+
],
128131
),
129132
transformer=LibcstTransformerPipeline(NanInjectionTransformer),
130133
detector=SemgrepSarifFileDetector(),

src/core_codemods/semgrep/semgrep_no_csrf_exempt.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212
)
1313
from codemodder.codemods.semgrep import SemgrepSarifFileDetector
1414
from codemodder.codemods.utils_mixin import NameResolutionMixin
15+
from codemodder.codetf import Reference
1516
from core_codemods.semgrep.api import SemgrepCodemod, semgrep_url_from_id
1617

1718

@@ -53,7 +54,9 @@ def leave_Decorator(
5354
)
5455
],
5556
),
56-
references=[],
57+
references=[
58+
Reference(url="https://cwe.mitre.org/data/definitions/352.html"),
59+
],
5760
),
5861
transformer=LibcstTransformerPipeline(RemoveCsrfExemptTransformer),
5962
detector=SemgrepSarifFileDetector(),

src/core_codemods/semgrep/semgrep_rsa_key_size.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212
NewArg,
1313
)
1414
from codemodder.codemods.semgrep import SemgrepSarifFileDetector
15+
from codemodder.codetf import Reference
1516
from codemodder.result import fuzzy_column_match, same_line
1617
from core_codemods.semgrep.api import SemgrepCodemod, semgrep_url_from_id
1718

@@ -74,7 +75,9 @@ def match_location(self, pos, result):
7475
)
7576
],
7677
),
77-
references=[],
78+
references=[
79+
Reference(url="https://cwe.mitre.org/data/definitions/326.html"),
80+
],
7881
),
7982
transformer=LibcstTransformerPipeline(RsaKeySizeTransformer),
8083
detector=SemgrepSarifFileDetector(),

src/core_codemods/subprocess_shell_false.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -79,6 +79,7 @@ def first_arg_is_not_string(self, original_node: cst.Call) -> bool:
7979
url="https://en.wikipedia.org/wiki/Code_injection#Shell_injection"
8080
),
8181
Reference(url="https://stackoverflow.com/a/3172488"),
82+
Reference(url="https://cwe.mitre.org/data/definitions/78.html"),
8283
],
8384
),
8485
transformer=LibcstTransformerPipeline(SubprocessShellFalseTransformer),

src/core_codemods/use_defused_xml.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,7 @@ def dependency(self) -> Dependency:
5151
Reference(
5252
url="https://cheatsheetseries.owasp.org/cheatsheets/XML_External_Entity_Prevention_Cheat_Sheet.html"
5353
),
54+
Reference(url="https://cwe.mitre.org/data/definitions/611.html"),
5455
],
5556
),
5657
transformer=LibcstTransformerPipeline(UseDefusedXmlTransformer),

0 commit comments

Comments
 (0)