Skip to content

Commit 460ed30

Browse files
committed
Fixed tests
1 parent eb3a76d commit 460ed30

File tree

2 files changed

+32
-4
lines changed

2 files changed

+32
-4
lines changed

go/ql/test/query-tests/Security/CWE-022/GorillaMuxSkipClean/TaintedPath.expected

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,13 @@
1+
#select
2+
| MuxClean.go:14:29:14:74 | call to Join | MuxClean.go:13:22:13:32 | call to Vars | MuxClean.go:14:29:14:74 | call to Join | This path depends on a $@. | MuxClean.go:13:22:13:32 | call to Vars | user-provided value |
13
edges
2-
| MuxClean.go:13:22:13:32 | call to Vars | MuxClean.go:14:58:14:73 | not_tainted_path | provenance | Src:MaD:524 |
3-
| MuxClean.go:14:58:14:73 | not_tainted_path | MuxClean.go:14:29:14:74 | call to Join | provenance | FunctionModel Sink:MaD:854 |
4+
| MuxClean.go:13:22:13:32 | call to Vars | MuxClean.go:14:58:14:73 | not_tainted_path | provenance | Src:MaD:2 |
5+
| MuxClean.go:14:58:14:73 | not_tainted_path | MuxClean.go:14:29:14:74 | call to Join | provenance | FunctionModel Sink:MaD:1 |
6+
models
7+
| 1 | Sink: io/ioutil; ; false; ReadFile; ; ; Argument[0]; path-injection; manual |
8+
| 2 | Source: github.com/gorilla/mux; ; true; Vars; ; ; ReturnValue; remote; manual |
49
nodes
510
| MuxClean.go:13:22:13:32 | call to Vars | semmle.label | call to Vars |
611
| MuxClean.go:14:29:14:74 | call to Join | semmle.label | call to Join |
712
| MuxClean.go:14:58:14:73 | not_tainted_path | semmle.label | not_tainted_path |
813
subpaths
9-
#select
10-
| MuxClean.go:14:29:14:74 | call to Join | MuxClean.go:13:22:13:32 | call to Vars | MuxClean.go:14:29:14:74 | call to Join | This path depends on a $@. | MuxClean.go:13:22:13:32 | call to Vars | user-provided value |
Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,25 @@
1+
#select
2+
| TaintedPath.go:17:29:17:40 | tainted_path | TaintedPath.go:14:18:14:22 | selection of URL | TaintedPath.go:17:29:17:40 | tainted_path | This path depends on a $@. | TaintedPath.go:14:18:14:22 | selection of URL | user-provided value |
3+
| TaintedPath.go:21:28:21:69 | call to Join | TaintedPath.go:14:18:14:22 | selection of URL | TaintedPath.go:21:28:21:69 | call to Join | This path depends on a $@. | TaintedPath.go:14:18:14:22 | selection of URL | user-provided value |
4+
| TaintedPath.go:68:28:68:57 | call to Clean | TaintedPath.go:14:18:14:22 | selection of URL | TaintedPath.go:68:28:68:57 | call to Clean | This path depends on a $@. | TaintedPath.go:14:18:14:22 | selection of URL | user-provided value |
5+
edges
6+
| TaintedPath.go:14:18:14:22 | selection of URL | TaintedPath.go:14:18:14:30 | call to Query | provenance | Src:MaD:2 MaD:3 |
7+
| TaintedPath.go:14:18:14:30 | call to Query | TaintedPath.go:17:29:17:40 | tainted_path | provenance | Sink:MaD:1 |
8+
| TaintedPath.go:14:18:14:30 | call to Query | TaintedPath.go:21:57:21:68 | tainted_path | provenance | |
9+
| TaintedPath.go:14:18:14:30 | call to Query | TaintedPath.go:68:39:68:56 | ...+... | provenance | |
10+
| TaintedPath.go:21:57:21:68 | tainted_path | TaintedPath.go:21:28:21:69 | call to Join | provenance | FunctionModel Sink:MaD:1 |
11+
| TaintedPath.go:68:39:68:56 | ...+... | TaintedPath.go:68:28:68:57 | call to Clean | provenance | MaD:4 Sink:MaD:1 |
12+
models
13+
| 1 | Sink: io/ioutil; ; false; ReadFile; ; ; Argument[0]; path-injection; manual |
14+
| 2 | Source: net/http; Request; true; URL; ; ; ; remote; manual |
15+
| 3 | Summary: net/url; URL; true; Query; ; ; Argument[receiver]; ReturnValue; taint; manual |
16+
| 4 | Summary: path; ; false; Clean; ; ; Argument[0]; ReturnValue; taint; manual |
17+
nodes
18+
| TaintedPath.go:14:18:14:22 | selection of URL | semmle.label | selection of URL |
19+
| TaintedPath.go:14:18:14:30 | call to Query | semmle.label | call to Query |
20+
| TaintedPath.go:17:29:17:40 | tainted_path | semmle.label | tainted_path |
21+
| TaintedPath.go:21:28:21:69 | call to Join | semmle.label | call to Join |
22+
| TaintedPath.go:21:57:21:68 | tainted_path | semmle.label | tainted_path |
23+
| TaintedPath.go:68:28:68:57 | call to Clean | semmle.label | call to Clean |
24+
| TaintedPath.go:68:39:68:56 | ...+... | semmle.label | ...+... |
25+
subpaths

0 commit comments

Comments
 (0)