Skip to content

Commit ec1a8cc

Browse files
apply suggestions from doc review
Co-authored-by: mc <[email protected]>
1 parent bf9bcc9 commit ec1a8cc

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

javascript/ql/src/Security/CWE-367/FileSystemRace.qhelp

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -24,14 +24,14 @@ Use file descriptors instead of file names whenever possible.
2424

2525
<example>
2626
<p>
27-
The following example shows a case where it is checked whether a file inside the <code>/tmp/</code> folder exists,
28-
and if it didn't exist the file is written to.
27+
The following example shows a case where the code checks whether a file inside the <code>/tmp/</code> folder exists,
28+
and if it doesn't, the file is written to that location.
2929
</p>
3030

3131
<sample src="examples/file-race-bad.js" />
3232

3333
<p>
34-
However, in a multi-user environment the file might be created by another user in between the existence check and the write.
34+
However, in a multi-user environment the file might be created by another user between the existence check and the write.
3535
</p>
3636

3737
<p>
@@ -44,7 +44,7 @@ This can be avoided by using <code>fs.open</code> to get a file descriptor, and
4444

4545
<references>
4646
<li>
47-
Wikipedia: <a href="https://en.wikipedia.org/wiki/Time-of-check_to_time-of-use">Time-of-check to time-of-use</a>
47+
Wikipedia: <a href="https://en.wikipedia.org/wiki/Time-of-check_to_time-of-use">Time-of-check to time-of-use.</a>
4848
</li>
4949

5050
<li>
@@ -55,7 +55,7 @@ Wikipedia: <a href="https://en.wikipedia.org/wiki/Time-of-check_to_time-of-use">
5555
</li>
5656
<li>
5757
NodeJS:
58-
<a href="https://nodejs.org/api/fs.html">The FS module</a>
58+
<a href="https://nodejs.org/api/fs.html">The FS module</a>.
5959
</li>
6060
</references>
6161
</qhelp>

0 commit comments

Comments
 (0)