Skip to content

Commit 8af1ed0

Browse files
committed
Fix applicationserviceinstance
1 parent 4bb5d64 commit 8af1ed0

File tree

2 files changed

+8
-5
lines changed

2 files changed

+8
-5
lines changed

javascript/frameworks/cap/lib/advanced_security/javascript/frameworks/cap/CDS.qll

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -120,7 +120,9 @@ abstract class ServiceInstance extends SourceNode {
120120
* ```
121121
*/
122122
class ServiceInstanceFromCdsServe extends ServiceInstance {
123-
ServiceInstanceFromCdsServe() { this = cdsServeCall() }
123+
string serviceName;
124+
125+
ServiceInstanceFromCdsServe() { this = cdsServeCall().getAPropertyRead(serviceName) }
124126

125127
override UserDefinedApplicationService getDefinition() {
126128
none() // TODO: how should we deal with serve("all")?
Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,11 @@
11
| applicationserviceinstance.js:2:11:2:60 | new cds ... ptions) |
2+
| applicationserviceinstance.js:3:12:3:67 | await n ... ptions) |
23
| applicationserviceinstance.js:3:18:3:67 | new cds ... ptions) |
34
| applicationserviceinstance.js:4:14:4:43 | cds.con ... rvice") |
45
| applicationserviceinstance.js:5:14:5:49 | await c ... rvice") |
56
| applicationserviceinstance.js:5:20:5:49 | cds.con ... rvice") |
6-
| applicationserviceinstance.js:6:18:6:42 | cds.ser ... rvice") |
7-
| applicationserviceinstance.js:7:24:7:48 | cds.ser ... rvice") |
8-
| applicationserviceinstance.js:10:18:10:45 | cdslib. ... rvice") |
7+
| applicationserviceinstance.js:6:9:6:12 | svc5 |
8+
| applicationserviceinstance.js:7:9:7:12 | svc4 |
9+
| applicationserviceinstance.js:10:9:10:12 | svc6 |
910
| applicationserviceinstance.js:12:22:12:48 | cds.con ... ice-2") |
10-
| applicationserviceinstance.js:16:5:16:8 | this |
11+
| applicationserviceinstance.js:15:7:15:6 | this |

0 commit comments

Comments
 (0)