Question

在struct panics invalid memory address or nil pointer dereference的字段上调用atomic.AddInt64，但在我们重新排列字段顺序时却不行;为什么呢？

使用此类型：

type CountHandler struct {
    c     *RequestContext
    count int64
}

此时调用atomic.AddInt64(&countHandler.count, 1)（字段c为零）恐慌。但是，当我们将其重写为：

时

type CountHandler struct {
    count int64
    c     *RequestContext
}

错误消失。

我想它应该是这样，因为Go以顺序方式将数据保存在内存中并且达到nil值会破坏此序列（字节）;但我想知道为什么会这样，因为指针应该有一个固定的大小nil或其他值。

这是Windows上的Go x86 1.4.2＆amp;完整的错误消息是：

2015/02/23 12:56:44 http: panic serving [::1]:51886: runtime error: invalid memory address or nil pointer dereference
goroutine 5 [running]:
net/http.func·011()
        c:/go/src/net/http/server.go:1130 +0xa8
sync/atomic.AddUint64(0x731144, 0x1, 0x0, 0x0, 0x263168)
        c:/go/src/sync/atomic/asm_386.s:118 +0xc
main.(*CountHandler).ServeHTTP(0x731140, 0x263180, 0x122f6380, 0x122f62a0)
        C:/Workshop/Devox/Workshop-Go/src/geoho/web/app/app.go:62 +0x42
github.com/julienschmidt/httprouter.func·001(0x263180, 0x122f6380, 0x122f62a0, 0x0, 0x0, 0x0)
        C:/Workshop/Devox/Workshop-Go/src/github.com/julienschmidt/httprouter/router.go:232 +0x4c
github.com/julienschmidt/httprouter.(*Router).ServeHTTP(0x122d5d20, 0x263180, 0x122f6380, 0x122f62a0)
        C:/Workshop/Devox/Workshop-Go/src/github.com/julienschmidt/httprouter/router.go:298 +0x141
net/http.serverHandler.ServeHTTP(0x122d2280, 0x263180, 0x122f6380, 0x122f62a0)
        c:/go/src/net/http/server.go:1703 +0x145
net/http.(*conn).serve(0x122e01e0)
        c:/go/src/net/http/server.go:1204 +0x9d8
created by net/http.(*Server).Serve
        c:/go/src/net/http/server.go:1751 +0x2ce

整个源代码是（这段代码错了，我只是在研究alice）：

package main

import (
    "fmt"
    "github.com/julienschmidt/httprouter"
    "github.com/justinas/alice"
    "net/http"

    "os"
    "sync/atomic"
)

// play with alice
func main() {
    c1 := alice.New(Counter, Texter).Then(nil)

    router := httprouter.New()
    router.Handler("GET", "/", c1)
    router.GET("/kill", kill)

    http.ListenAndServe(":27007", router)
}

func kill(rw http.ResponseWriter, rq *http.Request, pl httprouter.Params) {
    os.Exit(0)
}

var ch CountHandler

// constructors:

func Counter(h http.Handler) http.Handler {
    return &ch
}

func Texter(h http.Handler) http.Handler {
    var t TextHandler
    switch x := h.(type) {
    case *CountHandler:
        t.c = x.c
        t.text = fmt.Sprintf("called so far %d", atomic.LoadInt64(&x.count))
    }
    return &t
}

// handlers:

type RequestContext struct {
    val int
}

type CountHandler struct {
    c     *RequestContext
    count int64
}

func (c *CountHandler) ServeHTTP(rw http.ResponseWriter, req *http.Request) {
    atomic.AddInt64(&c.count, 1)
}

type TextHandler struct {
    c    *RequestContext
    text string
}

func (t *TextHandler) ServeHTTP(rw http.ResponseWriter, req *http.Request) {
    rw.Write([]byte(t.text))
}

Answer 1

在第一种情况下，原子更新的字段不正确aligned导致故障。

在ARM和x86-32上，调用者负责安排原子访问的64位字的64位对齐。全局变量或分配的结构或片中的第一个字可以依赖于64位对齐。

Answer 2

如果您偶然发现此错误，这里有一些技巧可以解决该问题：

如OP中所述，最简单的方法是将所有64位原子值放在结构的顶部：

c := struct {
    val   int64 // pos 0
    val2  int64 // pos 8
    valid bool  // pos 16
}{val2: 1}
fmt.Println(atomic.AddInt64(&c.val2, 1))

如果出于某种原因，您不想将此字段放在顶部，则始终可以在64位字段上方放置_ [4]byte，以确保正确填充该字段。

c := struct {
    val   int64   // pos 0
    valid bool    // pos 8
    _     [4]byte // pos 9; compiler adds additional [3]byte at pos 13 for alignment
    val2  int64   // pos 16, correctly aligned
}{val2: 1}
fmt.Println(atomic.AddInt64(&c.val2, 1)) // => 2

请注意，如果字段已经对齐，则此操作将无效；相反，如果以前没有惊慌，现在就会惊慌。

c := struct {
    val   int64   // pos 0
    _     [4]byte // pos 8; compiler adds no padding
    val2  int64   // pos 12, not a multiple of 8!
}{val2: 1}
fmt.Println(atomic.AddInt64(&c.val2, 1)) // => runtime error: invalid memory address [...]

您还可以依靠以下行为：64位元素片中的第一个元素将正确对齐：

c := struct {
    val   int64
    valid bool
    val2  []int64
}{val2: []int64{1}}
fmt.Println(atomic.AddInt64(&c.val2[0], 1))

请注意，这不适用于数组，因为它们的值直接存储在结构中，而不是像切片数据那样存储在堆中。

您可以实现的最后一个技巧是在结构中将字段声明为指向int64的指针；如果它指向的int64是对齐的，那么它将平稳运行。

c := struct {
    val   int64
    valid bool
    val2  *int64
}{val2: new(int64)}
fmt.Println(atomic.AddInt64(c.val2, 1))

如果您不想被sync/atomic弄脏，请记住sync.Mutex是比处理原子更清洁，更易理解的解决方案。

atomic.AddInt64导致无效的内存地址或nil指针取消引用

2 个答案: