Skip to content

Commit da43984

Browse files
committed
fix dependency cycle by removing superfluous classes
1 parent ee23799 commit da43984

File tree

1 file changed

+0
-26
lines changed
  • ruby/ql/lib/codeql/ruby/security

1 file changed

+0
-26
lines changed

ruby/ql/lib/codeql/ruby/security/XSS.qll

Lines changed: 0 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -274,19 +274,6 @@ module ReflectedXSS {
274274
/** A sanitizer guard for stored XSS vulnerabilities. */
275275
abstract class SanitizerGuard extends Shared::SanitizerGuard { }
276276

277-
// Consider all arbitrary XSS sinks to be reflected XSS sinks
278-
private class AnySink extends Sink instanceof Shared::Sink { }
279-
280-
// Consider all arbitrary XSS sanitizers to be reflected XSS sanitizers
281-
private class AnySanitizer extends Sanitizer instanceof Shared::Sanitizer { }
282-
283-
// Consider all arbitrary XSS sanitizer guards to be reflected XSS sanitizer guards
284-
private class AnySanitizerGuard extends SanitizerGuard instanceof Shared::SanitizerGuard {
285-
override predicate checks(CfgNode expr, boolean branch) {
286-
Shared::SanitizerGuard.super.checks(expr, branch)
287-
}
288-
}
289-
290277
/**
291278
* An additional step that is preserves dataflow in the context of reflected XSS.
292279
*/
@@ -335,19 +322,6 @@ module StoredXSS {
335322
/** A sanitizer guard for stored XSS vulnerabilities. */
336323
abstract class SanitizerGuard extends Shared::SanitizerGuard { }
337324

338-
// Consider all arbitrary XSS sinks to be stored XSS sinks
339-
private class AnySink extends Sink instanceof Shared::Sink { }
340-
341-
// Consider all arbitrary XSS sanitizers to be stored XSS sanitizers
342-
private class AnySanitizer extends Sanitizer instanceof Shared::Sanitizer { }
343-
344-
// Consider all arbitrary XSS sanitizer guards to be stored XSS sanitizer guards
345-
private class AnySanitizerGuard extends SanitizerGuard instanceof Shared::SanitizerGuard {
346-
override predicate checks(CfgNode expr, boolean branch) {
347-
Shared::SanitizerGuard.super.checks(expr, branch)
348-
}
349-
}
350-
351325
/**
352326
* An additional step that preserves dataflow in the context of stored XSS.
353327
*/

0 commit comments

Comments
 (0)