Skip to content

Commit 62c6bee

Browse files
Simplified UnsafeDeserializationRmi.ql
1 parent 1b51dd4 commit 62c6bee

File tree

1 file changed

+1
-5
lines changed

1 file changed

+1
-5
lines changed

java/ql/src/experimental/Security/CWE/CWE-502/UnsafeDeserializationRmi.ql

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -66,11 +66,7 @@ private class BindingUnsafeRemoteObjectConfig extends TaintTracking::Configurati
6666
exists(MethodAccess ma, Method m | m = ma.getMethod() |
6767
m.getDeclaringType().hasQualifiedName("java.rmi.server", "UnicastRemoteObject") and
6868
m.hasName("exportObject") and
69-
not ma.getArgument([2, 4])
70-
.getType()
71-
.(RefType)
72-
.getASupertype*()
73-
.hasQualifiedName("java.io", "ObjectInputFilter") and
69+
not m.getParameterType([2, 4]).(RefType).hasQualifiedName("java.io", "ObjectInputFilter") and
7470
ma.getArgument(0) = fromNode.asExpr() and
7571
ma = toNode.asExpr()
7672
)

0 commit comments

Comments
 (0)