New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Avoid negative integer overflow when iccOffset > chunkLength #795
Avoid negative integer overflow when iccOffset > chunkLength #795
Conversation
Codecov Report
@@ Coverage Diff @@
## 0.27-maintenance #795 +/- ##
====================================================
+ Coverage 62.67% 62.67% +<.01%
====================================================
Files 156 156
Lines 21523 21524 +1
====================================================
+ Hits 13490 13491 +1
Misses 8033 8033
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, I've added some minor suggestions and one info that needn't be addressed now.
6710899
to
cffac6e
Compare
b0b81ab
to
709830d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@D4N the two changes you requested were addressed, so I guess you can approve now 😉
Lgtm, please merge this! (Cannot approve right now) |
Fixes #790.