tree e51014e33208a59c54bbe4821698032ee143c7cd
parent 5a2505f3dbf049b47a091040906b7b2856339099
author Dmitri Shuralyov <dmitshur@golang.org> 1555711389 -0400
committer Dmitri Shuralyov <dmitshur@golang.org> 1555712478 +0000

database: use db.DeleteIndex instead of calling deleteIndex directly

The db.DeleteIndex method checks that db.RemoteClient is not nil
before dereferencing it. The direct call to deleteIndex did not,
and therefore panicked when db.RemoteClient was nil.

This makes Database.Block method consistent with Database.Put and
Database.Delete, which also use the db.DeleteIndex method.

With this change, go test ./database passes.

Change-Id: I90514bc1e855e125a9db57a93d2d7a88bce67e2e
Reviewed-on: https://go-review.googlesource.com/c/gddo/+/172961
Reviewed-by: Tuo Shan <shantuo@google.com>