tree a6511b3b83472556e1c290bd15aaec9f90048c83
parent 2a1c06a60797e734ab13afef967c60867445e1e5
author Russ Cox <rsc@golang.org> 1685996669 -0400
committer Russ Cox <rsc@golang.org> 1686054560 +0000

sumdb: handle panic from c.ReadRemote during ReadTiles

The go command tests use a network stack that sometimes
panics during read. This code was assuming a network stack
that returns errors instead. If a tile-reading goroutine panicked,
ReadTiles considered it done with data, err = nil, nil, and then
there was a race between ReadTiles failing with an error about
a suspiciously short tile and the panicking goroutine getting
to run and bring down the entire program.

Remove the race by converting the panic into an error during
ReadTiles.

For a test in golang/go#57001.

Change-Id: I9b18a244238e67c27a15b93f8397bf3ab44b06e6
Reviewed-on: https://go-review.googlesource.com/c/mod/+/501035
Run-TryBot: Russ Cox <rsc@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Michael Matloob <matloob@golang.org>