Skip to content

Commit 5791e8b

Browse files
committed
Slight renaming
1 parent 07de92c commit 5791e8b

File tree

1 file changed

+6
-4
lines changed
  • swift/ql/lib/codeql/swift/security

1 file changed

+6
-4
lines changed

swift/ql/lib/codeql/swift/security/XXE.qll

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -133,7 +133,9 @@ private class VulnerableAexmlDocument extends AexmlDocument {
133133
*/
134134
private class VulnerableAexmlOptions extends AexmlOptions {
135135
VulnerableAexmlOptions() {
136-
exists(ParserSettings parserSettings, AexmlShouldResolveExternalEntities sree, AssignExpr a |
136+
exists(
137+
AexmlParserSettings parserSettings, AexmlShouldResolveExternalEntities sree, AssignExpr a
138+
|
137139
a.getSource() = any(BooleanLiteralExpr b | b.getValue() = true) and
138140
a.getDest() = sree and
139141
sree.(MemberRefExpr).getBase() = parserSettings and
@@ -143,17 +145,17 @@ private class VulnerableAexmlOptions extends AexmlOptions {
143145
}
144146

145147
/** An expression of type `AEXMLOptions.ParserSettings`. */
146-
class ParserSettings extends Expr {
148+
private class AexmlParserSettings extends Expr {
147149
pragma[inline]
148-
ParserSettings() {
150+
AexmlParserSettings() {
149151
this.getType() instanceof AexmlOptionsParserSettingsType or
150152
this.getType() = any(OptionalType t | t.getBaseType() instanceof AexmlOptionsParserSettingsType) or
151153
this.getType() = any(LValueType t | t.getObjectType() instanceof AexmlOptionsParserSettingsType)
152154
}
153155
}
154156

155157
/** An expression of type `AEXMLOptions`. */
156-
class AexmlOptions extends Expr {
158+
private class AexmlOptions extends Expr {
157159
pragma[inline]
158160
AexmlOptions() {
159161
this.getType() instanceof AexmlOptionsType or

0 commit comments

Comments
 (0)