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

NestedDiagnosticContext crashed in multi-thread environment #4557

Open
siren186 opened this issue May 14, 2024 · 0 comments
Open

NestedDiagnosticContext crashed in multi-thread environment #4557

siren186 opened this issue May 14, 2024 · 0 comments
Labels

Comments

@siren186
Copy link

siren186 commented May 14, 2024

Describe the bug
A clear and concise description of what the bug is.

NestedDiagnosticContext& NestedDiagnosticContext::current()
{
	static NestedDiagnosticContext ndc;
	return ndc;
}

It should use Thread Local Storage to store the NDC context. Not a singleton.

Please add relevant environment information:

  • Windows 10
  • poco-1.13.3
@siren186 siren186 added the bug label May 14, 2024
@aleks-f aleks-f added this to the Release 1.14.0 milestone May 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: No status
Development

No branches or pull requests

2 participants