Fix thread safety issue in `msLoadMapFromString()` #4523

Closed
wants to merge 1 commit into
from

Projects

None yet

4 participants

@homme
Contributor
homme commented Nov 13, 2012

The TLOCK_PARSER thread mutex is now released after the call to msyylex_destroy() which otherwise clobbers global lexer variables that may be in use in another thread.

@homme homme Fix thread safety issue in `msLoadMapFromString()`
The `TLOCK_PARSER` thread mutex is now released after the call to
`msyylex_destroy()` which otherwise clobbers global lexer variables
that may be in use in another thread.
c2805f3
@unicolet
Contributor

looks good to merge to me

@tbonfort tbonfort was assigned Nov 14, 2012
@sdlime
Member
sdlime commented Nov 16, 2012

I'm curious how to merge this into branch-6-2 and master easily... Watching. Steve

@tbonfort tbonfort added a commit that referenced this pull request Dec 11, 2012
@homme @tbonfort homme + tbonfort Fix thread safety issue in msLoadMapFromString (#4523)
The `TLOCK_PARSER` thread mutex is now released after the call to
`msyylex_destroy()` which otherwise clobbers global lexer variables
that may be in use in another thread.

closes #4523
e9e4894
@tbonfort tbonfort added a commit that closed this pull request Dec 11, 2012
@homme @tbonfort homme + tbonfort Fix thread safety issue in msLoadMapFromString (#4523)
The `TLOCK_PARSER` thread mutex is now released after the call to
`msyylex_destroy()` which otherwise clobbers global lexer variables
that may be in use in another thread.

closes #4523
e9e4894
@tbonfort tbonfort closed this in e9e4894 Dec 11, 2012
@mkofahl mkofahl pushed a commit to faegi/mapserver that referenced this pull request Apr 9, 2013
@homme @tbonfort homme + tbonfort Fix thread safety issue in msLoadMapFromString (#4523)
The `TLOCK_PARSER` thread mutex is now released after the call to
`msyylex_destroy()` which otherwise clobbers global lexer variables
that may be in use in another thread.

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