Skip to content

Fix: Application hangs during initialization due to deadlock#45

Merged
aexzhou merged 1 commit intomainfrom
fix_config_init_deadlock
Feb 14, 2026
Merged

Fix: Application hangs during initialization due to deadlock#45
aexzhou merged 1 commit intomainfrom
fix_config_init_deadlock

Conversation

@aexzhou
Copy link
Collaborator

@aexzhou aexzhou commented Feb 14, 2026

The application was hanging during initialization because of a deadlock in config_manager.cpp

  1. saveConfig() acquired a lock on mutex_
  2. It then called saveToJson()
  3. saveToJson() called getConfigPath()
  4. getConfigPath() tried to acquire the same mutex_ that was already locked
    -> deadlock from the thread waiting on itself to release the lock

@aexzhou aexzhou self-assigned this Feb 14, 2026
@aexzhou aexzhou added this pull request to the merge queue Feb 14, 2026
Merged via the queue into main with commit 1e8131d Feb 14, 2026
2 checks passed
@aexzhou aexzhou deleted the fix_config_init_deadlock branch February 14, 2026 23:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments