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

segfault when reading features with no geometry #41

Closed
adrien-berchet opened this issue Jun 1, 2016 · 4 comments
Closed

segfault when reading features with no geometry #41

adrien-berchet opened this issue Jun 1, 2016 · 4 comments

Comments

@adrien-berchet
Copy link
Contributor

Hi,
I had an issue when trying to read some shapefiles in which some features have no geometry (feature->GetGeometryRef() returns 0). It raised a segfault in this case.
I can't make a pull request but just adding this to line 160 solves this issue:
if (!feature->GetGeometryRef()) continue;
Regards

@hugoledoux
Copy link
Member

thanks. But from which branch is this? And out of curiosity, why can't you make a pull request?

@adrien-berchet
Copy link
Contributor Author

Ups sorry, I meant line 100, not 160. And I did not say it is in the IOWorker.cpp file of the master branch.

For the pull request I have a permission issue:
ERROR: Permission to tudelft3d/pprepair.git denied to adrien-berchet-forcity.

@hugoledoux
Copy link
Member

You don't have write permission on our code, that's normal. Create a fork of our code, modify it and the create a pull request https://help.github.com/articles/creating-a-pull-request/

@adrien-berchet
Copy link
Contributor Author

adrien-berchet commented Jun 3, 2016

Indeed, sorry!
I opened a PR for that.

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

No branches or pull requests

2 participants