reftable: binary search over restart counter is broken
The binary search over the restart counter is subtly broken such that we always end up doing a linear search across the whole block. This is because the function passed to binsearch()
is supposed to return a boolean value, but we return the value of strbuf_strcmp()
directly.