googleapis / go-sql-spanner

Google Cloud Spanner driver for Go's database/sql package.
Apache License 2.0
104 stars 19 forks source link

Panic in TestBufferWriteMutations #254

Closed olavloite closed 3 months ago

olavloite commented 3 months ago
--- FAIL: TestBufferWriteMutations (0.01s)
panic: runtime error: invalid memory address or nil pointer dereference [recovered]
    panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0xe855cc]

goroutine 142 [running]:
testing.tRunner.func1.2({0x10bcae0, 0x1ec1760})
    /opt/hostedtoolcache/go/1.21.11/x64/src/testing/testing.go:1545 +0x238
testing.tRunner.func1()
    /opt/hostedtoolcache/go/1.21.11/x64/src/testing/testing.go:1548 +0x397
panic({0x10bcae0?, 0x1ec1760?})
    /opt/hostedtoolcache/go/1.21.11/x64/src/runtime/panic.go:914 +0x21f
cloud.google.com/go/spanner.(*sessionPool).take(0x0, {0x14b0880, 0x1f20800})
    /home/runner/go/pkg/mod/cloud.google.com/go/spanner@v1.63.0/session.go:1040 +0x8c
cloud.google.com/go/spanner.(*ReadWriteTransaction).begin(0xc0072c4f00, {0x14b0880, 0x1f20800})
    /home/runner/go/pkg/mod/cloud.google.com/go/spanner@v1.63.0/transaction.go:1501 +0x336
cloud.google.com/go/spanner.NewReadWriteStmtBasedTransactionWithOptions({0x14b0880, 0x1f20800}, 0xc00711b320, {{0x0, 0x0}, {0x0, 0x0}, 0x0, 0x0, 0x0})
    /home/runner/go/pkg/mod/cloud.google.com/go/spanner@v1.63.0/transaction.go:1757 +0x359
github.com/googleapis/go-sql-spanner.(*conn).BeginTx(0xc0013e1ce0, {0x14b0880?, 0x1f20800}, {0xc000727c30?, 0x58?})
    /home/runner/work/go-sql-spanner/go-sql-spanner/driver.go:976 +0x4e6
database/sql.ctxDriverBegin({0x14b0880, 0x1f20800}, 0xc000727e48, {0x14ae0e8, 0xc0013e1ce0})
    /opt/hostedtoolcache/go/1.21.11/x64/src/database/sql/ctxutil.go:104 +0x76
database/sql.(*DB).beginDC.func1()
    /opt/hostedtoolcache/go/1.21.11/x64/src/database/sql/sql.go:1868 +0xb8
database/sql.withLock({0x14abc48, 0xc00a5f0900}, 0xc000727d40)
    /opt/hostedtoolcache/go/1.21.11/x64/src/database/sql/sql.go:3502 +0x82
database/sql.(*DB).beginDC(0xc000212b60, {0x14b0880, 0x1f20800}, 0xc00a5f0900, 0xc0095504f0, 0x14b0880?)
    /opt/hostedtoolcache/go/1.21.11/x64/src/database/sql/sql.go:1864 +0xc5
database/sql.(*Conn).BeginTx(0xc0000b58c0, {0x14b0880, 0x1f20800}, 0x[52](https://github.com/googleapis/go-sql-spanner/actions/runs/9481858779/job/26125489973?pr=253#step:5:53)b5a0?)
    /opt/hostedtoolcache/go/1.21.11/x64/src/database/sql/sql.go:2077 +0x5f
github.com/googleapis/go-sql-spanner.TestBufferWriteMutations(0xc0003b6ea0)
    /home/runner/work/go-sql-spanner/go-sql-spanner/driver_with_mockserver_test.go:1384 +0x111
testing.tRunner(0xc0003b6ea0, 0x1377ac8)
    /opt/hostedtoolcache/go/1.21.11/x64/src/testing/testing.go:1595 +0xff
created by testing.(*T).Run in goroutine 1
    /opt/hostedtoolcache/go/1.21.11/x64/src/testing/testing.go:1648 +0x3ad
exit status 2
FAIL    github.com/googleapis/go-sql-spanner    0.[56](https://github.com/googleapis/go-sql-spanner/actions/runs/9481858779/job/26125489973?pr=253#step:5:57)7s
olavloite commented 3 months ago

Duplicate of #262