Skip to content

Conversation

@cmb69
Copy link
Member

@cmb69 cmb69 commented Jun 27, 2019

Since syslog's ident and facility parameters have been added to
config[1], vsyslog() segfaults on Windows, if openlog() has not been
called before. We bring back the removed lines to fix this.

[1] http://git.php.net/?p=php-src.git;a=commit;h=2475337bd8a0fad0dac03db3f5e7e9d331d53653

Since syslog's ident and facility parameters have been added to
config[1], vsyslog() segfaults on Windows, if openlog() has not been
called before.  We bring back the removed lines to fix this.

[1] <http://git.php.net/?p=php-src.git;a=commit;h=2475337bd8a0fad0dac03db3f5e7e9d331d53653>
@cmb69
Copy link
Member Author

cmb69 commented Jun 27, 2019

I'm not 100% sure that this is the best solution, but at least it fixes the segfault in test\run-tests\bug75042-3.phpt.

PS: see also #2701 (comment)

@cmb69 cmb69 requested a review from weltling June 27, 2019 09:39
@cmb69 cmb69 changed the title Fix #78212: No output Fix #78212: Segfault in built-in webserver Jul 11, 2019
@cmb69
Copy link
Member Author

cmb69 commented Jul 11, 2019

Applied as fa65f5e. Thanks!

@cmb69 cmb69 closed this Jul 11, 2019
@cmb69 cmb69 deleted the fix-78212 branch July 11, 2019 11:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants