2017-07-19 22:17:00 +01:00
|
|
|
package testutils
|
|
|
|
|
2020-04-01 21:18:39 +01:00
|
|
|
import "github.com/securego/gosec/v2"
|
2019-06-25 09:29:19 +01:00
|
|
|
|
2017-07-19 22:17:00 +01:00
|
|
|
// CodeSample encapsulates a snippet of source code that compiles, and how many errors should be detected
|
|
|
|
type CodeSample struct {
|
2018-09-28 09:42:25 +01:00
|
|
|
Code []string
|
|
|
|
Errors int
|
2019-06-25 09:29:19 +01:00
|
|
|
Config gosec.Config
|
2017-07-19 22:17:00 +01:00
|
|
|
}
|
|
|
|
|
|
|
|
var (
|
|
|
|
// SampleCodeG101 code snippets for hardcoded credentials
|
2018-09-28 09:42:25 +01:00
|
|
|
SampleCodeG101 = []CodeSample{{[]string{`
|
2017-07-19 22:17:00 +01:00
|
|
|
package main
|
|
|
|
import "fmt"
|
|
|
|
func main() {
|
|
|
|
username := "admin"
|
|
|
|
password := "f62e5bcda4fae4f82370da0c6f20697b8f8447ef"
|
|
|
|
fmt.Println("Doing something with: ", username, password)
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
2017-07-19 22:17:00 +01:00
|
|
|
// Entropy check should not report this error by default
|
|
|
|
package main
|
|
|
|
import "fmt"
|
|
|
|
func main() {
|
|
|
|
username := "admin"
|
|
|
|
password := "secret"
|
|
|
|
fmt.Println("Doing something with: ", username, password)
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 0, gosec.NewConfig()}, {[]string{`
|
2017-07-19 22:17:00 +01:00
|
|
|
package main
|
|
|
|
import "fmt"
|
|
|
|
var password = "f62e5bcda4fae4f82370da0c6f20697b8f8447ef"
|
|
|
|
func main() {
|
|
|
|
username := "admin"
|
|
|
|
fmt.Println("Doing something with: ", username, password)
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
2017-07-19 22:17:00 +01:00
|
|
|
package main
|
|
|
|
import "fmt"
|
|
|
|
const password = "f62e5bcda4fae4f82370da0c6f20697b8f8447ef"
|
|
|
|
func main() {
|
|
|
|
username := "admin"
|
|
|
|
fmt.Println("Doing something with: ", username, password)
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
2017-07-19 22:17:00 +01:00
|
|
|
package main
|
|
|
|
import "fmt"
|
|
|
|
const (
|
|
|
|
username = "user"
|
|
|
|
password = "f62e5bcda4fae4f82370da0c6f20697b8f8447ef"
|
|
|
|
)
|
|
|
|
func main() {
|
|
|
|
fmt.Println("Doing something with: ", username, password)
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
2017-07-19 22:17:00 +01:00
|
|
|
package main
|
|
|
|
var password string
|
|
|
|
func init() {
|
|
|
|
password = "f62e5bcda4fae4f82370da0c6f20697b8f8447ef"
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
2017-07-19 22:17:00 +01:00
|
|
|
package main
|
|
|
|
const (
|
|
|
|
ATNStateSomethingElse = 1
|
|
|
|
ATNStateTokenStart = 42
|
|
|
|
)
|
|
|
|
func main() {
|
|
|
|
println(ATNStateTokenStart)
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 0, gosec.NewConfig()}, {[]string{`
|
2017-07-19 22:17:00 +01:00
|
|
|
package main
|
|
|
|
const (
|
|
|
|
ATNStateTokenStart = "f62e5bcda4fae4f82370da0c6f20697b8f8447ef"
|
|
|
|
)
|
|
|
|
func main() {
|
|
|
|
println(ATNStateTokenStart)
|
2020-04-17 14:47:27 +01:00
|
|
|
}`}, 1, gosec.NewConfig()},
|
|
|
|
{[]string{`
|
|
|
|
package main
|
|
|
|
import "fmt"
|
|
|
|
func main() {
|
|
|
|
var password string
|
|
|
|
if password == "f62e5bcda4fae4f82370da0c6f20697b8f8447ef" {
|
2020-06-29 12:21:15 +01:00
|
|
|
fmt.Println("password equality")
|
2020-04-17 14:47:27 +01:00
|
|
|
}
|
|
|
|
}`}, 1, gosec.NewConfig()},
|
|
|
|
{[]string{`
|
|
|
|
package main
|
|
|
|
import "fmt"
|
|
|
|
func main() {
|
|
|
|
var password string
|
|
|
|
if password != "f62e5bcda4fae4f82370da0c6f20697b8f8447ef" {
|
2020-06-29 12:21:15 +01:00
|
|
|
fmt.Println("password equality")
|
2020-04-17 14:47:27 +01:00
|
|
|
}
|
|
|
|
}`}, 1, gosec.NewConfig()},
|
|
|
|
{[]string{`
|
|
|
|
package main
|
|
|
|
import "fmt"
|
|
|
|
func main() {
|
|
|
|
var p string
|
|
|
|
if p != "f62e5bcda4fae4f82370da0c6f20697b8f8447ef" {
|
2020-06-29 12:21:15 +01:00
|
|
|
fmt.Println("password equality")
|
2020-04-17 14:47:27 +01:00
|
|
|
}
|
|
|
|
}`}, 0, gosec.NewConfig()}}
|
2017-07-19 22:17:00 +01:00
|
|
|
|
|
|
|
// SampleCodeG102 code snippets for network binding
|
|
|
|
SampleCodeG102 = []CodeSample{
|
|
|
|
// Bind to all networks explicitly
|
2018-09-28 09:42:25 +01:00
|
|
|
{[]string{`
|
2017-07-19 22:17:00 +01:00
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"log"
|
|
|
|
"net"
|
|
|
|
)
|
|
|
|
func main() {
|
|
|
|
l, err := net.Listen("tcp", "0.0.0.0:2000")
|
|
|
|
if err != nil {
|
|
|
|
log.Fatal(err)
|
|
|
|
}
|
|
|
|
defer l.Close()
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()},
|
2017-07-19 22:17:00 +01:00
|
|
|
|
|
|
|
// Bind to all networks implicitly (default if host omitted)
|
2018-09-28 09:42:25 +01:00
|
|
|
{[]string{`
|
2017-07-19 22:17:00 +01:00
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"log"
|
|
|
|
"net"
|
|
|
|
)
|
|
|
|
func main() {
|
|
|
|
l, err := net.Listen("tcp", ":2000")
|
|
|
|
if err != nil {
|
|
|
|
log.Fatal(err)
|
|
|
|
}
|
|
|
|
defer l.Close()
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()},
|
2018-12-02 14:37:42 +00:00
|
|
|
// Bind to all networks indirectly through a parsing function
|
|
|
|
{[]string{`
|
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"log"
|
|
|
|
"net"
|
|
|
|
)
|
|
|
|
func parseListenAddr(listenAddr string) (network string, addr string) {
|
|
|
|
return "", ""
|
|
|
|
}
|
|
|
|
func main() {
|
|
|
|
addr := ":2000"
|
|
|
|
l, err := net.Listen(parseListenAddr(addr))
|
|
|
|
if err != nil {
|
|
|
|
log.Fatal(err)
|
|
|
|
}
|
|
|
|
defer l.Close()
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()},
|
2018-12-02 14:37:42 +00:00
|
|
|
// Bind to all networks indirectly through a parsing function
|
|
|
|
{[]string{`
|
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"log"
|
|
|
|
"net"
|
|
|
|
)
|
|
|
|
const addr = ":2000"
|
|
|
|
func parseListenAddr(listenAddr string) (network string, addr string) {
|
|
|
|
return "", ""
|
|
|
|
}
|
|
|
|
func main() {
|
|
|
|
l, err := net.Listen(parseListenAddr(addr))
|
|
|
|
if err != nil {
|
|
|
|
log.Fatal(err)
|
|
|
|
}
|
|
|
|
defer l.Close()
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()},
|
2018-12-02 15:28:51 +00:00
|
|
|
{[]string{`
|
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"log"
|
|
|
|
"net"
|
|
|
|
)
|
|
|
|
const addr = "0.0.0.0:2000"
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
l, err := net.Listen("tcp", addr)
|
|
|
|
if err != nil {
|
|
|
|
log.Fatal(err)
|
|
|
|
}
|
|
|
|
defer l.Close()
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()},
|
2017-07-19 22:17:00 +01:00
|
|
|
}
|
|
|
|
// SampleCodeG103 find instances of unsafe blocks for auditing purposes
|
|
|
|
SampleCodeG103 = []CodeSample{
|
2018-09-28 09:42:25 +01:00
|
|
|
{[]string{`
|
2017-07-19 22:17:00 +01:00
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"fmt"
|
|
|
|
"unsafe"
|
|
|
|
)
|
|
|
|
type Fake struct{}
|
|
|
|
func (Fake) Good() {}
|
|
|
|
func main() {
|
|
|
|
unsafeM := Fake{}
|
|
|
|
unsafeM.Good()
|
|
|
|
intArray := [...]int{1, 2}
|
|
|
|
fmt.Printf("\nintArray: %v\n", intArray)
|
|
|
|
intPtr := &intArray[0]
|
|
|
|
fmt.Printf("\nintPtr=%p, *intPtr=%d.\n", intPtr, *intPtr)
|
|
|
|
addressHolder := uintptr(unsafe.Pointer(intPtr)) + unsafe.Sizeof(intArray[0])
|
|
|
|
intPtr = (*int)(unsafe.Pointer(addressHolder))
|
|
|
|
fmt.Printf("\nintPtr=%p, *intPtr=%d.\n\n", intPtr, *intPtr)
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 3, gosec.NewConfig()}}
|
2017-07-19 22:17:00 +01:00
|
|
|
|
|
|
|
// SampleCodeG104 finds errors that aren't being handled
|
|
|
|
SampleCodeG104 = []CodeSample{
|
2018-09-28 09:42:25 +01:00
|
|
|
{[]string{`
|
2017-07-19 22:17:00 +01:00
|
|
|
package main
|
|
|
|
import "fmt"
|
|
|
|
func test() (int,error) {
|
|
|
|
return 0, nil
|
|
|
|
}
|
|
|
|
func main() {
|
|
|
|
v, _ := test()
|
|
|
|
fmt.Println(v)
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 0, gosec.NewConfig()}, {[]string{`
|
2017-07-19 22:17:00 +01:00
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"io/ioutil"
|
|
|
|
"os"
|
|
|
|
"fmt"
|
|
|
|
)
|
|
|
|
func a() error {
|
|
|
|
return fmt.Errorf("This is an error")
|
|
|
|
}
|
|
|
|
func b() {
|
|
|
|
fmt.Println("b")
|
|
|
|
ioutil.WriteFile("foo.txt", []byte("bar"), os.ModeExclusive)
|
|
|
|
}
|
|
|
|
func c() string {
|
|
|
|
return fmt.Sprintf("This isn't anything")
|
|
|
|
}
|
|
|
|
func main() {
|
|
|
|
_ = a()
|
|
|
|
a()
|
|
|
|
b()
|
|
|
|
c()
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 2, gosec.NewConfig()}, {[]string{`
|
2017-07-19 22:17:00 +01:00
|
|
|
package main
|
|
|
|
import "fmt"
|
|
|
|
func test() error {
|
|
|
|
return nil
|
|
|
|
}
|
|
|
|
func main() {
|
|
|
|
e := test()
|
|
|
|
fmt.Println(e)
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 0, gosec.NewConfig()}, {[]string{`
|
2018-11-11 08:41:40 +00:00
|
|
|
// +build go1.10
|
|
|
|
|
2018-10-18 00:13:56 +01:00
|
|
|
package main
|
|
|
|
import "strings"
|
|
|
|
func main() {
|
|
|
|
var buf strings.Builder
|
2018-11-10 15:39:44 +00:00
|
|
|
_, err := buf.WriteString("test string")
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
2018-11-11 08:41:40 +00:00
|
|
|
}`, `
|
|
|
|
package main
|
|
|
|
func dummy(){}
|
2019-06-25 10:15:11 +01:00
|
|
|
`}, 0, gosec.NewConfig()}, {[]string{`
|
|
|
|
package main
|
2020-03-15 14:42:26 +00:00
|
|
|
|
|
|
|
import (
|
|
|
|
"bytes"
|
|
|
|
)
|
|
|
|
|
|
|
|
type a struct {
|
|
|
|
buf *bytes.Buffer
|
|
|
|
}
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
a := &a{
|
|
|
|
buf: new(bytes.Buffer),
|
|
|
|
}
|
|
|
|
a.buf.Write([]byte{0})
|
|
|
|
}
|
|
|
|
`}, 0, gosec.NewConfig()}, {[]string{`
|
|
|
|
package main
|
2019-06-25 10:15:11 +01:00
|
|
|
import (
|
|
|
|
"io/ioutil"
|
|
|
|
"os"
|
|
|
|
"fmt"
|
|
|
|
)
|
|
|
|
func a() {
|
|
|
|
fmt.Println("a")
|
|
|
|
ioutil.WriteFile("foo.txt", []byte("bar"), os.ModeExclusive)
|
|
|
|
}
|
|
|
|
func main() {
|
|
|
|
a()
|
2020-01-28 13:11:00 +00:00
|
|
|
}`}, 0, gosec.Config{"G104": map[string]interface{}{"ioutil": []interface{}{"WriteFile"}}}}, {[]string{`
|
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"bytes"
|
|
|
|
"fmt"
|
|
|
|
"io"
|
|
|
|
"os"
|
|
|
|
"strings"
|
|
|
|
)
|
|
|
|
|
|
|
|
func createBuffer() *bytes.Buffer {
|
|
|
|
return new(bytes.Buffer)
|
|
|
|
}
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
new(bytes.Buffer).WriteString("*bytes.Buffer")
|
|
|
|
fmt.Fprintln(os.Stderr, "fmt")
|
|
|
|
new(strings.Builder).WriteString("*strings.Builder")
|
|
|
|
_, pw := io.Pipe()
|
|
|
|
pw.CloseWithError(io.EOF)
|
|
|
|
|
|
|
|
createBuffer().WriteString("*bytes.Buffer")
|
|
|
|
b := createBuffer()
|
|
|
|
b.WriteString("*bytes.Buffer")
|
|
|
|
}`}, 0, gosec.NewConfig()}} // it shoudn't return any errors because all method calls are whitelisted by default
|
2017-07-19 22:17:00 +01:00
|
|
|
|
2019-01-14 11:37:40 +00:00
|
|
|
// SampleCodeG104Audit finds errors that aren't being handled in audit mode
|
|
|
|
SampleCodeG104Audit = []CodeSample{
|
|
|
|
{[]string{`
|
|
|
|
package main
|
|
|
|
import "fmt"
|
|
|
|
func test() (int,error) {
|
|
|
|
return 0, nil
|
|
|
|
}
|
|
|
|
func main() {
|
|
|
|
v, _ := test()
|
|
|
|
fmt.Println(v)
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.Config{gosec.Globals: map[gosec.GlobalOption]string{gosec.Audit: "enabled"}}}, {[]string{`
|
2019-01-14 11:37:40 +00:00
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"io/ioutil"
|
|
|
|
"os"
|
|
|
|
"fmt"
|
|
|
|
)
|
|
|
|
func a() error {
|
|
|
|
return fmt.Errorf("This is an error")
|
|
|
|
}
|
|
|
|
func b() {
|
|
|
|
fmt.Println("b")
|
|
|
|
ioutil.WriteFile("foo.txt", []byte("bar"), os.ModeExclusive)
|
|
|
|
}
|
|
|
|
func c() string {
|
|
|
|
return fmt.Sprintf("This isn't anything")
|
|
|
|
}
|
|
|
|
func main() {
|
|
|
|
_ = a()
|
|
|
|
a()
|
|
|
|
b()
|
|
|
|
c()
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 3, gosec.Config{gosec.Globals: map[gosec.GlobalOption]string{gosec.Audit: "enabled"}}}, {[]string{`
|
2019-01-14 11:37:40 +00:00
|
|
|
package main
|
|
|
|
import "fmt"
|
|
|
|
func test() error {
|
|
|
|
return nil
|
|
|
|
}
|
|
|
|
func main() {
|
|
|
|
e := test()
|
|
|
|
fmt.Println(e)
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 0, gosec.Config{gosec.Globals: map[gosec.GlobalOption]string{gosec.Audit: "enabled"}}}, {[]string{`
|
2019-01-14 11:37:40 +00:00
|
|
|
// +build go1.10
|
|
|
|
|
|
|
|
package main
|
|
|
|
import "strings"
|
|
|
|
func main() {
|
|
|
|
var buf strings.Builder
|
|
|
|
_, err := buf.WriteString("test string")
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
}`, `
|
|
|
|
package main
|
|
|
|
func dummy(){}
|
2019-06-25 09:29:19 +01:00
|
|
|
`}, 0, gosec.Config{gosec.Globals: map[gosec.GlobalOption]string{gosec.Audit: "enabled"}}}}
|
2017-12-28 06:54:10 +00:00
|
|
|
|
2018-02-06 15:56:26 +00:00
|
|
|
// SampleCodeG106 - ssh InsecureIgnoreHostKey
|
2018-09-28 09:42:25 +01:00
|
|
|
SampleCodeG106 = []CodeSample{{[]string{`
|
2018-02-06 15:56:26 +00:00
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"golang.org/x/crypto/ssh"
|
|
|
|
)
|
|
|
|
func main() {
|
2018-02-07 08:23:52 +00:00
|
|
|
_ = ssh.InsecureIgnoreHostKey()
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}}
|
2018-09-04 07:55:03 +01:00
|
|
|
|
|
|
|
// SampleCodeG107 - SSRF via http requests with variable url
|
2018-09-28 09:42:25 +01:00
|
|
|
SampleCodeG107 = []CodeSample{{[]string{`
|
2019-09-17 11:22:43 +01:00
|
|
|
// Input from the std in is considered insecure
|
2018-09-04 07:55:03 +01:00
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"net/http"
|
|
|
|
"io/ioutil"
|
|
|
|
"fmt"
|
|
|
|
"os"
|
2019-02-13 08:52:12 +00:00
|
|
|
"bufio"
|
2018-09-04 07:55:03 +01:00
|
|
|
)
|
|
|
|
func main() {
|
2019-02-13 08:52:12 +00:00
|
|
|
in := bufio.NewReader(os.Stdin)
|
|
|
|
url, err := in.ReadString('\n')
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
2018-09-04 07:55:03 +01:00
|
|
|
resp, err := http.Get(url)
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
defer resp.Body.Close()
|
|
|
|
body, err := ioutil.ReadAll(resp.Body)
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
fmt.Printf("%s", body)
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
2019-10-04 12:25:38 +01:00
|
|
|
// Variable defined a package level can be changed at any time
|
|
|
|
// regardless of the initial value
|
2019-09-17 11:22:43 +01:00
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"fmt"
|
|
|
|
"io/ioutil"
|
|
|
|
"net/http"
|
|
|
|
)
|
|
|
|
|
|
|
|
var url string = "https://www.google.com"
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
resp, err := http.Get(url)
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
defer resp.Body.Close()
|
|
|
|
body, err := ioutil.ReadAll(resp.Body)
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
fmt.Printf("%s", body)
|
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
|
|
|
// Environmental variables are not considered as secure source
|
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"net/http"
|
|
|
|
"io/ioutil"
|
|
|
|
"fmt"
|
|
|
|
"os"
|
|
|
|
)
|
|
|
|
func main() {
|
|
|
|
url := os.Getenv("tainted_url")
|
|
|
|
resp, err := http.Get(url)
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
defer resp.Body.Close()
|
|
|
|
body, err := ioutil.ReadAll(resp.Body)
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
fmt.Printf("%s", body)
|
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
2019-10-04 12:25:38 +01:00
|
|
|
// Constant variables or hard-coded strings are secure
|
2018-09-04 07:55:03 +01:00
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"fmt"
|
|
|
|
"net/http"
|
|
|
|
)
|
|
|
|
const url = "http://127.0.0.1"
|
|
|
|
func main() {
|
|
|
|
resp, err := http.Get(url)
|
|
|
|
if err != nil {
|
|
|
|
fmt.Println(err)
|
2019-10-04 12:25:38 +01:00
|
|
|
}
|
|
|
|
fmt.Println(resp.Status)
|
|
|
|
}`}, 0, gosec.NewConfig()}, {[]string{`
|
|
|
|
// A variable at function scope which is initialized to
|
|
|
|
// a constant string is secure (e.g. cannot be changed concurrently)
|
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"fmt"
|
|
|
|
"net/http"
|
|
|
|
)
|
|
|
|
func main() {
|
|
|
|
var url string = "http://127.0.0.1"
|
|
|
|
resp, err := http.Get(url)
|
|
|
|
if err != nil {
|
|
|
|
fmt.Println(err)
|
|
|
|
}
|
|
|
|
fmt.Println(resp.Status)
|
|
|
|
}`}, 0, gosec.NewConfig()}, {[]string{`
|
|
|
|
// A variable at function scope which is initialized to
|
|
|
|
// a constant string is secure (e.g. cannot be changed concurrently)
|
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"fmt"
|
|
|
|
"net/http"
|
|
|
|
)
|
|
|
|
func main() {
|
|
|
|
url := "http://127.0.0.1"
|
|
|
|
resp, err := http.Get(url)
|
|
|
|
if err != nil {
|
|
|
|
fmt.Println(err)
|
|
|
|
}
|
|
|
|
fmt.Println(resp.Status)
|
|
|
|
}`}, 0, gosec.NewConfig()}, {[]string{`
|
|
|
|
// A variable at function scope which is initialized to
|
|
|
|
// a constant string is secure (e.g. cannot be changed concurrently)
|
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"fmt"
|
|
|
|
"net/http"
|
|
|
|
)
|
|
|
|
func main() {
|
|
|
|
url1 := "test"
|
|
|
|
var url2 string = "http://127.0.0.1"
|
|
|
|
url2 = url1
|
|
|
|
resp, err := http.Get(url2)
|
|
|
|
if err != nil {
|
|
|
|
fmt.Println(err)
|
|
|
|
}
|
|
|
|
fmt.Println(resp.Status)
|
|
|
|
}`}, 0, gosec.NewConfig()}, {[]string{`
|
2020-06-29 12:21:15 +01:00
|
|
|
// An exported variable declared a packaged scope is not secure
|
2019-10-04 12:25:38 +01:00
|
|
|
// because it can changed at any time
|
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"fmt"
|
|
|
|
"net/http"
|
|
|
|
)
|
|
|
|
|
|
|
|
var Url string
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
resp, err := http.Get(Url)
|
|
|
|
if err != nil {
|
|
|
|
fmt.Println(err)
|
|
|
|
}
|
|
|
|
fmt.Println(resp.Status)
|
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
|
|
|
// An url provided as a function argument is not secure
|
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"fmt"
|
|
|
|
"net/http"
|
|
|
|
)
|
|
|
|
func get(url string) {
|
|
|
|
resp, err := http.Get(url)
|
|
|
|
if err != nil {
|
|
|
|
fmt.Println(err)
|
|
|
|
}
|
|
|
|
fmt.Println(resp.Status)
|
|
|
|
}
|
|
|
|
func main() {
|
|
|
|
url := "http://127.0.0.1"
|
|
|
|
get(url)
|
|
|
|
}`}, 1, gosec.NewConfig()}}
|
|
|
|
|
2019-09-20 09:46:06 +01:00
|
|
|
// SampleCodeG108 - pprof endpoint automatically exposed
|
|
|
|
SampleCodeG108 = []CodeSample{{[]string{`
|
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"fmt"
|
|
|
|
"log"
|
|
|
|
"net/http"
|
|
|
|
_ "net/http/pprof"
|
|
|
|
)
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
http.HandleFunc("/", func(w http.ResponseWriter, r *http.Request) {
|
|
|
|
fmt.Fprintf(w, "Hello World!")
|
|
|
|
})
|
|
|
|
log.Fatal(http.ListenAndServe(":8080", nil))
|
2019-09-24 09:10:00 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"fmt"
|
|
|
|
"log"
|
|
|
|
"net/http"
|
|
|
|
)
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
http.HandleFunc("/", func(w http.ResponseWriter, r *http.Request) {
|
|
|
|
fmt.Fprintf(w, "Hello World!")
|
|
|
|
})
|
|
|
|
log.Fatal(http.ListenAndServe(":8080", nil))
|
|
|
|
}`}, 0, gosec.NewConfig()}}
|
2020-01-06 08:55:52 +00:00
|
|
|
|
|
|
|
// SampleCodeG109 - Potential Integer OverFlow
|
|
|
|
SampleCodeG109 = []CodeSample{
|
|
|
|
{[]string{`
|
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"fmt"
|
|
|
|
"strconv"
|
|
|
|
)
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
bigValue, err := strconv.Atoi("2147483648")
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
value := int32(bigValue)
|
|
|
|
fmt.Println(value)
|
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"fmt"
|
|
|
|
"strconv"
|
|
|
|
)
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
bigValue, err := strconv.Atoi("32768")
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
if int16(bigValue) < 0 {
|
|
|
|
fmt.Println(bigValue)
|
|
|
|
}
|
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"fmt"
|
|
|
|
"strconv"
|
|
|
|
)
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
bigValue, err := strconv.Atoi("2147483648")
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
fmt.Println(bigValue)
|
|
|
|
}`}, 0, gosec.NewConfig()}, {[]string{`
|
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"fmt"
|
|
|
|
"strconv"
|
|
|
|
)
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
bigValue, err := strconv.Atoi("2147483648")
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
fmt.Println(bigValue)
|
|
|
|
test()
|
|
|
|
}
|
|
|
|
|
|
|
|
func test() {
|
|
|
|
bigValue := 30
|
|
|
|
value := int32(bigValue)
|
|
|
|
fmt.Println(value)
|
2020-01-21 09:13:12 +00:00
|
|
|
}`}, 0, gosec.NewConfig()}, {[]string{`
|
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"fmt"
|
|
|
|
"strconv"
|
|
|
|
)
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
value := 10
|
|
|
|
if value == 10 {
|
|
|
|
value, _ := strconv.Atoi("2147483648")
|
|
|
|
fmt.Println(value)
|
|
|
|
}
|
|
|
|
v := int32(value)
|
|
|
|
fmt.Println(v)
|
2020-01-06 08:55:52 +00:00
|
|
|
}`}, 0, gosec.NewConfig()}}
|
|
|
|
|
2020-01-19 19:40:19 +00:00
|
|
|
// SampleCodeG110 - potential DoS vulnerability via decompression bomb
|
|
|
|
SampleCodeG110 = []CodeSample{
|
|
|
|
{[]string{`
|
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"bytes"
|
|
|
|
"compress/zlib"
|
|
|
|
"io"
|
|
|
|
"os"
|
|
|
|
)
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
buff := []byte{120, 156, 202, 72, 205, 201, 201, 215, 81, 40, 207,
|
|
|
|
47, 202, 73, 225, 2, 4, 0, 0, 255, 255, 33, 231, 4, 147}
|
|
|
|
b := bytes.NewReader(buff)
|
|
|
|
|
|
|
|
r, err := zlib.NewReader(b)
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
2021-01-01 19:45:13 +00:00
|
|
|
_, err = io.Copy(os.Stdout, r)
|
2020-07-27 08:21:51 +01:00
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
|
|
|
|
r.Close()
|
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"bytes"
|
|
|
|
"compress/zlib"
|
|
|
|
"io"
|
|
|
|
"os"
|
|
|
|
)
|
2020-01-19 19:40:19 +00:00
|
|
|
|
2020-07-27 08:21:51 +01:00
|
|
|
func main() {
|
|
|
|
buff := []byte{120, 156, 202, 72, 205, 201, 201, 215, 81, 40, 207,
|
|
|
|
47, 202, 73, 225, 2, 4, 0, 0, 255, 255, 33, 231, 4, 147}
|
|
|
|
b := bytes.NewReader(buff)
|
|
|
|
|
|
|
|
r, err := zlib.NewReader(b)
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
buf := make([]byte, 8)
|
2021-01-01 19:45:13 +00:00
|
|
|
_, err = io.CopyBuffer(os.Stdout, r, buf)
|
2020-07-27 08:21:51 +01:00
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
2020-01-19 19:40:19 +00:00
|
|
|
r.Close()
|
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"archive/zip"
|
|
|
|
"io"
|
|
|
|
"os"
|
|
|
|
"strconv"
|
|
|
|
)
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
r, err := zip.OpenReader("tmp.zip")
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
defer r.Close()
|
|
|
|
|
|
|
|
for i, f := range r.File {
|
|
|
|
out, err := os.OpenFile("output" + strconv.Itoa(i), os.O_WRONLY|os.O_CREATE|os.O_TRUNC, f.Mode())
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
|
|
|
|
rc, err := f.Open()
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
|
|
|
|
_, err = io.Copy(out, rc)
|
|
|
|
|
|
|
|
out.Close()
|
|
|
|
rc.Close()
|
|
|
|
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"io"
|
|
|
|
"os"
|
|
|
|
)
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
s, err := os.Open("src")
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
defer s.Close()
|
|
|
|
|
|
|
|
d, err := os.Create("dst")
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
defer d.Close()
|
|
|
|
|
|
|
|
_, err = io.Copy(d, s)
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
}`}, 0, gosec.NewConfig()}}
|
|
|
|
|
2017-12-28 06:54:10 +00:00
|
|
|
// SampleCodeG201 - SQL injection via format string
|
|
|
|
SampleCodeG201 = []CodeSample{
|
2018-09-28 09:42:25 +01:00
|
|
|
{[]string{`
|
2017-12-28 06:54:10 +00:00
|
|
|
// Format string without proper quoting
|
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"database/sql"
|
|
|
|
"fmt"
|
|
|
|
"os"
|
|
|
|
)
|
|
|
|
|
|
|
|
func main(){
|
|
|
|
db, err := sql.Open("sqlite3", ":memory:")
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
q := fmt.Sprintf("SELECT * FROM foo where name = '%s'", os.Args[1])
|
|
|
|
rows, err := db.Query(q)
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
defer rows.Close()
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
2020-05-25 13:19:00 +01:00
|
|
|
// Format string without proper quoting case insensitive
|
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"database/sql"
|
|
|
|
"fmt"
|
|
|
|
"os"
|
|
|
|
)
|
|
|
|
|
|
|
|
func main(){
|
|
|
|
db, err := sql.Open("sqlite3", ":memory:")
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
q := fmt.Sprintf("select * from foo where name = '%s'", os.Args[1])
|
|
|
|
rows, err := db.Query(q)
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
defer rows.Close()
|
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
|
|
|
// Format string without proper quoting with context
|
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"context"
|
|
|
|
"database/sql"
|
|
|
|
"fmt"
|
|
|
|
"os"
|
|
|
|
)
|
|
|
|
|
|
|
|
func main(){
|
|
|
|
db, err := sql.Open("sqlite3", ":memory:")
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
q := fmt.Sprintf("select * from foo where name = '%s'", os.Args[1])
|
|
|
|
rows, err := db.QueryContext(context.Background(), q)
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
defer rows.Close()
|
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
|
|
|
// Format string without proper quoting with transation
|
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"context"
|
|
|
|
"database/sql"
|
|
|
|
"fmt"
|
|
|
|
"os"
|
|
|
|
)
|
|
|
|
|
|
|
|
func main(){
|
|
|
|
db, err := sql.Open("sqlite3", ":memory:")
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
tx, err := db.Begin()
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
defer tx.Rollback()
|
|
|
|
q := fmt.Sprintf("select * from foo where name = '%s'", os.Args[1])
|
|
|
|
rows, err := tx.QueryContext(context.Background(), q)
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
defer rows.Close()
|
|
|
|
if err := tx.Commit(); err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
2018-01-22 18:45:07 +00:00
|
|
|
// Format string false positive, safe string spec.
|
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"database/sql"
|
|
|
|
"fmt"
|
|
|
|
"os"
|
|
|
|
)
|
|
|
|
|
|
|
|
func main(){
|
|
|
|
db, err := sql.Open("sqlite3", ":memory:")
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
q := fmt.Sprintf("SELECT * FROM foo where id = %d", os.Args[1])
|
|
|
|
rows, err := db.Query(q)
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
defer rows.Close()
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 0, gosec.NewConfig()}, {[]string{`
|
2017-12-28 06:54:10 +00:00
|
|
|
// Format string false positive
|
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"database/sql"
|
|
|
|
)
|
2018-09-28 09:42:25 +01:00
|
|
|
const staticQuery = "SELECT * FROM foo WHERE age < 32"
|
2017-12-28 06:54:10 +00:00
|
|
|
func main(){
|
|
|
|
db, err := sql.Open("sqlite3", ":memory:")
|
|
|
|
if err != nil {
|
2018-09-28 09:42:25 +01:00
|
|
|
panic(err)
|
2017-12-28 06:54:10 +00:00
|
|
|
}
|
|
|
|
rows, err := db.Query(staticQuery)
|
|
|
|
if err != nil {
|
2018-09-28 09:42:25 +01:00
|
|
|
panic(err)
|
2017-12-28 06:54:10 +00:00
|
|
|
}
|
|
|
|
defer rows.Close()
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 0, gosec.NewConfig()}, {[]string{`
|
2018-09-25 08:40:05 +01:00
|
|
|
// Format string false positive, quoted formatter argument.
|
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"database/sql"
|
|
|
|
"fmt"
|
|
|
|
"os"
|
|
|
|
"github.com/lib/pq"
|
|
|
|
)
|
|
|
|
|
|
|
|
func main(){
|
|
|
|
db, err := sql.Open("postgres", "localhost")
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
q := fmt.Sprintf("SELECT * FROM %s where id = 1", pq.QuoteIdentifier(os.Args[1]))
|
|
|
|
rows, err := db.Query(q)
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
defer rows.Close()
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 0, gosec.NewConfig()}, {[]string{`
|
2019-06-12 20:53:17 +01:00
|
|
|
// false positive
|
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"database/sql"
|
|
|
|
"fmt"
|
|
|
|
)
|
|
|
|
|
|
|
|
const Table = "foo"
|
|
|
|
func main(){
|
|
|
|
db, err := sql.Open("sqlite3", ":memory:")
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
q := fmt.Sprintf("SELECT * FROM %s where id = 1", Table)
|
|
|
|
rows, err := db.Query(q)
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
defer rows.Close()
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 0, gosec.NewConfig()}, {[]string{`
|
2018-11-05 08:28:47 +00:00
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"fmt"
|
|
|
|
)
|
|
|
|
|
|
|
|
func main(){
|
|
|
|
fmt.Sprintln()
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 0, gosec.NewConfig()}}
|
2017-12-28 06:54:10 +00:00
|
|
|
|
|
|
|
// SampleCodeG202 - SQL query string building via string concatenation
|
|
|
|
SampleCodeG202 = []CodeSample{
|
2018-09-28 09:42:25 +01:00
|
|
|
{[]string{`
|
2017-12-28 06:54:10 +00:00
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"database/sql"
|
|
|
|
"os"
|
|
|
|
)
|
|
|
|
func main(){
|
|
|
|
db, err := sql.Open("sqlite3", ":memory:")
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
rows, err := db.Query("SELECT * FROM foo WHERE name = " + os.Args[1])
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
defer rows.Close()
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
2020-05-25 13:19:00 +01:00
|
|
|
// case insensitive match
|
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"database/sql"
|
|
|
|
"os"
|
|
|
|
)
|
|
|
|
func main(){
|
|
|
|
db, err := sql.Open("sqlite3", ":memory:")
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
rows, err := db.Query("select * from foo where name = " + os.Args[1])
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
defer rows.Close()
|
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
|
|
|
// context match
|
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"context"
|
|
|
|
"database/sql"
|
|
|
|
"os"
|
|
|
|
)
|
|
|
|
func main(){
|
|
|
|
db, err := sql.Open("sqlite3", ":memory:")
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
rows, err := db.QueryContext(context.Background(), "select * from foo where name = " + os.Args[1])
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
defer rows.Close()
|
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
|
|
|
// DB transation check
|
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"context"
|
|
|
|
"database/sql"
|
|
|
|
"os"
|
|
|
|
)
|
|
|
|
func main(){
|
|
|
|
db, err := sql.Open("sqlite3", ":memory:")
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
tx, err := db.Begin()
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
defer tx.Rollback()
|
|
|
|
rows, err := tx.QueryContext(context.Background(), "select * from foo where name = " + os.Args[1])
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
defer rows.Close()
|
|
|
|
if err := tx.Commit(); err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
2020-05-25 14:42:43 +01:00
|
|
|
// multiple string concatenation
|
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"database/sql"
|
|
|
|
"os"
|
|
|
|
)
|
|
|
|
func main(){
|
|
|
|
db, err := sql.Open("sqlite3", ":memory:")
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
rows, err := db.Query("SELECT * FROM foo" + "WHERE name = " + os.Args[1])
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
defer rows.Close()
|
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
2017-12-28 06:54:10 +00:00
|
|
|
// false positive
|
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"database/sql"
|
|
|
|
)
|
|
|
|
var staticQuery = "SELECT * FROM foo WHERE age < "
|
|
|
|
func main(){
|
|
|
|
db, err := sql.Open("sqlite3", ":memory:")
|
2018-09-28 08:46:59 +01:00
|
|
|
if err != nil {
|
2017-12-28 06:54:10 +00:00
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
rows, err := db.Query(staticQuery + "32")
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
2018-09-28 08:46:59 +01:00
|
|
|
defer rows.Close()
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 0, gosec.NewConfig()}, {[]string{`
|
2017-12-28 06:54:10 +00:00
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"database/sql"
|
|
|
|
)
|
|
|
|
const age = "32"
|
|
|
|
var staticQuery = "SELECT * FROM foo WHERE age < "
|
|
|
|
func main(){
|
|
|
|
db, err := sql.Open("sqlite3", ":memory:")
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
rows, err := db.Query(staticQuery + age)
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
defer rows.Close()
|
|
|
|
}
|
2019-06-25 09:29:19 +01:00
|
|
|
`}, 0, gosec.NewConfig()}, {[]string{`
|
2018-09-28 08:46:59 +01:00
|
|
|
package main
|
|
|
|
const gender = "M"
|
2018-09-28 09:42:25 +01:00
|
|
|
`, `
|
2018-09-28 08:46:59 +01:00
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"database/sql"
|
|
|
|
)
|
|
|
|
const age = "32"
|
|
|
|
var staticQuery = "SELECT * FROM foo WHERE age < "
|
|
|
|
func main(){
|
|
|
|
db, err := sql.Open("sqlite3", ":memory:")
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
rows, err := db.Query("SELECT * FROM foo WHERE gender = " + gender)
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
defer rows.Close()
|
|
|
|
}
|
2019-06-25 09:29:19 +01:00
|
|
|
`}, 0, gosec.NewConfig()}}
|
2017-12-28 06:54:10 +00:00
|
|
|
|
|
|
|
// SampleCodeG203 - Template checks
|
|
|
|
SampleCodeG203 = []CodeSample{
|
2018-09-28 09:42:25 +01:00
|
|
|
{[]string{`
|
2017-12-28 06:54:10 +00:00
|
|
|
// We assume that hardcoded template strings are safe as the programmer would
|
|
|
|
// need to be explicitly shooting themselves in the foot (as below)
|
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"html/template"
|
|
|
|
"os"
|
|
|
|
)
|
|
|
|
const tmpl = ""
|
|
|
|
func main() {
|
|
|
|
t := template.Must(template.New("ex").Parse(tmpl))
|
|
|
|
v := map[string]interface{}{
|
|
|
|
"Title": "Test <b>World</b>",
|
|
|
|
"Body": template.HTML("<script>alert(1)</script>"),
|
|
|
|
}
|
|
|
|
t.Execute(os.Stdout, v)
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 0, gosec.NewConfig()}, {[]string{
|
2017-12-28 06:54:10 +00:00
|
|
|
`
|
|
|
|
// Using a variable to initialize could potentially be dangerous. Under the
|
|
|
|
// current model this will likely produce some false positives.
|
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"html/template"
|
|
|
|
"os"
|
|
|
|
)
|
|
|
|
const tmpl = ""
|
|
|
|
func main() {
|
|
|
|
a := "something from another place"
|
|
|
|
t := template.Must(template.New("ex").Parse(tmpl))
|
|
|
|
v := map[string]interface{}{
|
|
|
|
"Title": "Test <b>World</b>",
|
|
|
|
"Body": template.HTML(a),
|
|
|
|
}
|
|
|
|
t.Execute(os.Stdout, v)
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{
|
2017-12-28 06:54:10 +00:00
|
|
|
`
|
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"html/template"
|
|
|
|
"os"
|
|
|
|
)
|
|
|
|
const tmpl = ""
|
|
|
|
func main() {
|
|
|
|
a := "something from another place"
|
|
|
|
t := template.Must(template.New("ex").Parse(tmpl))
|
|
|
|
v := map[string]interface{}{
|
|
|
|
"Title": "Test <b>World</b>",
|
|
|
|
"Body": template.JS(a),
|
|
|
|
}
|
|
|
|
t.Execute(os.Stdout, v)
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{
|
2017-12-28 06:54:10 +00:00
|
|
|
`
|
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"html/template"
|
|
|
|
"os"
|
|
|
|
)
|
|
|
|
const tmpl = ""
|
|
|
|
func main() {
|
|
|
|
a := "something from another place"
|
|
|
|
t := template.Must(template.New("ex").Parse(tmpl))
|
|
|
|
v := map[string]interface{}{
|
|
|
|
"Title": "Test <b>World</b>",
|
|
|
|
"Body": template.URL(a),
|
|
|
|
}
|
|
|
|
t.Execute(os.Stdout, v)
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}}
|
2017-12-28 06:54:10 +00:00
|
|
|
|
|
|
|
// SampleCodeG204 - Subprocess auditing
|
2018-09-28 09:42:25 +01:00
|
|
|
SampleCodeG204 = []CodeSample{{[]string{`
|
2017-12-28 06:54:10 +00:00
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"log"
|
|
|
|
"os/exec"
|
2019-09-16 15:15:06 +01:00
|
|
|
"context"
|
2017-12-28 06:54:10 +00:00
|
|
|
)
|
|
|
|
func main() {
|
2020-03-11 14:18:38 +00:00
|
|
|
err := exec.CommandContext(context.Background(), "git", "rev-parse", "--show-toplavel").Run()
|
2017-12-28 06:54:10 +00:00
|
|
|
if err != nil {
|
|
|
|
log.Fatal(err)
|
|
|
|
}
|
|
|
|
log.Printf("Command finished with error: %v", err)
|
2020-03-11 14:18:38 +00:00
|
|
|
}`}, 0, gosec.NewConfig()}, {[]string{`
|
2019-09-16 15:15:06 +01:00
|
|
|
// Calling any function which starts a new process with using
|
|
|
|
// command line arguments as it's arguments is considered dangerous
|
2017-12-28 06:54:10 +00:00
|
|
|
package main
|
2018-06-03 22:43:28 +01:00
|
|
|
import (
|
2021-01-01 19:45:13 +00:00
|
|
|
"context"
|
2018-06-03 22:43:28 +01:00
|
|
|
"log"
|
2019-09-16 15:15:06 +01:00
|
|
|
"os"
|
2018-06-03 22:43:28 +01:00
|
|
|
"os/exec"
|
|
|
|
)
|
|
|
|
func main() {
|
2020-03-11 14:18:38 +00:00
|
|
|
err := exec.CommandContext(context.Background(), os.Args[0], "5").Run()
|
2019-09-16 15:15:06 +01:00
|
|
|
if err != nil {
|
2018-06-03 22:43:28 +01:00
|
|
|
log.Fatal(err)
|
|
|
|
}
|
2019-09-16 15:15:06 +01:00
|
|
|
log.Printf("Command finished with error: %v", err)
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
2019-09-16 15:15:06 +01:00
|
|
|
// Initializing a local variable using a environmental
|
|
|
|
// variable is consider as a dangerous user input
|
2018-06-03 22:43:28 +01:00
|
|
|
package main
|
2017-12-28 06:54:10 +00:00
|
|
|
import (
|
|
|
|
"log"
|
|
|
|
"os"
|
|
|
|
"os/exec"
|
|
|
|
)
|
|
|
|
func main() {
|
|
|
|
run := "sleep" + os.Getenv("SOMETHING")
|
|
|
|
cmd := exec.Command(run, "5")
|
|
|
|
err := cmd.Start()
|
|
|
|
if err != nil {
|
|
|
|
log.Fatal(err)
|
|
|
|
}
|
|
|
|
log.Printf("Waiting for command to finish...")
|
|
|
|
err = cmd.Wait()
|
|
|
|
log.Printf("Command finished with error: %v", err)
|
2019-09-16 15:15:06 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
|
|
|
// gosec doesn't have enough context to decide that the
|
|
|
|
// command argument of the RunCmd function is harcoded string
|
|
|
|
// and that's why it's better to warn the user so he can audit it
|
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"log"
|
|
|
|
"os/exec"
|
|
|
|
)
|
|
|
|
|
|
|
|
func RunCmd(command string) {
|
|
|
|
cmd := exec.Command(command, "5")
|
|
|
|
err := cmd.Start()
|
|
|
|
if err != nil {
|
|
|
|
log.Fatal(err)
|
|
|
|
}
|
|
|
|
log.Printf("Waiting for command to finish...")
|
|
|
|
err = cmd.Wait()
|
|
|
|
}
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
RunCmd("sleep")
|
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
|
|
|
// syscall.Exec function called with harcoded arguments
|
|
|
|
// shouldn't be consider as a command injection
|
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"fmt"
|
|
|
|
"syscall"
|
|
|
|
)
|
|
|
|
func main() {
|
|
|
|
err := syscall.Exec("/bin/cat", []string{"/etc/passwd"}, nil)
|
|
|
|
if err != nil {
|
|
|
|
fmt.Printf("Error: %v\n", err)
|
|
|
|
}
|
|
|
|
}`}, 0, gosec.NewConfig()},
|
|
|
|
{[]string{`
|
2020-03-11 13:00:30 +00:00
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"fmt"
|
|
|
|
"syscall"
|
|
|
|
)
|
|
|
|
|
|
|
|
func RunCmd(command string) {
|
|
|
|
_, err := syscall.ForkExec(command, []string{}, nil)
|
|
|
|
if err != nil {
|
|
|
|
fmt.Printf("Error: %v\n", err)
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
RunCmd("sleep")
|
|
|
|
}`}, 1, gosec.NewConfig(),
|
|
|
|
},
|
|
|
|
{[]string{`
|
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"fmt"
|
|
|
|
"syscall"
|
|
|
|
)
|
|
|
|
|
|
|
|
func RunCmd(command string) {
|
2021-01-01 19:45:13 +00:00
|
|
|
_, _, err := syscall.StartProcess(command, []string{}, nil)
|
2020-03-11 13:00:30 +00:00
|
|
|
if err != nil {
|
|
|
|
fmt.Printf("Error: %v\n", err)
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
RunCmd("sleep")
|
|
|
|
}`}, 1, gosec.NewConfig(),
|
|
|
|
},
|
|
|
|
{[]string{`
|
2019-09-16 15:15:06 +01:00
|
|
|
// starting a process with a variable as an argument
|
|
|
|
// even if not constant is not considered as dangerous
|
|
|
|
// because it has harcoded value
|
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"log"
|
|
|
|
"os/exec"
|
|
|
|
)
|
|
|
|
func main() {
|
|
|
|
run := "sleep"
|
|
|
|
cmd := exec.Command(run, "5")
|
|
|
|
err := cmd.Start()
|
|
|
|
if err != nil {
|
|
|
|
log.Fatal(err)
|
|
|
|
}
|
|
|
|
log.Printf("Waiting for command to finish...")
|
|
|
|
err = cmd.Wait()
|
|
|
|
log.Printf("Command finished with error: %v", err)
|
|
|
|
}`}, 0, gosec.NewConfig()}}
|
2017-12-28 06:54:10 +00:00
|
|
|
|
|
|
|
// SampleCodeG301 - mkdir permission check
|
2018-09-28 09:42:25 +01:00
|
|
|
SampleCodeG301 = []CodeSample{{[]string{`
|
2017-12-28 06:54:10 +00:00
|
|
|
package main
|
2019-09-24 09:15:56 +01:00
|
|
|
|
|
|
|
import (
|
|
|
|
"fmt"
|
|
|
|
"os"
|
|
|
|
)
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
err := os.Mkdir("/tmp/mydir", 0777)
|
|
|
|
if err != nil {
|
|
|
|
fmt.Println("Error when creating a directory!")
|
|
|
|
return
|
|
|
|
}
|
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"fmt"
|
|
|
|
"os"
|
|
|
|
)
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
err := os.MkdirAll("/tmp/mydir", 0777)
|
|
|
|
if err != nil {
|
|
|
|
fmt.Println("Error when creating a directory!")
|
|
|
|
return
|
|
|
|
}
|
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"fmt"
|
|
|
|
"os"
|
|
|
|
)
|
|
|
|
|
2017-12-28 06:54:10 +00:00
|
|
|
func main() {
|
2019-09-24 09:15:56 +01:00
|
|
|
err := os.Mkdir("/tmp/mydir", 0600)
|
|
|
|
if err != nil {
|
|
|
|
fmt.Println("Error when creating a directory!")
|
|
|
|
return
|
|
|
|
}
|
|
|
|
}`}, 0, gosec.NewConfig()}}
|
2017-12-28 06:54:10 +00:00
|
|
|
|
|
|
|
// SampleCodeG302 - file create / chmod permissions check
|
2018-09-28 09:42:25 +01:00
|
|
|
SampleCodeG302 = []CodeSample{{[]string{`
|
2017-12-28 06:54:10 +00:00
|
|
|
package main
|
2019-09-24 09:15:56 +01:00
|
|
|
|
|
|
|
import (
|
|
|
|
"fmt"
|
|
|
|
"os"
|
|
|
|
)
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
err := os.Chmod("/tmp/somefile", 0777)
|
|
|
|
if err != nil {
|
|
|
|
fmt.Println("Error when changing file permissions!")
|
|
|
|
return
|
|
|
|
}
|
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"fmt"
|
|
|
|
"os"
|
|
|
|
)
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
_, err := os.OpenFile("/tmp/thing", os.O_CREATE|os.O_WRONLY, 0666)
|
|
|
|
if err != nil {
|
|
|
|
fmt.Println("Error opening a file!")
|
|
|
|
return
|
|
|
|
}
|
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"fmt"
|
|
|
|
"os"
|
|
|
|
)
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
err := os.Chmod("/tmp/mydir", 0400)
|
|
|
|
if err != nil {
|
|
|
|
fmt.Println("Error")
|
|
|
|
return
|
|
|
|
}
|
|
|
|
}`}, 0, gosec.NewConfig()}, {[]string{`
|
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"fmt"
|
|
|
|
"os"
|
|
|
|
)
|
|
|
|
|
2017-12-28 06:54:10 +00:00
|
|
|
func main() {
|
2019-09-24 09:15:56 +01:00
|
|
|
_, err := os.OpenFile("/tmp/thing", os.O_CREATE|os.O_WRONLY, 0600)
|
|
|
|
if err != nil {
|
|
|
|
fmt.Println("Error opening a file!")
|
|
|
|
return
|
|
|
|
}
|
|
|
|
}
|
|
|
|
`}, 0, gosec.NewConfig()}}
|
2017-12-28 06:54:10 +00:00
|
|
|
|
|
|
|
// SampleCodeG303 - bad tempfile permissions & hardcoded shared path
|
2018-09-28 09:42:25 +01:00
|
|
|
SampleCodeG303 = []CodeSample{{[]string{`
|
2017-12-28 06:54:10 +00:00
|
|
|
package samples
|
2019-09-24 09:15:56 +01:00
|
|
|
|
2017-12-28 06:54:10 +00:00
|
|
|
import (
|
2019-09-24 09:15:56 +01:00
|
|
|
"fmt"
|
2017-12-28 06:54:10 +00:00
|
|
|
"io/ioutil"
|
|
|
|
)
|
2019-09-24 09:15:56 +01:00
|
|
|
|
2017-12-28 06:54:10 +00:00
|
|
|
func main() {
|
2019-09-24 09:15:56 +01:00
|
|
|
err := ioutil.WriteFile("/tmp/demo2", []byte("This is some data"), 0644)
|
|
|
|
if err != nil {
|
|
|
|
fmt.Println("Error while writing!")
|
|
|
|
}
|
|
|
|
}`}, 1, gosec.NewConfig()}}
|
2017-12-28 06:54:10 +00:00
|
|
|
|
2018-03-08 23:23:27 +00:00
|
|
|
// SampleCodeG304 - potential file inclusion vulnerability
|
2018-09-28 09:42:25 +01:00
|
|
|
SampleCodeG304 = []CodeSample{{[]string{`
|
2018-03-08 23:23:27 +00:00
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"os"
|
|
|
|
"io/ioutil"
|
|
|
|
"log"
|
|
|
|
)
|
|
|
|
func main() {
|
2019-09-24 09:15:56 +01:00
|
|
|
f := os.Getenv("tainted_file")
|
|
|
|
body, err := ioutil.ReadFile(f)
|
|
|
|
if err != nil {
|
|
|
|
log.Printf("Error: %v\n", err)
|
|
|
|
}
|
|
|
|
log.Print(body)
|
2018-03-08 23:23:27 +00:00
|
|
|
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
2018-03-08 23:23:27 +00:00
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"fmt"
|
|
|
|
"log"
|
|
|
|
"net/http"
|
|
|
|
"os"
|
|
|
|
)
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
http.HandleFunc("/bar", func(w http.ResponseWriter, r *http.Request) {
|
2018-08-28 05:34:07 +01:00
|
|
|
title := r.URL.Query().Get("title")
|
2018-03-08 23:23:27 +00:00
|
|
|
f, err := os.Open(title)
|
|
|
|
if err != nil {
|
|
|
|
fmt.Printf("Error: %v\n", err)
|
|
|
|
}
|
|
|
|
body := make([]byte, 5)
|
2018-03-09 05:28:56 +00:00
|
|
|
if _, err = f.Read(body); err != nil {
|
2018-03-08 23:23:27 +00:00
|
|
|
fmt.Printf("Error: %v\n", err)
|
|
|
|
}
|
|
|
|
fmt.Fprintf(w, "%s", body)
|
|
|
|
})
|
|
|
|
log.Fatal(http.ListenAndServe(":3000", nil))
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
2018-08-28 05:34:07 +01:00
|
|
|
package main
|
|
|
|
|
2020-06-16 12:29:03 +01:00
|
|
|
import (
|
|
|
|
"fmt"
|
|
|
|
"log"
|
|
|
|
"net/http"
|
|
|
|
"os"
|
|
|
|
)
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
http.HandleFunc("/bar", func(w http.ResponseWriter, r *http.Request) {
|
|
|
|
title := r.URL.Query().Get("title")
|
|
|
|
f, err := os.OpenFile(title, os.O_RDWR|os.O_CREATE, 0755)
|
|
|
|
if err != nil {
|
|
|
|
fmt.Printf("Error: %v\n", err)
|
|
|
|
}
|
|
|
|
body := make([]byte, 5)
|
|
|
|
if _, err = f.Read(body); err != nil {
|
|
|
|
fmt.Printf("Error: %v\n", err)
|
|
|
|
}
|
|
|
|
fmt.Fprintf(w, "%s", body)
|
|
|
|
})
|
|
|
|
log.Fatal(http.ListenAndServe(":3000", nil))
|
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
|
|
|
package main
|
|
|
|
|
2018-08-28 05:34:07 +01:00
|
|
|
import (
|
|
|
|
"log"
|
|
|
|
"os"
|
|
|
|
"io/ioutil"
|
|
|
|
)
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
f2 := os.Getenv("tainted_file2")
|
|
|
|
body, err := ioutil.ReadFile("/tmp/" + f2)
|
|
|
|
if err != nil {
|
|
|
|
log.Printf("Error: %v\n", err)
|
|
|
|
}
|
|
|
|
log.Print(body)
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
2018-08-28 05:34:07 +01:00
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"bufio"
|
|
|
|
"fmt"
|
|
|
|
"os"
|
|
|
|
"path/filepath"
|
|
|
|
)
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
reader := bufio.NewReader(os.Stdin)
|
|
|
|
fmt.Print("Please enter file to read: ")
|
|
|
|
file, _ := reader.ReadString('\n')
|
|
|
|
file = file[:len(file)-1]
|
|
|
|
f, err := os.Open(filepath.Join("/tmp/service/", file))
|
|
|
|
if err != nil {
|
|
|
|
fmt.Printf("Error: %v\n", err)
|
|
|
|
}
|
|
|
|
contents := make([]byte, 15)
|
|
|
|
if _, err = f.Read(contents); err != nil {
|
|
|
|
fmt.Printf("Error: %v\n", err)
|
|
|
|
}
|
|
|
|
fmt.Println(string(contents))
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
2018-08-28 05:34:07 +01:00
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"log"
|
|
|
|
"os"
|
|
|
|
"io/ioutil"
|
|
|
|
"path/filepath"
|
|
|
|
)
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
dir := os.Getenv("server_root")
|
|
|
|
f3 := os.Getenv("tainted_file3")
|
|
|
|
// edge case where both a binary expression and file Join are used.
|
|
|
|
body, err := ioutil.ReadFile(filepath.Join("/var/"+dir, f3))
|
|
|
|
if err != nil {
|
|
|
|
log.Printf("Error: %v\n", err)
|
|
|
|
}
|
|
|
|
log.Print(body)
|
2020-08-17 20:05:28 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"os"
|
|
|
|
"path/filepath"
|
|
|
|
)
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
repoFile := "path_of_file"
|
|
|
|
cleanRepoFile := filepath.Clean(repoFile)
|
2021-01-01 19:45:13 +00:00
|
|
|
_, err := os.OpenFile(cleanRepoFile, os.O_RDONLY, 0600)
|
2020-08-17 20:05:28 +01:00
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
}
|
|
|
|
`}, 0, gosec.NewConfig()}, {[]string{`
|
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"os"
|
|
|
|
"path/filepath"
|
|
|
|
)
|
|
|
|
|
|
|
|
func openFile(filePath string) {
|
2021-01-01 19:45:13 +00:00
|
|
|
_, err := os.OpenFile(filepath.Clean(filePath), os.O_RDONLY, 0600)
|
2020-08-17 20:05:28 +01:00
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
repoFile := "path_of_file"
|
|
|
|
openFile(repoFile)
|
|
|
|
}
|
2020-08-19 07:39:55 +01:00
|
|
|
`}, 0, gosec.NewConfig()}, {[]string{`
|
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"os"
|
|
|
|
"path/filepath"
|
|
|
|
)
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
repoFile := "path_of_file"
|
2021-01-04 08:19:01 +00:00
|
|
|
relFile, err := filepath.Rel("./", repoFile)
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
_, err = os.OpenFile(relFile, os.O_RDONLY, 0600)
|
2020-08-19 07:39:55 +01:00
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2020-08-17 20:05:28 +01:00
|
|
|
`}, 0, gosec.NewConfig()}}
|
2018-03-08 23:23:27 +00:00
|
|
|
|
2020-08-02 16:09:02 +01:00
|
|
|
// SampleCodeG305 - File path traversal when extracting zip/tar archives
|
2018-09-28 09:42:25 +01:00
|
|
|
SampleCodeG305 = []CodeSample{{[]string{`
|
2018-07-18 13:31:07 +01:00
|
|
|
package unzip
|
|
|
|
|
|
|
|
import (
|
|
|
|
"archive/zip"
|
|
|
|
"io"
|
|
|
|
"os"
|
|
|
|
"path/filepath"
|
|
|
|
)
|
|
|
|
|
|
|
|
func unzip(archive, target string) error {
|
|
|
|
reader, err := zip.OpenReader(archive)
|
|
|
|
if err != nil {
|
|
|
|
return err
|
|
|
|
}
|
|
|
|
|
|
|
|
if err := os.MkdirAll(target, 0750); err != nil {
|
|
|
|
return err
|
|
|
|
}
|
|
|
|
|
|
|
|
for _, file := range reader.File {
|
|
|
|
path := filepath.Join(target, file.Name)
|
|
|
|
if file.FileInfo().IsDir() {
|
|
|
|
os.MkdirAll(path, file.Mode()) // #nosec
|
|
|
|
continue
|
|
|
|
}
|
|
|
|
|
|
|
|
fileReader, err := file.Open()
|
|
|
|
if err != nil {
|
|
|
|
return err
|
|
|
|
}
|
|
|
|
defer fileReader.Close()
|
|
|
|
|
|
|
|
targetFile, err := os.OpenFile(path, os.O_WRONLY|os.O_CREATE|os.O_TRUNC, file.Mode())
|
|
|
|
if err != nil {
|
|
|
|
return err
|
|
|
|
}
|
|
|
|
defer targetFile.Close()
|
|
|
|
|
|
|
|
if _, err := io.Copy(targetFile, fileReader); err != nil {
|
|
|
|
return err
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
return nil
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
2018-07-18 13:31:07 +01:00
|
|
|
package unzip
|
|
|
|
|
|
|
|
import (
|
|
|
|
"archive/zip"
|
|
|
|
"io"
|
|
|
|
"os"
|
|
|
|
"path/filepath"
|
|
|
|
)
|
|
|
|
|
|
|
|
func unzip(archive, target string) error {
|
|
|
|
reader, err := zip.OpenReader(archive)
|
|
|
|
if err != nil {
|
|
|
|
return err
|
|
|
|
}
|
|
|
|
|
|
|
|
if err := os.MkdirAll(target, 0750); err != nil {
|
|
|
|
return err
|
|
|
|
}
|
|
|
|
|
|
|
|
for _, file := range reader.File {
|
|
|
|
archiveFile := file.Name
|
|
|
|
path := filepath.Join(target, archiveFile)
|
|
|
|
if file.FileInfo().IsDir() {
|
|
|
|
os.MkdirAll(path, file.Mode()) // #nosec
|
|
|
|
continue
|
|
|
|
}
|
|
|
|
|
|
|
|
fileReader, err := file.Open()
|
|
|
|
if err != nil {
|
|
|
|
return err
|
|
|
|
}
|
|
|
|
defer fileReader.Close()
|
|
|
|
|
|
|
|
targetFile, err := os.OpenFile(path, os.O_WRONLY|os.O_CREATE|os.O_TRUNC, file.Mode())
|
|
|
|
if err != nil {
|
|
|
|
return err
|
|
|
|
}
|
|
|
|
defer targetFile.Close()
|
|
|
|
|
|
|
|
if _, err := io.Copy(targetFile, fileReader); err != nil {
|
|
|
|
return err
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
return nil
|
2020-08-02 16:09:02 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
|
|
|
package zip
|
|
|
|
|
|
|
|
import (
|
|
|
|
"archive/zip"
|
|
|
|
"io"
|
|
|
|
"os"
|
|
|
|
"path"
|
|
|
|
)
|
|
|
|
|
|
|
|
func extractFile(f *zip.File, destPath string) error {
|
|
|
|
filePath := path.Join(destPath, f.Name)
|
|
|
|
os.MkdirAll(path.Dir(filePath), os.ModePerm)
|
|
|
|
|
|
|
|
rc, err := f.Open()
|
|
|
|
if err != nil {
|
|
|
|
return err
|
|
|
|
}
|
|
|
|
defer rc.Close()
|
|
|
|
|
|
|
|
fw, err := os.Create(filePath)
|
|
|
|
if err != nil {
|
|
|
|
return err
|
|
|
|
}
|
|
|
|
defer fw.Close()
|
|
|
|
|
|
|
|
if _, err = io.Copy(fw, rc); err != nil {
|
|
|
|
return err
|
|
|
|
}
|
|
|
|
|
|
|
|
if f.FileInfo().Mode()&os.ModeSymlink != 0 {
|
|
|
|
return nil
|
|
|
|
}
|
|
|
|
|
|
|
|
if err = os.Chtimes(filePath, f.ModTime(), f.ModTime()); err != nil {
|
|
|
|
return err
|
|
|
|
}
|
|
|
|
return os.Chmod(filePath, f.FileInfo().Mode())
|
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
|
|
|
package tz
|
|
|
|
|
|
|
|
import (
|
|
|
|
"archive/tar"
|
|
|
|
"io"
|
|
|
|
"os"
|
|
|
|
"path"
|
|
|
|
)
|
|
|
|
|
|
|
|
func extractFile(f *tar.Header, tr *tar.Reader, destPath string) error {
|
|
|
|
filePath := path.Join(destPath, f.Name)
|
|
|
|
os.MkdirAll(path.Dir(filePath), os.ModePerm)
|
|
|
|
|
|
|
|
fw, err := os.Create(filePath)
|
|
|
|
if err != nil {
|
|
|
|
return err
|
|
|
|
}
|
|
|
|
defer fw.Close()
|
|
|
|
|
|
|
|
if _, err = io.Copy(fw, tr); err != nil {
|
|
|
|
return err
|
|
|
|
}
|
|
|
|
|
|
|
|
if f.FileInfo().Mode()&os.ModeSymlink != 0 {
|
|
|
|
return nil
|
|
|
|
}
|
|
|
|
|
|
|
|
if err = os.Chtimes(filePath, f.FileInfo().ModTime(), f.FileInfo().ModTime()); err != nil {
|
|
|
|
return err
|
|
|
|
}
|
|
|
|
return os.Chmod(filePath, f.FileInfo().Mode())
|
2020-02-28 11:48:18 +00:00
|
|
|
}`}, 1, gosec.NewConfig()}}
|
|
|
|
|
|
|
|
// SampleCodeG306 - Poor permissions for WriteFile
|
|
|
|
SampleCodeG306 = []CodeSample{
|
|
|
|
{[]string{`package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"bufio"
|
|
|
|
"fmt"
|
|
|
|
"io/ioutil"
|
|
|
|
"os"
|
|
|
|
)
|
|
|
|
|
|
|
|
func check(e error) {
|
|
|
|
if e != nil {
|
|
|
|
panic(e)
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
func main() {
|
2020-06-29 12:21:15 +01:00
|
|
|
|
2020-02-28 11:48:18 +00:00
|
|
|
d1 := []byte("hello\ngo\n")
|
|
|
|
err := ioutil.WriteFile("/tmp/dat1", d1, 0744)
|
|
|
|
check(err)
|
|
|
|
|
|
|
|
allowed := ioutil.WriteFile("/tmp/dat1", d1, 0600)
|
|
|
|
check(allowed)
|
2020-06-29 12:21:15 +01:00
|
|
|
|
2020-02-28 11:48:18 +00:00
|
|
|
f, err := os.Create("/tmp/dat2")
|
|
|
|
check(err)
|
2020-06-29 12:21:15 +01:00
|
|
|
|
2020-02-28 11:48:18 +00:00
|
|
|
defer f.Close()
|
2020-06-29 12:21:15 +01:00
|
|
|
|
2020-02-28 11:48:18 +00:00
|
|
|
d2 := []byte{115, 111, 109, 101, 10}
|
|
|
|
n2, err := f.Write(d2)
|
2020-03-01 20:45:37 +00:00
|
|
|
|
|
|
|
defer check(err)
|
|
|
|
fmt.Printf("wrote %d bytes\n", n2)
|
2020-06-29 12:21:15 +01:00
|
|
|
|
2020-03-01 20:45:37 +00:00
|
|
|
n3, err := f.WriteString("writes\n")
|
|
|
|
fmt.Printf("wrote %d bytes\n", n3)
|
2020-06-29 12:21:15 +01:00
|
|
|
|
2020-03-01 20:45:37 +00:00
|
|
|
f.Sync()
|
2020-06-29 12:21:15 +01:00
|
|
|
|
2020-03-01 20:45:37 +00:00
|
|
|
w := bufio.NewWriter(f)
|
|
|
|
n4, err := w.WriteString("buffered\n")
|
|
|
|
fmt.Printf("wrote %d bytes\n", n4)
|
2020-06-29 12:21:15 +01:00
|
|
|
|
2020-03-01 20:45:37 +00:00
|
|
|
w.Flush()
|
|
|
|
|
|
|
|
}`}, 1, gosec.NewConfig()}}
|
|
|
|
// SampleCodeG307 - Unsafe defer of os.Close
|
|
|
|
SampleCodeG307 = []CodeSample{
|
|
|
|
{[]string{`package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"bufio"
|
|
|
|
"fmt"
|
|
|
|
"io/ioutil"
|
|
|
|
"os"
|
|
|
|
)
|
|
|
|
|
|
|
|
func check(e error) {
|
|
|
|
if e != nil {
|
|
|
|
panic(e)
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
|
|
|
|
d1 := []byte("hello\ngo\n")
|
|
|
|
err := ioutil.WriteFile("/tmp/dat1", d1, 0744)
|
|
|
|
check(err)
|
|
|
|
|
|
|
|
allowed := ioutil.WriteFile("/tmp/dat1", d1, 0600)
|
|
|
|
check(allowed)
|
|
|
|
|
|
|
|
f, err := os.Create("/tmp/dat2")
|
2020-02-28 11:48:18 +00:00
|
|
|
check(err)
|
2020-03-01 20:45:37 +00:00
|
|
|
|
|
|
|
defer f.Close()
|
|
|
|
|
|
|
|
d2 := []byte{115, 111, 109, 101, 10}
|
|
|
|
n2, err := f.Write(d2)
|
|
|
|
|
|
|
|
defer check(err)
|
2020-02-28 11:48:18 +00:00
|
|
|
fmt.Printf("wrote %d bytes\n", n2)
|
2020-06-29 12:21:15 +01:00
|
|
|
|
2020-02-28 11:48:18 +00:00
|
|
|
n3, err := f.WriteString("writes\n")
|
|
|
|
fmt.Printf("wrote %d bytes\n", n3)
|
2020-06-29 12:21:15 +01:00
|
|
|
|
2020-02-28 11:48:18 +00:00
|
|
|
f.Sync()
|
2020-06-29 12:21:15 +01:00
|
|
|
|
2020-02-28 11:48:18 +00:00
|
|
|
w := bufio.NewWriter(f)
|
|
|
|
n4, err := w.WriteString("buffered\n")
|
|
|
|
fmt.Printf("wrote %d bytes\n", n4)
|
2020-06-29 12:21:15 +01:00
|
|
|
|
2020-02-28 11:48:18 +00:00
|
|
|
w.Flush()
|
|
|
|
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}}
|
2018-07-18 13:31:07 +01:00
|
|
|
|
2017-07-19 22:17:00 +01:00
|
|
|
// SampleCodeG401 - Use of weak crypto MD5
|
|
|
|
SampleCodeG401 = []CodeSample{
|
2018-09-28 09:42:25 +01:00
|
|
|
{[]string{`
|
2017-07-19 22:17:00 +01:00
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"crypto/md5"
|
|
|
|
"fmt"
|
|
|
|
"io"
|
|
|
|
"log"
|
|
|
|
"os"
|
|
|
|
)
|
2020-03-01 20:45:37 +00:00
|
|
|
|
2017-07-19 22:17:00 +01:00
|
|
|
func main() {
|
|
|
|
f, err := os.Open("file.txt")
|
|
|
|
if err != nil {
|
|
|
|
log.Fatal(err)
|
|
|
|
}
|
|
|
|
defer f.Close()
|
|
|
|
|
2020-03-01 20:45:37 +00:00
|
|
|
defer func() {
|
|
|
|
err := f.Close()
|
|
|
|
if err != nil {
|
|
|
|
log.Printf("error closing the file: %s", err)
|
|
|
|
}
|
|
|
|
}()
|
|
|
|
|
2017-07-19 22:17:00 +01:00
|
|
|
h := md5.New()
|
|
|
|
if _, err := io.Copy(h, f); err != nil {
|
|
|
|
log.Fatal(err)
|
|
|
|
}
|
|
|
|
fmt.Printf("%x", h.Sum(nil))
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}}
|
2017-12-28 06:54:10 +00:00
|
|
|
|
2018-08-08 15:38:57 +01:00
|
|
|
// SampleCodeG401b - Use of weak crypto SHA1
|
|
|
|
SampleCodeG401b = []CodeSample{
|
2018-09-28 09:42:25 +01:00
|
|
|
{[]string{`
|
2018-08-08 15:38:57 +01:00
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"crypto/sha1"
|
|
|
|
"fmt"
|
|
|
|
"io"
|
|
|
|
"log"
|
|
|
|
"os"
|
|
|
|
)
|
|
|
|
func main() {
|
|
|
|
f, err := os.Open("file.txt")
|
|
|
|
if err != nil {
|
|
|
|
log.Fatal(err)
|
|
|
|
}
|
|
|
|
defer f.Close()
|
|
|
|
|
|
|
|
h := sha1.New()
|
|
|
|
if _, err := io.Copy(h, f); err != nil {
|
|
|
|
log.Fatal(err)
|
|
|
|
}
|
|
|
|
fmt.Printf("%x", h.Sum(nil))
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}}
|
2018-08-08 15:38:57 +01:00
|
|
|
|
2017-12-28 06:54:10 +00:00
|
|
|
// SampleCodeG402 - TLS settings
|
2018-09-28 09:42:25 +01:00
|
|
|
SampleCodeG402 = []CodeSample{{[]string{`
|
2017-12-28 06:54:10 +00:00
|
|
|
// InsecureSkipVerify
|
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"crypto/tls"
|
|
|
|
"fmt"
|
|
|
|
"net/http"
|
|
|
|
)
|
|
|
|
func main() {
|
|
|
|
tr := &http.Transport{
|
|
|
|
TLSClientConfig: &tls.Config{InsecureSkipVerify: true},
|
|
|
|
}
|
|
|
|
|
|
|
|
client := &http.Client{Transport: tr}
|
|
|
|
_, err := client.Get("https://golang.org/")
|
|
|
|
if err != nil {
|
|
|
|
fmt.Println(err)
|
|
|
|
}
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{
|
2017-12-28 06:54:10 +00:00
|
|
|
`
|
|
|
|
// Insecure minimum version
|
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"crypto/tls"
|
|
|
|
"fmt"
|
|
|
|
"net/http"
|
|
|
|
)
|
|
|
|
func main() {
|
|
|
|
tr := &http.Transport{
|
|
|
|
TLSClientConfig: &tls.Config{MinVersion: 0},
|
|
|
|
}
|
|
|
|
client := &http.Client{Transport: tr}
|
|
|
|
_, err := client.Get("https://golang.org/")
|
|
|
|
if err != nil {
|
|
|
|
fmt.Println(err)
|
|
|
|
}
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
2017-12-28 06:54:10 +00:00
|
|
|
// Insecure max version
|
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"crypto/tls"
|
|
|
|
"fmt"
|
|
|
|
"net/http"
|
|
|
|
)
|
|
|
|
func main() {
|
|
|
|
tr := &http.Transport{
|
|
|
|
TLSClientConfig: &tls.Config{MaxVersion: 0},
|
|
|
|
}
|
|
|
|
client := &http.Client{Transport: tr}
|
|
|
|
_, err := client.Get("https://golang.org/")
|
|
|
|
if err != nil {
|
|
|
|
fmt.Println(err)
|
|
|
|
}
|
|
|
|
}
|
2019-06-25 09:29:19 +01:00
|
|
|
`}, 1, gosec.NewConfig()}, {
|
2018-09-28 09:42:25 +01:00
|
|
|
[]string{`
|
2017-12-28 06:54:10 +00:00
|
|
|
// Insecure ciphersuite selection
|
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"crypto/tls"
|
|
|
|
"fmt"
|
|
|
|
"net/http"
|
|
|
|
)
|
|
|
|
func main() {
|
|
|
|
tr := &http.Transport{
|
|
|
|
TLSClientConfig: &tls.Config{CipherSuites: []uint16{
|
2019-09-09 14:14:41 +01:00
|
|
|
tls.TLS_RSA_WITH_AES_128_GCM_SHA256,
|
2017-12-28 06:54:10 +00:00
|
|
|
tls.TLS_ECDHE_ECDSA_WITH_AES_128_GCM_SHA256,
|
|
|
|
},},
|
|
|
|
}
|
|
|
|
client := &http.Client{Transport: tr}
|
|
|
|
_, err := client.Get("https://golang.org/")
|
|
|
|
if err != nil {
|
|
|
|
fmt.Println(err)
|
|
|
|
}
|
2020-06-23 10:34:08 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}, {[]string{`
|
|
|
|
// secure max version when min version is specified
|
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"crypto/tls"
|
|
|
|
"fmt"
|
|
|
|
"net/http"
|
|
|
|
)
|
|
|
|
func main() {
|
|
|
|
tr := &http.Transport{
|
|
|
|
TLSClientConfig: &tls.Config{MaxVersion: 0, MinVersion: tls.VersionTLS13},
|
|
|
|
}
|
|
|
|
client := &http.Client{Transport: tr}
|
|
|
|
_, err := client.Get("https://golang.org/")
|
|
|
|
if err != nil {
|
|
|
|
fmt.Println(err)
|
|
|
|
}
|
2021-02-11 09:52:16 +00:00
|
|
|
}`}, 0, gosec.NewConfig()}, {[]string{`
|
|
|
|
package p0
|
|
|
|
|
|
|
|
import "crypto/tls"
|
|
|
|
|
|
|
|
func TlsConfig0() *tls.Config {
|
|
|
|
var v uint16 = 0
|
|
|
|
return &tls.Config{MinVersion: v}
|
|
|
|
}
|
|
|
|
`, `
|
|
|
|
package p0
|
|
|
|
|
|
|
|
import "crypto/tls"
|
|
|
|
|
|
|
|
func TlsConfig1() *tls.Config {
|
|
|
|
return &tls.Config{MinVersion: 0x0304}
|
|
|
|
}
|
|
|
|
`}, 1, gosec.NewConfig()}}
|
2017-12-28 06:54:10 +00:00
|
|
|
|
|
|
|
// SampleCodeG403 - weak key strength
|
|
|
|
SampleCodeG403 = []CodeSample{
|
2018-09-28 09:42:25 +01:00
|
|
|
{[]string{`
|
2017-12-28 06:54:10 +00:00
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"crypto/rand"
|
|
|
|
"crypto/rsa"
|
|
|
|
"fmt"
|
|
|
|
)
|
|
|
|
func main() {
|
|
|
|
//Generate Private Key
|
|
|
|
pvk, err := rsa.GenerateKey(rand.Reader, 1024)
|
|
|
|
if err != nil {
|
|
|
|
fmt.Println(err)
|
|
|
|
}
|
|
|
|
fmt.Println(pvk)
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}}
|
2017-12-28 06:54:10 +00:00
|
|
|
|
|
|
|
// SampleCodeG404 - weak random number
|
|
|
|
SampleCodeG404 = []CodeSample{
|
2018-09-28 09:42:25 +01:00
|
|
|
{[]string{`
|
2017-12-28 06:54:10 +00:00
|
|
|
package main
|
|
|
|
import "crypto/rand"
|
|
|
|
func main() {
|
|
|
|
good, _ := rand.Read(nil)
|
|
|
|
println(good)
|
2020-06-15 11:26:56 +01:00
|
|
|
}`}, 0, gosec.NewConfig()},
|
|
|
|
{[]string{`
|
2017-12-28 06:54:10 +00:00
|
|
|
package main
|
|
|
|
import "math/rand"
|
|
|
|
func main() {
|
|
|
|
bad := rand.Int()
|
|
|
|
println(bad)
|
2020-06-15 11:26:56 +01:00
|
|
|
}`}, 1, gosec.NewConfig()},
|
|
|
|
{[]string{`
|
2017-12-28 06:54:10 +00:00
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"crypto/rand"
|
|
|
|
mrand "math/rand"
|
|
|
|
)
|
|
|
|
func main() {
|
|
|
|
good, _ := rand.Read(nil)
|
|
|
|
println(good)
|
2020-06-15 11:10:08 +01:00
|
|
|
bad := mrand.Int31()
|
|
|
|
println(bad)
|
2020-06-15 11:26:56 +01:00
|
|
|
}`}, 1, gosec.NewConfig()},
|
|
|
|
{[]string{`
|
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"math/rand"
|
|
|
|
)
|
|
|
|
func main() {
|
2021-01-01 19:45:13 +00:00
|
|
|
gen := rand.New(rand.NewSource(10))
|
2020-06-15 11:26:56 +01:00
|
|
|
bad := gen.Int()
|
|
|
|
println(bad)
|
|
|
|
}`}, 1, gosec.NewConfig()},
|
|
|
|
{[]string{`
|
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"math/rand"
|
|
|
|
)
|
|
|
|
func main() {
|
|
|
|
bad := rand.Intn(10)
|
|
|
|
println(bad)
|
2020-06-15 11:10:08 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}}
|
2017-12-28 06:54:10 +00:00
|
|
|
|
2020-06-29 12:21:15 +01:00
|
|
|
// SampleCodeG501 - Blocklisted import MD5
|
2017-12-28 06:54:10 +00:00
|
|
|
SampleCodeG501 = []CodeSample{
|
2018-09-28 09:42:25 +01:00
|
|
|
{[]string{`
|
2017-12-28 06:54:10 +00:00
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"crypto/md5"
|
|
|
|
"fmt"
|
|
|
|
"os"
|
|
|
|
)
|
|
|
|
func main() {
|
|
|
|
for _, arg := range os.Args {
|
|
|
|
fmt.Printf("%x - %s\n", md5.Sum([]byte(arg)), arg)
|
|
|
|
}
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}}
|
2017-12-28 06:54:10 +00:00
|
|
|
|
2020-06-29 12:21:15 +01:00
|
|
|
// SampleCodeG502 - Blocklisted import DES
|
2017-12-28 06:54:10 +00:00
|
|
|
SampleCodeG502 = []CodeSample{
|
2018-09-28 09:42:25 +01:00
|
|
|
{[]string{`
|
2017-12-28 06:54:10 +00:00
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"crypto/cipher"
|
|
|
|
"crypto/des"
|
|
|
|
"crypto/rand"
|
|
|
|
"encoding/hex"
|
|
|
|
"fmt"
|
|
|
|
"io"
|
|
|
|
)
|
|
|
|
func main() {
|
|
|
|
block, err := des.NewCipher([]byte("sekritz"))
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
plaintext := []byte("I CAN HAZ SEKRIT MSG PLZ")
|
|
|
|
ciphertext := make([]byte, des.BlockSize+len(plaintext))
|
|
|
|
iv := ciphertext[:des.BlockSize]
|
|
|
|
if _, err := io.ReadFull(rand.Reader, iv); err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
stream := cipher.NewCFBEncrypter(block, iv)
|
|
|
|
stream.XORKeyStream(ciphertext[des.BlockSize:], plaintext)
|
|
|
|
fmt.Println("Secret message is: %s", hex.EncodeToString(ciphertext))
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}}
|
2017-12-28 06:54:10 +00:00
|
|
|
|
2020-06-29 12:21:15 +01:00
|
|
|
// SampleCodeG503 - Blocklisted import RC4
|
2018-09-28 09:42:25 +01:00
|
|
|
SampleCodeG503 = []CodeSample{{[]string{`
|
2017-12-28 06:54:10 +00:00
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"crypto/rc4"
|
|
|
|
"encoding/hex"
|
|
|
|
"fmt"
|
|
|
|
)
|
|
|
|
func main() {
|
|
|
|
cipher, err := rc4.NewCipher([]byte("sekritz"))
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
plaintext := []byte("I CAN HAZ SEKRIT MSG PLZ")
|
|
|
|
ciphertext := make([]byte, len(plaintext))
|
|
|
|
cipher.XORKeyStream(ciphertext, plaintext)
|
|
|
|
fmt.Println("Secret message is: %s", hex.EncodeToString(ciphertext))
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}}
|
2017-12-28 06:54:10 +00:00
|
|
|
|
2020-06-29 12:21:15 +01:00
|
|
|
// SampleCodeG504 - Blocklisted import CGI
|
2018-09-28 09:42:25 +01:00
|
|
|
SampleCodeG504 = []CodeSample{{[]string{`
|
2017-12-28 06:54:10 +00:00
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"net/http/cgi"
|
|
|
|
"net/http"
|
|
|
|
)
|
|
|
|
func main() {
|
|
|
|
cgi.Serve(http.FileServer(http.Dir("/usr/share/doc")))
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}}
|
2020-06-29 12:21:15 +01:00
|
|
|
// SampleCodeG505 - Blocklisted import SHA1
|
2018-08-08 15:38:57 +01:00
|
|
|
SampleCodeG505 = []CodeSample{
|
2018-09-28 09:42:25 +01:00
|
|
|
{[]string{`
|
2018-08-08 15:38:57 +01:00
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"crypto/sha1"
|
|
|
|
"fmt"
|
|
|
|
"os"
|
|
|
|
)
|
|
|
|
func main() {
|
|
|
|
for _, arg := range os.Args {
|
|
|
|
fmt.Printf("%x - %s\n", sha1.Sum([]byte(arg)), arg)
|
|
|
|
}
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}}
|
2019-12-19 17:39:33 +00:00
|
|
|
|
2020-05-20 16:17:44 +01:00
|
|
|
// SampleCodeG601 - Implicit aliasing over range statement
|
|
|
|
SampleCodeG601 = []CodeSample{
|
|
|
|
{[]string{`
|
|
|
|
package main
|
2019-12-19 17:39:33 +00:00
|
|
|
|
|
|
|
import "fmt"
|
|
|
|
|
|
|
|
var vector []*string
|
|
|
|
func appendVector(s *string) {
|
|
|
|
vector = append(vector, s)
|
|
|
|
}
|
|
|
|
|
|
|
|
func printVector() {
|
|
|
|
for _, item := range vector {
|
|
|
|
fmt.Printf("%s", *item)
|
|
|
|
}
|
|
|
|
fmt.Println()
|
|
|
|
}
|
|
|
|
|
2021-01-04 08:19:01 +00:00
|
|
|
func foo() (int, **string, *string) {
|
2019-12-19 17:39:33 +00:00
|
|
|
for _, item := range vector {
|
|
|
|
return 0, &item, item
|
|
|
|
}
|
2021-01-04 08:19:01 +00:00
|
|
|
return 0, nil, nil
|
2019-12-19 17:39:33 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
for _, item := range []string{"A", "B", "C"} {
|
|
|
|
appendVector(&item)
|
|
|
|
}
|
|
|
|
|
|
|
|
printVector()
|
|
|
|
|
|
|
|
zero, c_star, c := foo()
|
2021-01-01 19:45:13 +00:00
|
|
|
fmt.Printf("%d %v %s", zero, c_star, c)
|
2020-05-20 16:17:44 +01:00
|
|
|
}`,
|
|
|
|
}, 1, gosec.NewConfig()},
|
|
|
|
{[]string{`
|
|
|
|
// see: github.com/securego/gosec/issues/475
|
|
|
|
package main
|
|
|
|
import (
|
|
|
|
"fmt"
|
|
|
|
)
|
|
|
|
func main() {
|
|
|
|
sampleMap := map[string]string{}
|
|
|
|
sampleString := "A string"
|
|
|
|
for sampleString, _ = range sampleMap {
|
|
|
|
fmt.Println(sampleString)
|
|
|
|
}
|
|
|
|
}`}, 0, gosec.NewConfig()},
|
|
|
|
}
|
2019-12-19 17:39:33 +00:00
|
|
|
|
2020-05-20 16:17:44 +01:00
|
|
|
// SampleCodeBuildTag - G601 build tags
|
|
|
|
SampleCodeBuildTag = []CodeSample{{[]string{`
|
2019-04-30 15:57:32 +01:00
|
|
|
// +build tag
|
2018-04-20 00:45:04 +01:00
|
|
|
package main
|
|
|
|
func main() {
|
|
|
|
fmt.Println("no package imported error")
|
2019-06-25 09:29:19 +01:00
|
|
|
}`}, 1, gosec.NewConfig()}}
|
2020-01-15 15:56:50 +00:00
|
|
|
|
|
|
|
// SampleCodeCgo - Cgo file sample
|
|
|
|
SampleCodeCgo = []CodeSample{{[]string{`
|
|
|
|
package main
|
|
|
|
|
|
|
|
import (
|
|
|
|
"fmt"
|
|
|
|
"unsafe"
|
|
|
|
)
|
|
|
|
|
|
|
|
/*
|
|
|
|
#include <stdlib.h>
|
|
|
|
#include <stdio.h>
|
|
|
|
#include <string.h>
|
|
|
|
|
|
|
|
int printData(unsigned char *data) {
|
|
|
|
return printf("cData: %lu \"%s\"\n", (long unsigned int)strlen(data), data);
|
|
|
|
}
|
|
|
|
*/
|
|
|
|
import "C"
|
|
|
|
|
|
|
|
func main() {
|
|
|
|
// Allocate C data buffer.
|
|
|
|
width, height := 8, 2
|
|
|
|
lenData := width * height
|
|
|
|
// add string terminating null byte
|
|
|
|
cData := (*C.uchar)(C.calloc(C.size_t(lenData+1), C.sizeof_uchar))
|
|
|
|
|
|
|
|
// When no longer in use, free C allocations.
|
|
|
|
defer C.free(unsafe.Pointer(cData))
|
|
|
|
|
|
|
|
// Go slice reference to C data buffer,
|
|
|
|
// minus string terminating null byte
|
|
|
|
gData := (*[1 << 30]byte)(unsafe.Pointer(cData))[:lenData:lenData]
|
|
|
|
|
|
|
|
// Write and read cData via gData.
|
|
|
|
for i := range gData {
|
|
|
|
gData[i] = '.'
|
|
|
|
}
|
|
|
|
copy(gData[0:], "Data")
|
|
|
|
gData[len(gData)-1] = 'X'
|
|
|
|
fmt.Printf("gData: %d %q\n", len(gData), gData)
|
|
|
|
C.printData(cData)
|
|
|
|
}
|
|
|
|
`}, 0, gosec.NewConfig()}}
|
2017-07-19 22:17:00 +01:00
|
|
|
)
|