Mysql: * sql.DB無効なメモリアドレスまたはnilポインタの逆参照

次の情報が必要です。

• Go-MySQL-Driverバージョン（またはgit SHA）
• Goバージョン（コンソールでgo versionを実行）

goバージョンgo1.1.2darwin / amd64
Go-MySQL-Driverは最新バージョンです

昨日リリースしたバグ修正リリースであるv1.0.3ではパニックは発生しないはずだと思います（数分からgo get ）。

しかし、私はさらに調査しています。スタックトレースは私には本当に奇妙に見えます。 パニックは、 stmt.Close （変数mc ）のnil接続によって引き起こされます。この場合、上記のコードでは接続がnilなることはありません。

v1.0.3は、最初にmc == nilチェックするため、少なくともnil-pointerパニックを防ぐことができます。

このエラーを再現できますか？

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 amount =？WHERE id =？"、quantity + 1、id）はエラーです

以前にそのような状況を見つけたことがないようです、私のシステムはOS X10.9です

row.Scan(&id, &sources, &quantity, &spider_date)がエラーを返すかどうかを確認します。
v1.0.3を試しましたか？ 私が言ったように、それは少なくともパニックを避けるべきです。

何度か試しましたが、古いバージョンを削除して、もう一度取得します
私はに変更しました：

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

このバグを再現できたと思います。 繰り返しになりますが、このバグはhttps://codereview.appspot.com/14920046 / Go issue 5718 （または＃98、＃142）に関連しています。
stmt.Execは、ドライバーによるエラーを無視し、不良接続をプールに戻します。

v1.0.3には＃143が含まれており、少なくともパニックを回避します。 数時間以内にリリースされるv1.1には、これをもう少し改善する＃151が含まれます。

Goバージョンにhttps://codereview.appspot.com/14920046で手動でパッチを適用するか、この修正が含まれる次の安定版リリースがリリースされるまでstmt.Exec回避することをお勧めします。 それはおそらく2014年6月のGo1.3になるでしょう...

そして、上記のコードへのメモ：
stmt 1回だけ使用する場合は、手動で準備する必要はありません。 db.Exec使用するだけです。ちなみに、これはstmt.Execようにバグがありません。

上記のコードを変更する方法は次のとおりです。

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
}

わかりました、ありがとう！

私もこの問題を抱えています。goバージョンはgo1.7.4 linux/amd64 、go-sql-driverが最新です。 AWSのRDSインスタンスに接続しています。

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

これは関数_getJSONであり、結果のJSONエンコードされた文字列を返します。

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あなたのデータベースはnilです。

なぜそれはゼロでなければならないのですか？ 関数var db *sql.DB外にdb varがあり、メイン関数で接続を開きました

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

次に、 _getJSON()呼び出されます。 資格情報はOKです。 たぶんそれはAWSの問題ですか？ ローカルホストで試してみます。

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

次に、dbはmainのローカル変数であり、使用するグローバル変数_getJSON()ではありません。

完全に再現可能なコードを提供しないと、誰も指摘できません
コードに競合状態がある場合。
go build -raceはあなたの友達です。

@メタン、あなたは正しかった、私はdb, err := [...]を変更しなければならず、 db, err = [...]もエラーを宣言した。 私の悪い、気晴らしのエラー。

同じ問題が発生しました。 これが問題だったことに驚いた。
@danilopolaniによる解決策は

@ danilopolani @メタンあなたたちは私の時間を節約しました

@danilopolaniそうです、助けてくれてありがとう、私は長い間この問題を探してきました、ありがとう

同じ問題が発生しています.. @ danilopolaniソリューションを使用するにはどうすればよいですか？

おかげで@メタン@danilopolani私は同じ間違いをしました、幸運にも私はこの議論を見つけました。

OMG、私は解決策が変数のスコープについてであるとは想像もできませんでした。
どうもありがとうございました。