tree b099e8942724753d0264fa9e6db068f4b1f08250
parent 609630e7ba9e5458bd1968a6e9d5121fa5a2c34f
author Tom Cherry <tomcherry@google.com> 1591044230 -0700
committer Tom Cherry <tomcherry@google.com> 1591047896 -0700

logd: fix bug in FlushTo when requesting exact sequence number

SimpleLogBuffer::FlushTo() attempts to find the iterator matching a
given sequence number, but the logic is wrong and will always skip one
element forward.  This change fixes this and adds a test for the
situation.

This likely contributed to some test instability in the past, but was
identified because subsequent changes that track the start value
closer exacerbated this issue.

Test: existing and new unit tests
Change-Id: Iba2e654e94234693dba20d4747a60bc79d195673