compare end of key file before reading it #565

Closed
wants to merge 1 commit into
from

Projects

None yet

2 participants

@vogxn
Contributor
vogxn commented Jul 29, 2015

have been running this change for a week and haven't encountered the IndexError again.

@bitprophet
Member

This is re #547, presumably? thanks!

@bitprophet
Member

Seems like a reasonable approach, if you can write a quick test that fails w/o this change and succeeds with it in place, that'd be awesome. Thanks!

@bitprophet bitprophet added this to the 1.13.4 / 1.14.3 / 1.15.3 milestone Jul 30, 2015
@bitprophet bitprophet modified the milestone: 1.15.3, 1.16 Oct 2, 2015
@bitprophet bitprophet modified the milestone: 1.15.4, 1.16 Nov 2, 2015
@bitprophet bitprophet added a commit that closed this pull request Nov 3, 2015
@bitprophet bitprophet Changelog closes #565 fcacbe4
@bitprophet bitprophet closed this in fcacbe4 Nov 3, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment