Browse Source

TT#14008 flag Coverity lock-order false positive

The order is indeed reversed, which is why "trylock" is used so the
process doesn't deadlock.

Change-Id: I9142ad3851d1b3917820c88fe2ea316be63745ad
pull/1295/head
Richard Fuchs 5 years ago
parent
commit
a057fffe50
1 changed files with 1 additions and 0 deletions
  1. +1
    -0
      daemon/call.c

+ 1
- 0
daemon/call.c View File

@ -2924,6 +2924,7 @@ restart:
first_call = NULL;
if (rtpe_call_iterators[i].first) {
first_call = rtpe_call_iterators[i].first->data;
// coverity[lock_order : FALSE]
if (mutex_trylock(&first_call->iterator[i].lock)) {
mutex_unlock(&rtpe_call_iterators[i].lock);
continue; // retry


Loading…
Cancel
Save