Fix Qview crashing while viewing GeoTiffs#5898
Conversation
|
The build and test suite have started for your pull request. To view your build log, please reference the build with source version: "PR_5898". Additionally, check the latest "dev" source version to identify existing test failures. Please note that you are not responsible for the test failures that exist on both your PR and the dev branch. |
1 similar comment
|
The build and test suite have started for your pull request. To view your build log, please reference the build with source version: "PR_5898". Additionally, check the latest "dev" source version to identify existing test failures. Please note that you are not responsible for the test failures that exist on both your PR and the dev branch. |
fc11ab8 to
ce13f54
Compare
|
The build and test suite have started for your pull request. To view your build log, please reference the build with source version: "PR_5898". Additionally, check the latest "dev" source version to identify existing test failures. Please note that you are not responsible for the test failures that exist on both your PR and the dev branch. |
1 similar comment
|
The build and test suite have started for your pull request. To view your build log, please reference the build with source version: "PR_5898". Additionally, check the latest "dev" source version to identify existing test failures. Please note that you are not responsible for the test failures that exist on both your PR and the dev branch. |
f6b8818 to
2c144cb
Compare
|
The build and test suite have started for your pull request. To view your build log, please reference the build with source version: "PR_5898". Additionally, check the latest "dev" source version to identify existing test failures. Please note that you are not responsible for the test failures that exist on both your PR and the dev branch. |
2c144cb to
ea4b4e0
Compare
|
The build and test suite have started for your pull request. To view your build log, please reference the build with source version: "PR_5898". Additionally, check the latest "dev" source version to identify existing test failures. Please note that you are not responsible for the test failures that exist on both your PR and the dev branch. |
ea4b4e0 to
883ae60
Compare
|
The build and test suite have started for your pull request. To view your build log, please reference the build with source version: "PR_5898". Additionally, check the latest "dev" source version to identify existing test failures. Please note that you are not responsible for the test failures that exist on both your PR and the dev branch. |
Kelvinrr
left a comment
There was a problem hiding this comment.
tests pass, some warnings for correctness but this PR was opened before that was imeplemented.
Description
The
GdalIoHandlerhas to manage buffers requesting data outside of the image. It creates a smaller buffer, then copies the overlap back into the originally requested buffer. When working with buffers that have a scale less than 1.0, indexing into the buffer can result in going off the end of the buffer.This change ensures that the buffer cannot write off the end of the buffer when copying between scaled buffers.
Related Issue
Gdal Integration
How Has This Been Validated?
Validated through qview locally
Types of changes
Checklist:
Licensing
This project is mostly composed of free and unencumbered software released into the public domain, and we are unlikely to accept contributions that are not also released into the public domain. Somewhere near the top of each file should have these words: