Fixed load_secrets to acquire/release lock in level 0 only

The write_lock call fails with EDEADLK and unlocks in the
next recursion level.

diff --git a/src/charon/plugins/stroke/stroke_cred.c b/src/charon/plugins/stroke/stroke_cred.c
index f878180e9812e92cdf511538d6b53eb0d7fb2a33..e34613efd8e1e3cf82559835e0a0d65077186bc3 100644 (file)
--- a/src/charon/plugins/stroke/stroke_cred.c
+++ b/src/charon/plugins/stroke/stroke_cred.c
@@ -793,9 +793,10 @@ static void load_secrets(private_stroke_cred_t *this, char *file, int level,
        fclose(fd);
        src = chunk;
 
-       this->lock->write_lock(this->lock);
        if (level == 0)
        {
+               this->lock->write_lock(this->lock);
+
                /* flush secrets on non-recursive invocation */
                while (this->shared->remove_last(this->shared,
                                                                                 (void**)&shared) == SUCCESS)
@@ -1106,7 +1107,10 @@ static void load_secrets(private_stroke_cred_t *this, char *file, int level,
                }
        }
 error:
-       this->lock->unlock(this->lock);
+       if (level == 0)
+       {
+               this->lock->unlock(this->lock);
+       }
        chunk_clear(&chunk);
 }