On Thu, 21 Dec 2017 20:15:00 -0500, "Ted Unangst" wrote: > Mark Karpilovskij wrote: > > If only a single call to SipHash_Update is performed or if the size of > > processed data is a multiple of sizeof(ctx->buf), this bug does nothing. > > However when we performed several updates of various lengths, the data > > written to ctx->buf were too long and the call to memcpy overwrote other > > data, which led to various unexpected behavior. > > I concur. Here's a diff for review.
Looks correct since len is the remainder. OK millert@ - todd
