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

LIBKML writing: validate longitude, latitude to be in range #10491

Merged
merged 1 commit into from
Aug 10, 2024

Conversation

rouault
Copy link
Member

@rouault rouault commented Jul 24, 2024

  • For longitudes, accept in range [-540,540] and normalize to [-180,180]. Error out beyond
  • For latitudes, accept in range [-90,90]. Error out beyond

Fixes #10483

@rouault rouault added this to the 3.10.0 milestone Jul 24, 2024
@coveralls
Copy link
Collaborator

coveralls commented Jul 24, 2024

Coverage Status

coverage: 69.304% (+0.009%) from 69.295%
when pulling d8f6998 on rouault:fix_10483
into 3122857 on OSGeo:master.

- For longitudes, accept in range [-540,540] and normalize to [-180,180].
  Error out beyond
- For latitudes, accept in range [-90,90]. Error out beyond

Fixes OSGeo#10483
@rouault rouault merged commit 6c19088 into OSGeo:master Aug 10, 2024
35 checks passed
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.

LIBKML: do not produce invalid KML
2 participants