Skip to content
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

Fix ASAN issue with SQLite3VFS class (fixes #1901) #1902

Merged
merged 1 commit into from Feb 2, 2020

Conversation

rouault
Copy link
Member

@rouault rouault commented Jan 31, 2020

@nyalldawson I'm confident this should fix the ASAN error, but can you perhaps give this a try ? My ASAN (and the one we use on Travis) is probably too old to detect this class of issue

@nyalldawson
Copy link
Contributor

@nyalldawson nyalldawson commented Jan 31, 2020

I'll give it a try, thanks!

@nyalldawson
Copy link
Contributor

@nyalldawson nyalldawson commented Feb 2, 2020

Yep, can confirm that this fixes it. Thanks!

@rouault
Copy link
Member Author

@rouault rouault commented Feb 2, 2020

Cool. Thanks for the report and testing. Merging

@rouault rouault merged commit 6aadc40 into OSGeo:master Feb 2, 2020
4 checks passed
@kbevers kbevers added this to the 7.0.0 milestone Feb 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants