Skip to content

Commit 1dfe9e9

Browse files
committed
changes based on review
1 parent 9a3176d commit 1dfe9e9

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

  • javascript/ql/src/semmle/javascript/frameworks

javascript/ql/src/semmle/javascript/frameworks/Files.qll

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -11,14 +11,14 @@ abstract private class FileNameProducer extends DataFlow::Node {
1111
/**
1212
* Gets a file name produced by this producer.
1313
*/
14-
abstract DataFlow::Node getAFileNameSource();
14+
abstract DataFlow::Node getAFileName();
1515
}
1616

1717
/**
1818
* A node that contains a file name, and is produced by a `ProducesFileNames`.
1919
*/
2020
private class ProducedFileName extends FileNameSource {
21-
ProducedFileName() { this = any(ProducesFileNames producer).getAFileNameSource() }
21+
ProducedFileName() { this = any(FileNameProducer producer).getAFileName() }
2222
}
2323

2424
/**
@@ -176,12 +176,12 @@ private class WriteFileAtomic extends FileSystemWriteAccess, DataFlow::CallNode
176176
/**
177177
* A call to the library `recursive-readdir`.
178178
*/
179-
private class RecursiveReadDir extends FileSystemAccess, ProducesFileNames, DataFlow::CallNode {
179+
private class RecursiveReadDir extends FileSystemAccess, FileNameProducer, DataFlow::CallNode {
180180
RecursiveReadDir() { this = DataFlow::moduleImport("recursive-readdir").getACall() }
181181

182182
override DataFlow::Node getAPathArgument() { result = getArgument(0) }
183183

184-
override DataFlow::Node getAFileNameSource() { result = getCallback([1 .. 2]).getParameter(1) }
184+
override DataFlow::Node getAFileName() { result = getCallback([1 .. 2]).getParameter(1) }
185185
}
186186

187187
/**

0 commit comments

Comments
 (0)