Skip to content

Commit 143e168

Browse files
Apply suggestions from code review
Co-authored-by: Ben Ahmady <[email protected]>
1 parent 947b094 commit 143e168

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

java/ql/src/Security/CWE/CWE-022/TaintedPath.qhelp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,12 +41,12 @@ such as "/etc/passwd".</p>
4141
<sample src="TaintedPath.java" />
4242

4343
<p>Simply checking that the path is under a trusted location (such as a known public folder) is not enough,
44-
however, since the path could contain relative components such as "..". To fix this, we check that the it does
44+
however, since the path could contain relative components such as "..". To fix this, check that it does
4545
not contain ".." and starts with the public folder.</p>
4646

4747
<sample src="TaintedPathGood.java" />
4848

49-
<p>Alternatively, if we only want to allow simple filenames without a path component, we can remove all path
49+
<p>Alternatively, if you only want to allow simple filenames without a path component, you can remove all path
5050
separators ("/" or "\") and all ".." sequences from the input before using it to construct a file path.</p>
5151

5252
<sample src="TaintedPathGood2.java" />

0 commit comments

Comments
 (0)