Skip to content

Commit e6e8e3d

Browse files
committed
Taint doesn't flow through panicking functions
1 parent adc9b77 commit e6e8e3d

2 files changed

Lines changed: 0 additions & 120 deletions

File tree

  • go/ql
    • lib/semmle/go/frameworks/stdlib
    • test/library-tests/semmle/go/frameworks/StdlibTaintFlow

go/ql/lib/semmle/go/frameworks/stdlib/Log.qll

Lines changed: 0 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -69,30 +69,6 @@ module Log {
6969
FunctionOutput outp;
7070

7171
MethodModels() {
72-
// signature: func (*Logger) Fatal(v ...interface{})
73-
this.hasQualifiedName("log", "Logger", "Fatal") and
74-
(inp.isParameter(_) and outp.isReceiver())
75-
or
76-
// signature: func (*Logger) Fatalf(format string, v ...interface{})
77-
this.hasQualifiedName("log", "Logger", "Fatalf") and
78-
(inp.isParameter(_) and outp.isReceiver())
79-
or
80-
// signature: func (*Logger) Fatalln(v ...interface{})
81-
this.hasQualifiedName("log", "Logger", "Fatalln") and
82-
(inp.isParameter(_) and outp.isReceiver())
83-
or
84-
// signature: func (*Logger) Panic(v ...interface{})
85-
this.hasQualifiedName("log", "Logger", "Panic") and
86-
(inp.isParameter(_) and outp.isReceiver())
87-
or
88-
// signature: func (*Logger) Panicf(format string, v ...interface{})
89-
this.hasQualifiedName("log", "Logger", "Panicf") and
90-
(inp.isParameter(_) and outp.isReceiver())
91-
or
92-
// signature: func (*Logger) Panicln(v ...interface{})
93-
this.hasQualifiedName("log", "Logger", "Panicln") and
94-
(inp.isParameter(_) and outp.isReceiver())
95-
or
9672
// signature: func (*Logger) Print(v ...interface{})
9773
this.hasQualifiedName("log", "Logger", "Print") and
9874
(inp.isParameter(_) and outp.isReceiver())

go/ql/test/library-tests/semmle/go/frameworks/StdlibTaintFlow/Log.go

Lines changed: 0 additions & 96 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)