Fix big-endian test failures and use-after-free issues in error handling #2408
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
On little-endian hosts, `int` here caused incorrect results at positions beyond the range of int. On big-endian hosts, it caused incorrect results at all positions.
Calling bgzf_close() (and hts_close()) frees the file pointer, so the errcode field cannot be accessed afterwards. Report errors as per errno instead as is done in vcfconvert.c. Also report the right file pointer's errcode after bgzf_write() failure.
|
Thank you |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The incorrect type in
mpileup_get_val()in plugins/vrfs.c caused incorrect results and test failures on big-endian machines, asposwas essentially always set to 0 (being the high-order bits of theint64_t).Accessing
fileptr->errcodeafter closing thefileptris invalid as generally the file structure will have been deallocated. This replaces that withstrerror(errno), as is already done in vcfconvert.c. Also report the right file pointer'serrcodeafterbgzf_write()failure.And some other Makefile fixes and other miscellanea.