Opened 15 years ago

Closed 5 years ago

#2905 closed defect (wontfix)

Need to handle reak geokey in some cases

Reported by: esrixz Owned by: warmerdam
Priority: normal Milestone: closed_because_of_github_migration
Component: default Version: unspecified
Severity: normal Keywords:
Cc:

Description

Change made in ReadKey(…) in geo_new.c Changes removed

else if (offset + count > tempData->tk_asciiParamsLength)

return (0);

In case tempData->tk_asciiParamsLength is 0 but offset + count > 0, We need to read the geo key

Attachments (1)

geo_new.c (9.1 KB ) - added by esrixz 15 years ago.
fixed code

Download all attachments as: .zip

Change History (5)

by esrixz, 15 years ago

Attachment: geo_new.c added

fixed code

comment:1 by Jukka Rahkonen, 9 years ago

No reaction to the patch ever. Is is useful or should we close the ticket?

comment:2 by Jukka Rahkonen, 9 years ago

It would be nice to know what issue this fix is made to fix. Who understands the meaning of this:

"In case tempData->tk_asciiParamsLength is 0 but offset + count > 0, We need to read the geo key"

comment:3 by Even Rouault, 9 years ago

Looks like a safety/security bug fix in case of corrupted/hostile file

comment:4 by Even Rouault, 5 years ago

Milestone: closed_because_of_github_migration
Resolution: wontfix
Status: newclosed

This ticket has been automatically closed because Trac is no longer used for GDAL bug tracking, since the project has migrated to GitHub. If you believe this ticket is still valid, you may file it to https://github.com/OSGeo/gdal/issues if it is not already reported there.

Note: See TracTickets for help on using tickets.