Skip to content

Commit b98ddc7

Browse files
authored
Update DangerousUseOfExceptionBlocks.ql
1 parent 6dec118 commit b98ddc7

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

cpp/ql/src/experimental/Security/CWE/CWE-476/DangerousUseOfExceptionBlocks.ql

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -116,8 +116,8 @@ predicate pointerDereference(CatchAnyBlock cb, Variable vr, Variable vro) {
116116
pragma[inline]
117117
predicate newThrowDelete(CatchAnyBlock cb, Variable vro) {
118118
exists(Expr e0, AssignExpr ase, NewOrNewArrayExpr nae |
119-
ase = vro.getAnAccess().getEnclosingStmt().(ExprStmt).getExpr().(AssignExpr) and
120-
nae = ase.getRValue().(NewOrNewArrayExpr) and
119+
ase = vro.getAnAccess().getEnclosingStmt().(ExprStmt).getExpr() and
120+
nae = ase.getRValue() and
121121
not nae.getAChild*().toString() = "nothrow" and
122122
(
123123
e0 = nae or
@@ -185,8 +185,8 @@ where
185185
exp.getEnclosingStmt().getParentStmt*() = cb and
186186
exists(VariableAccess va |
187187
(
188-
va = exp.(DeleteArrayExpr).getExpr().(VariableAccess) or
189-
va = exp.(DeleteExpr).getExpr().(VariableAccess)
188+
va = exp.(DeleteArrayExpr).getExpr() or
189+
va = exp.(DeleteExpr).getExpr()
190190
) and
191191
va.getEnclosingStmt() = exp.getEnclosingStmt() and
192192
vro = va.getTarget()

0 commit comments

Comments
 (0)