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

postgis_wkt_import_fuzzer is crashing on startup #746

Closed
Dor1s opened this Issue Jul 28, 2017 · 4 comments

Comments

Projects
None yet
3 participants
@Dor1s
Collaborator

Dor1s commented Jul 28, 2017

https://oss-fuzz.com/v2/performance-report/libFuzzer_postgis_wkt_import_fuzzer/libfuzzer_asan_postgis/latest

from the log:

...
Time ran: 0.023100
ParseDictionaryFile: error in line 1
		(
...

The dictionary needs to be update in accordance with documentation regarding dictionaries: https://github.com/google/oss-fuzz/blob/master/docs/new_project_guide.md#dictionaries

tl;dr wrap all elements into double quotes

@Dor1s

This comment has been minimized.

Show comment
Hide comment
@Dor1s

Dor1s Jul 28, 2017

Collaborator

Not sure who is responsible for that, so cc'ing @robe2 @rouault

Collaborator

Dor1s commented Jul 28, 2017

Not sure who is responsible for that, so cc'ing @robe2 @rouault

@rouault

This comment has been minimized.

Show comment
Hide comment
@rouault
Contributor

rouault commented Jul 29, 2017

strk pushed a commit to postgis/postgis that referenced this issue Jul 29, 2017

OSS FUZZ Items must be surrounded by double quote. See ​google/oss-fu…
…zz#746

patch from Even Rouault
Closes #3797

git-svn-id: http://svn.osgeo.org/postgis/trunk@15515 b70326c6-7e19-0410-871a-916f4a2858ee
@robe2

This comment has been minimized.

Show comment
Hide comment
@robe2

robe2 Jul 29, 2017

Contributor

Committed patch.

Contributor

robe2 commented Jul 29, 2017

Committed patch.

@Dor1s

This comment has been minimized.

Show comment
Hide comment
@Dor1s

Dor1s Jul 31, 2017

Collaborator

Thanks @rouault and @robe2 ! I've just checked recent logs, the fuzz target is working well.

Collaborator

Dor1s commented Jul 31, 2017

Thanks @rouault and @robe2 ! I've just checked recent logs, the fuzz target is working well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment