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.

Sign up for GitHub

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

Jump to bottom

Segmentation fault in Robin inverse method on NaN input #463

Closed
rouault opened this issue Dec 12, 2016 · 1 comment
Closed

Segmentation fault in Robin inverse method on NaN input #463

rouault opened this issue Dec 12, 2016 · 1 comment
Milestone
5.0.0

Comments

@rouault
Copy link
Member

rouault commented Dec 12, 2016

When the s_inverse() method is called with a lp.phi = NaN value, i = floor(lp.phi * NODES); evaluates to INT_MIN, which causes an out-of-bounds read.

Was found with American Fuzzy Lop on a gdalinfo fuzzing session on a geotiff input.
@rouault
Copy link
Member Author

rouault commented Dec 12, 2016

Fixed per bc7453d

@rouault rouault closed this as completed Dec 12, 2016
rouault referenced this issue Dec 12, 2016
@rouault
PJ_robin: avoid out-of-bounds read on NaN values
bc7453d 
Found with AFL on gdalinfo on s_inverse(). s_forward() might also have the
same issue, so fixing that too.
@kbevers kbevers added this to the 4.9.4 milestone Dec 12, 2016
@kbevers kbevers modified the milestones: 5.0.0-b, 5.0.0 Feb 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
5.0.0
Development

No branches or pull requests
2 participants
@rouault @kbevers