Project

General

Profile

Actions

Bug #1592

closed

Deadlock when detection thread dies and main thread attempts to restart

Added by Mathew Oldham almost 6 years ago. Updated almost 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Target version:
-
Affected Versions:
Effort:
Difficulty:
Label:

Description

When a thread dies and is restarted there is a deadlock. TmThreadCheckThreadState() takes a lock on tv_root_lock as it iterates through the list of ThreadVars and performs a check as to whether the thread is running or not. If the thread has failed, it will potentially attempt to restart the thread calling TmThreadRestartThread(). This subsequently calls TmThreadSpawn() which eventually calls TmThreadAppend() which also attempts to claim a lock on tv_root_mutex. Since the lock was already taken by TmThreadCheckThreadState() this eventually causes a deadlock.

Though not the best solution, the easiest solution would be to remove SCMutexLock/Unlock calls from TmThreadAppend() as TmThreadAppend() is only called from TmThreadSpawn(). It seems like TmThreadAppend() could also be moved into an internal static/inline function as well. It looks like there may be other functions with potentially similar issues. For example TmThreadRemove() is not called by any function, has SCMutexLock/Unlock calls and if there was a higher order function that calls it, the same issue would occur. This may be a candidate for internal static/inline as well.

I think the better solution would be to add in recursive mutexes using PTHREAD_RECURSIVE_MUTEX_INITIALIZER and then define tv_root_lock as a recursive mutex. I'm not suggesting everything switch to recrusive mutexes, just tv_root_lock initially.

Actions #1

Updated by Andreas Herz about 5 years ago

  • Assignee set to OISF Dev
  • Target version set to TBD
Actions #2

Updated by Victor Julien almost 5 years ago

  • Status changed from New to Closed
  • Assignee changed from OISF Dev to Victor Julien
  • Target version deleted (TBD)

I have removed the thread restart logic as it was broken in various ways.

Actions

Also available in: Atom PDF