Mysql: * sql.DB adresse mémoire invalide ou déréférence de pointeur nul

Nous avons besoin des informations suivantes:

• Votre version du pilote Go-MySQL (ou git SHA)
• Votre version Go (exécutez go version dans votre console)

aller à la version go1.1.2 darwin / amd64
Go-MySQL-Driver est la dernière version

Je pense que la panique ne devrait pas se produire avec la v1.0.3 , qui est une version de correction de go get -able depuis quelques minutes).

Mais j'étudie plus loin, le stacktrace me semble vraiment étrange. La panique est causée par une nil -connexion dans stmt.Close (variable mc ) où une connexion ne devrait jamais être nil avec le code ci-dessus.

La v1.0.3 peut au moins empêcher la panique du pointeur nul, car elle vérifie d'abord mc == nil .

Pouvez-vous reproduire cette erreur?

panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0x1 addr=0x41 pc=0x5304bc]
goroutine 25466 [running]:
github.com/go-sql-driver/mysql.(*mysqlConn).writeCommandPacketUint32(0x0, 0x305e92619, 0xc22b2fe650, 0x2b8b3c6167c0)
    /Users/jinke/golang/src/github.com/go-sql-driver/mysql/packets.go:327 +0x1c
github.com/go-sql-driver/mysql.(*mysqlStmt).Close(0xc22b87a6f0, 0x0, 0x0)
    /Users/jinke/golang/src/github.com/go-sql-driver/mysql/statement.go:24 +0x46
