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

deadlock in SetVLOGLevel #29

Closed
romange opened this issue May 19, 2015 · 0 comments
Closed

deadlock in SetVLOGLevel #29

romange opened this issue May 19, 2015 · 0 comments

Comments

@romange
Copy link
Contributor

romange commented May 19, 2015

SetVLOGLevel locks vmodule_lock however RAW_VLOG at the end also implicitly locks it via InitVLOG3__. This causes deadlock/segfault.

See #20 to fix it
@ukai ukai closed this as completed in cd37eec Aug 6, 2015
durswd pushed a commit to durswd/glog that referenced this issue Sep 2, 2019
@ukai
Merge pull request google#20 from UbimoLTD/patch-1
8da2185 
Don't call RAW_VLOG with locking vmodule_lock

SetVLOGLevel locks vmodule_lock however RAW_VLOG at the end also implicitly locks it via InitVLOG3__.
This causes deadlock/segfault.

fixes google#29
@mspang mspang mentioned this issue Jan 7, 2020
Closed
@autofuzzoss autofuzzoss mentioned this issue Apr 11, 2022
Closed
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
1 participant
@romange