tree 8d3a21c5161e23c5b1973308d4ff4c7c538607ea
parent 8be58fba63529bbb3f25f43fe10179356ca2497e
author Sander van Harmelen <sander@vanharmelen.nl> 1565214582 +0200
committer Rebecca Stambler <rstambler@golang.org> 1565217307 +0000

x/tools/cmd/gopls: prevent nil pointer dereferences

The `files` slice is used twice. First it's used to get all results from `ph.Parse`, and then it's reused to filter all `nil` values (which may have been returned by the `ph.Parse` method).

After the loop to "filter" out all the `nil` values, we also need to strip the remaining values.

I also changed the ordering so that we first check the errors and
only then perform this loop. That way the code will return earlier
when the context was canceled.

Partially fixes #33531 by prevention the panic reported in that issue.

Change-Id: I09478e765adcd0384ec4745921eb5c5aea405ef2
Reviewed-on: https://go-review.googlesource.com/c/tools/+/189397
Reviewed-by: Rebecca Stambler <rstambler@golang.org>
Run-TryBot: Rebecca Stambler <rstambler@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>