Merge branch 'standard-lib-pt-22' into from-331-to-337

Author: gagliardetto

ql/src/semmle/go/frameworks/Stdlib.qll

@@ -35,6 +35,8 @@ import semmle.go.frameworks.stdlib.PathFilepath
 import semmle.go.frameworks.stdlib.Reflect
 import semmle.go.frameworks.stdlib.Strconv
 import semmle.go.frameworks.stdlib.Strings
+import semmle.go.frameworks.stdlib.Sync
+import semmle.go.frameworks.stdlib.SyncAtomic
 import semmle.go.frameworks.stdlib.TextScanner
 import semmle.go.frameworks.stdlib.TextTabwriter
 import semmle.go.frameworks.stdlib.TextTemplate

ql/src/semmle/go/frameworks/stdlib/Sync.qll

@@ -0,0 +1,44 @@
+/**
+ * Provides classes modeling security-relevant aspects of the `sync` package.
+ */
+
+import go
+
+/** Provides models of commonly used functions in the `sync` package. */
+module Sync {
+  private class MethodModels extends TaintTracking::FunctionModel, Method {
+    FunctionInput inp;
+    FunctionOutput outp;
+
+    MethodModels() {
+      // signature: func (*Map).Load(key interface{}) (value interface{}, ok bool)
+      this.hasQualifiedName("sync", "Map", "Load") and
+      (inp.isReceiver() and outp.isResult(0))
+      or
+      // signature: func (*Map).LoadOrStore(key interface{}, value interface{}) (actual interface{}, loaded bool)
+      this.hasQualifiedName("sync", "Map", "LoadOrStore") and
+      (
+        inp.isReceiver() and outp.isResult(0)
+        or
+        inp.isParameter(_) and
+        (outp.isReceiver() or outp.isResult(0))
+      )
+      or
+      // signature: func (*Map).Store(key interface{}, value interface{})
+      this.hasQualifiedName("sync", "Map", "Store") and
+      (inp.isParameter(_) and outp.isReceiver())
+      or
+      // signature: func (*Pool).Get() interface{}
+      this.hasQualifiedName("sync", "Pool", "Get") and
+      (inp.isReceiver() and outp.isResult())
+      or
+      // signature: func (*Pool).Put(x interface{})
+      this.hasQualifiedName("sync", "Pool", "Put") and
+      (inp.isParameter(0) and outp.isReceiver())
+    }
+
+    override predicate hasTaintFlow(FunctionInput input, FunctionOutput output) {
+      input = inp and output = outp
+    }
+  }
+}

ql/src/semmle/go/frameworks/stdlib/SyncAtomic.qll

@@ -0,0 +1,85 @@
+/**
+ * Provides classes modeling security-relevant aspects of the `sync/atomic` package.
+ */
+
+import go
+
+/** Provides models of commonly used functions in the `sync/atomic` package. */
+module SyncAtomic {
+  private class FunctionModels extends TaintTracking::FunctionModel {
+    FunctionInput inp;
+    FunctionOutput outp;
+
+    FunctionModels() {
+      // signature: func AddUintptr(addr *uintptr, delta uintptr) (new uintptr)
+      hasQualifiedName("sync/atomic", "AddUintptr") and
+      (
+        inp.isParameter(1) and
+        (outp.isParameter(0) or outp.isResult())
+      )
+      or
+      // signature: func CompareAndSwapPointer(addr *unsafe.Pointer, old unsafe.Pointer, new unsafe.Pointer) (swapped bool)
+      hasQualifiedName("sync/atomic", "CompareAndSwapPointer") and
+      (inp.isParameter(2) and outp.isParameter(0))
+      or
+      // signature: func CompareAndSwapUintptr(addr *uintptr, old uintptr, new uintptr) (swapped bool)
+      hasQualifiedName("sync/atomic", "CompareAndSwapUintptr") and
+      (inp.isParameter(2) and outp.isParameter(0))
+      or
+      // signature: func LoadPointer(addr *unsafe.Pointer) (val unsafe.Pointer)
+      hasQualifiedName("sync/atomic", "LoadPointer") and
+      (inp.isParameter(0) and outp.isResult())
+      or
+      // signature: func LoadUintptr(addr *uintptr) (val uintptr)
+      hasQualifiedName("sync/atomic", "LoadUintptr") and
+      (inp.isParameter(0) and outp.isResult())
+      or
+      // signature: func StorePointer(addr *unsafe.Pointer, val unsafe.Pointer)
+      hasQualifiedName("sync/atomic", "StorePointer") and
+      (inp.isParameter(1) and outp.isParameter(0))
+      or
+      // signature: func StoreUintptr(addr *uintptr, val uintptr)
+      hasQualifiedName("sync/atomic", "StoreUintptr") and
+      (inp.isParameter(1) and outp.isParameter(0))
+      or
+      // signature: func SwapPointer(addr *unsafe.Pointer, new unsafe.Pointer) (old unsafe.Pointer)
+      hasQualifiedName("sync/atomic", "SwapPointer") and
+      (
+        inp.isParameter(1) and outp.isParameter(0)
+        or
+        inp.isParameter(0) and outp.isResult()
+      )
+      or
+      // signature: func SwapUintptr(addr *uintptr, new uintptr) (old uintptr)
+      hasQualifiedName("sync/atomic", "SwapUintptr") and
+      (
+        inp.isParameter(1) and outp.isParameter(0)
+        or
+        inp.isParameter(0) and outp.isResult()
+      )
+    }
+
+    override predicate hasTaintFlow(FunctionInput input, FunctionOutput output) {
+      input = inp and output = outp
+    }
+  }
+
+  private class MethodModels extends TaintTracking::FunctionModel, Method {
+    FunctionInput inp;
+    FunctionOutput outp;
+
+    MethodModels() {
+      // signature: func (*Value).Load() (x interface{})
+      this.hasQualifiedName("sync/atomic", "Value", "Load") and
+      (inp.isReceiver() and outp.isResult())
+      or
+      // signature: func (*Value).Store(x interface{})
+      this.hasQualifiedName("sync/atomic", "Value", "Store") and
+      (inp.isParameter(0) and outp.isReceiver())
+    }
+
+    override predicate hasTaintFlow(FunctionInput input, FunctionOutput output) {
+      input = inp and output = outp
+    }
+  }
+}