DeepSource
Dashboard Resources Pricing Discover Directory Log in

Run your first analysis.

Find thousands of code security and quality issues in your codebase, before they end up in production.

Start now
Go

Go

Made by DeepSource
Use Analyzer

Potential issue in Query call GO-W4010

Bug risk
Major

(*database/sql).Exec should be used in place of (*database/sql).Query when the result returned is not required as ignoring the result may lead to a connection leak.

Bad practice

package main

import (
    "database/sql"
)

func foo(db *sql.DB) {
    _, err := db.Query(...) // use Exec when the returned result is not needed
}

Recommended

package main

import (
    "database/sql"
)

func foo(db *sql.DB) {
    _, err := db.Query(...) // use Exec when the returned result is not needed
}