diff --git a/go/ql/lib/semmle/go/frameworks/stdlib/Log.qll b/go/ql/lib/semmle/go/frameworks/stdlib/Log.qll index ad5353fb699..1ff1a4b320f 100644 --- a/go/ql/lib/semmle/go/frameworks/stdlib/Log.qll +++ b/go/ql/lib/semmle/go/frameworks/stdlib/Log.qll @@ -47,6 +47,19 @@ module Log { override predicate mayReturnNormally() { none() } } + /** A log function which must panic. */ + private class PanicLogFunction extends Function { + PanicLogFunction() { + exists(string fn | fn = ["Panic", "Panicf", "Panicln"] | + this.hasQualifiedName("log", fn) + or + this.(Method).hasQualifiedName("log", "Logger", fn) + ) + } + + override predicate mustPanic() { any() } + } + // These models are not implemented using Models-as-Data because they represent reverse flow. private class FunctionModels extends TaintTracking::FunctionModel { FunctionInput inp;