database/sql.(*driverConn).finalClose(0xc202a42780, 0xc2014fd1c0, 0x2210b5a80)
    /usr/local/go/src/pkg/database/sql/sql.go:Jump {0 autohome 25 44 258 1977 hkyjh 12.68 394 397 2011-9-27  http://k.autohome.com.cn/spec/8288/hkyjh 2013-11-02 14:21:38 6}
285 +0x87
database/sql.func·002(0xc2000ca200, 0xc2000aebd0)
    /usr/local/go/src/pkg/database/sql/sql.go:372 +0x2c
database/sql.(*driverConn).Close(0xc202a42780, 0x32, 0x9)
    /usr/local/go/src/pkg/database/sql/sql.go:278 +0x174
database/sql.(*DB).putConn(0xc2000ca1e0, 0xc202a42780, 0x0, 0x0)
    /usr/local/go/src/pkg/database/sql/sql.go:598 +0x2e2
database/sql.func·004()
    /usr/local/go/src/pkg/database/sql/sql.go:664 +0x41
database/sql.(*DB).exec(0xc2000ca1e0, 0x738330, 0x40, 0x2b8b3c616cb8, 0x2, ...)
    /usr/local/go/src/pkg/database/sql/sql.go:661 +0xb2
database/sql.(*DB).Exec(0xc2000ca1e0, 0x738330, 0x40, 0x2b8b3c616cb8, 0x2, ...)
    /usr/local/go/src/pkg/database/sql/sql.go:650 +0x98
cds_spider/newCar/koubei.AddOk(0x6dee10, 0x8, 0x1, 0x0, 0x0, ...)
    /Users/jinke/golang/src/cds_spider/newCar/koubei/log.go:49 +0x568
cds_spider/newCar/koubei.(*KouBei).Save(0xc20a0a74d0, 0x0, 0x0, 0x0)
    /Users/jinke/golang/src/cds_spider/newCar/koubei/koubei.go:29 +0x3b7
main.func·006()
    /Users/jinke/golang/src/cds_spider/newCar/koubei/main/autohome.go:241 +0x37
created by main.func·007

/Users/jinke/golang/src/cds_spider/newCar/koubei/log.go

func AddOk(source string, count int) error {
    row := dbpool.DB.QueryRow("SELECT * FROM cheduoshao_spider_status.koubei WHERE source=? AND spider_date=?", source, gcode.Date())
    var (
        id, quantity         int
        sources, spider_date string
    )

    row.Scan(&id, &sources, &quantity, &spider_date)

    if id == 0 {
        stmt, err := dbpool.DB.Prepare("INSERT cheduoshao_spider_status.koubei SET source=?, quantity=?, spider_date=?")
        if err != nil {
            return err
        }
        defer stmt.Close()

        _, err = stmt.Exec(source, 1, gcode.Date())
        if err != nil {
            return err
        }
    } else { 
        dbpool.DB.Exec("UPDATE main.koubei SET quantity=? WHERE id=?", quantity+1, id)
    }

    return nil
}

dbpool.DB.Exec ("UPDATE main.koubei SET quantité =? WHERE id =?", quantité + 1, id) IS ERROR

Semble n'avoir jamais trouvé une telle situation auparavant, mon système est OS X 10.9

Vérifiez si row.Scan(&id, &sources, &quantity, &spider_date) renvoie une erreur.
Avez-vous essayé la v1.0.3? Comme je l'ai dit, il faudrait au moins éviter les paniques.

J'ai essayé plusieurs fois de plus, je viens de supprimer l'ancienne version, allez la récupérer
J'ai changé pour:

err := row.Scan(&id, &sources, &quantity, &spider_date)
    if err != nil {
        return err
    }

Je pense que j'ai pu reproduire ce bug. Encore une fois, ce bug est lié à https://codereview.appspot.com/14920046 / Go issue 5718 (ou # 98, # 142).
stmt.Exec ignore les erreurs du pilote et rétablit les mauvaises connexions au pool.

La v1.0.3 inclut # 143, ce qui évite au moins les paniques. La v1.1, qui sortira dans quelques heures, inclura # 151, ce qui améliore un peu plus cela.

Je suggère de patcher manuellement votre version Go avec https://codereview.appspot.com/14920046 ou d'éviter stmt.Exec jusqu'à ce que la prochaine version stable avec ce correctif inclus soit publiée. Ce serait probablement Go 1.3 en juin 2014 ...

Et une note à votre code ci-dessus:
Si vous n'utilisez un stmt qu'une seule fois, il n'y a aucune raison de le préparer manuellement. Utilisez simplement db.Exec , qui n'est d'ailleurs pas buggé comme stmt.Exec .

Voici comment je modifierais votre code ci-dessus:

func AddOk(source string, count int) error {
    var (
        id, quantity         int
        sources, spider_date string
    )
    if err := dbpool.DB.QueryRow("SELECT * FROM cheduoshao_spider_status.koubei WHERE source=? AND spider_date=?", source, gcode.Date()).Scan(&id, &sources, &quantity, &spider_date); err != nil {
        return err
    }

    if id == 0 {
        if _, err =: dbpool.DB.Exec("INSERT cheduoshao_spider_status.koubei SET source=?, quantity=?, spider_date=?", source, 1, gcode.Date()); err != nil {
            return err
        }
    } else { 
        if _, err =: dbpool.DB.Exec("UPDATE main.koubei SET quantity=? WHERE id=?", quantity+1, id); err != nil {
            return err
        }
    }

    return nil
}

D'accord merci!

J'ai aussi ce problème, la version go est go1.7.4 linux/amd64 et go-sql-driver est le dernier. Je me connecte à une instance RDS d'AWS.

runtime error: invalid memory address or nil pointer dereference
/usr/local/go/src/runtime/panic.go:458 (0x42c653)
    gopanic: reflectcall(nil, unsafe.Pointer(d.fn), deferArgs(d), uint32(d.siz), uint32(d.siz))
/usr/local/go/src/runtime/panic.go:62 (0x42b1ad)
    panicmem: panic(memoryError)
/usr/local/go/src/runtime/sigpanic_unix.go:24 (0x441054)
    sigpanic: panicmem()
/usr/local/go/src/database/sql/sql.go:781 (0x557d0a)
    (*DB).conn: db.mu.Lock()
/usr/local/go/src/database/sql/sql.go:1074 (0x559fbb)
    (*DB).query: ci, err := db.conn(strategy)
/usr/local/go/src/database/sql/sql.go:1062 (0x559dc0)
    (*DB).Query: rows, err = db.query(query, args, cachedOrNewConn)
/home/danilo/Go/src/github.com/DaniloMeritocracy/autocomplete/autocomplete.go:192 (0x402e27)
    _getJSON: rows, err := db.Query(sqlString)
/home/danilo/Go/src/github.com/DaniloMeritocracy/autocomplete/autocomplete.go:130 (0x402b54)
    school: res, err := _getJSON("SELECT * FROM university WHERE LOWER(name) LIKE '" + strings.Replace(term, "'", "\\'", -1) + "%' LIMIT 15")
/home/danilo/Go/src/github.com/DaniloMeritocracy/autocomplete/autocomplete.go:56 (0x403985)
    main.func1: results = school(term)
/home/danilo/Go/src/gopkg.in/gin-gonic/gin.v1/context.go:97 (0x45d53a)
    (*Context).Next: c.handlers[c.index](c)
/home/danilo/Go/src/gopkg.in/gin-gonic/gin.v1/recovery.go:45 (0x46d31a)
    RecoveryWithWriter.func1: c.Next()
/home/danilo/Go/src/gopkg.in/gin-gonic/gin.v1/context.go:97 (0x45d53a)
    (*Context).Next: c.handlers[c.index](c)
/home/danilo/Go/src/gopkg.in/gin-gonic/gin.v1/logger.go:72 (0x46c40f)
    LoggerWithWriter.func1: c.Next()
/home/danilo/Go/src/gopkg.in/gin-gonic/gin.v1/context.go:97 (0x45d53a)
    (*Context).Next: c.handlers[c.index](c)
/home/danilo/Go/src/gopkg.in/gin-gonic/gin.v1/gin.go:284 (0x4638ce)
    (*Engine).handleHTTPRequest: context.Next()
/home/danilo/Go/src/gopkg.in/gin-gonic/gin.v1/gin.go:265 (0x4631b0)
    (*Engine).ServeHTTP: engine.handleHTTPRequest(c)
/usr/local/go/src/net/http/server.go:2202 (0x4e1fdd)
    serverHandler.ServeHTTP: handler.ServeHTTP(rw, req)
/usr/local/go/src/net/http/server.go:1579 (0x4de947)
    (*conn).serve: serverHandler{c.server}.ServeHTTP(w, w.req)
/usr/local/go/src/runtime/asm_amd64.s:2086 (0x45c0f1)
    goexit: BYTE    $0x90   // NOP

Il s'agit de la fonction _getJSON , qui renvoie une chaîne codée en JSON des résultats:

func _getJSON(sqlString string) (string, error) {
    rows, err := db.Query(sqlString)
    if err != nil {
        return "", err
    }
    defer rows.Close()
    columns, err := rows.Columns()
    if err != nil {
        return "", err
    }
    count := len(columns)
    tableData := make([]map[string]interface{}, 0)
    values := make([]interface{}, count)
    valuePtrs := make([]interface{}, count)
    for rows.Next() {
        for i := 0; i < count; i++ {
            valuePtrs[i] = &values[i]
        }
        rows.Scan(valuePtrs...)
        entry := make(map[string]interface{})
        for i, col := range columns {
            var v interface{}
            val := values[i]
            b, ok := val.([]byte)
            if ok {
                v = string(b)
            } else {
                v = val
            }
            entry[col] = v
        }
        tableData = append(tableData, entry)
    }
    jsonData, err := json.Marshal(tableData)
    if err != nil {
        return "", err
    }
    //fmt.Println(string(jsonData))
    return string(jsonData), nil
}

@DaniloPolani Votre db est nul.

Pourquoi devrait-il être nul? J'ai une variable db en dehors des fonctions var db *sql.DB et dans la fonction principale j'ai ouvert la connexion avec

db, err := sql.Open("mysql", "<USERNAME>:<PASSWORD>@tcp(<HOST>:3306)/<DB_NAME>")
    if err != nil {
        panic(err.Error())
    }
    defer db.Close()

Ensuite, il est appelé en _getJSON() . Les informations d'identification sont OK. C'est peut-être un problème AWS? Je vais essayer avec localhost.

db, err := sql.Open("mysql", "<USERNAME>:<PASSWORD>@tcp(<HOST>:3306)/<DB_NAME>")

Ensuite, db est la variable locale de main, et non la variable globale _getJSON() using.

Sans vous fournir un code reproductible complet, personne ne peut le signaler
où votre code a une condition de concurrence.
go build -race est votre ami.

@methane , tu avais raison, j'ai dû changer db, err := [...] avec db, err = [...] déclarant également une erreur. Ma mauvaise erreur de distraction.

Je viens de rencontrer le même problème. Surpris que c'était un problème.
La solution de @danilopolani fonctionne.

@danilopolani @methane vous avez sauvé mon temps

@danilopolani Vous avez raison, merci beaucoup pour votre aide, je cherche ce problème depuis longtemps, merci

rencontrer le même problème .. comment utiliser la solution @danilopolani ?

Merci @methane @danilopolani J'ai fait la même erreur, heureusement j'ai trouvé cette discussion.

OMG, je n'aurais jamais pu imaginer que la solution concernait la portée des variables.
Merci beaucoup les gars.