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

Not initializing tempString2 in m_sax_parser.f90 #31

Open
bcorfman opened this issue Sep 12, 2013 · 1 comment
Open

Not initializing tempString2 in m_sax_parser.f90 #31

bcorfman opened this issue Sep 12, 2013 · 1 comment

Comments

@bcorfman
Copy link

In 4.1.2, you have the following code on line 1515 of m_sax_parser.f90:
if (associated(tempString2)) deallocate(tempString2)

However, tempString2 is nowhere initialized to null. This results in the program crashing whenever you call saxParse twice within the same run. (In my case, I was calling it via parseFile.)
andreww added a commit that referenced this issue Sep 13, 2013
@andreww
Make sure we start off with null() temp vstrings
f58d44d 
See issue #31. This should avoid the reported crash,
but the tests I've tried don't trip the segfault.
But anyway, add two tests for sax with double
parsing.
@andreww
Copy link
Owner

andreww commented Sep 13, 2013

This should now be fixed in the development version in master on github. I would be grateful if you can test this as I've not actually been able to get the parser to crash.

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
No milestone
Development

No branches or pull requests
2 participants
@andreww @bcorfman