[opensaml2] 11/11: SSPCPP-756 Fix self deadlock

Etienne Dysli Metref edm-guest at moszumanska.debian.org
Thu Nov 16 08:08:24 UTC 2017


This is an automated email from the git hooks/post-receive script.

edm-guest pushed a commit to branch master
in repository opensaml2.

commit 1296a4e7e7399286e916a697c08dbea5bbc15151
Author: Rod Widdowson <rdw at steadingsoftware.com>
Date:   Thu Nov 9 16:29:02 2017 +0000

    SSPCPP-756 Fix self deadlock
    
    https://issues.shibboleth.net/jira/browse/SSPCPP-756
    
    When using a Locker to guarda lock and doing the locking oneself,
    it helps to use the correct constructor.
---
 saml/saml2/metadata/impl/AbstractDynamicMetadataProvider.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/saml/saml2/metadata/impl/AbstractDynamicMetadataProvider.cpp b/saml/saml2/metadata/impl/AbstractDynamicMetadataProvider.cpp
index 7fe90df..9c4aafa 100644
--- a/saml/saml2/metadata/impl/AbstractDynamicMetadataProvider.cpp
+++ b/saml/saml2/metadata/impl/AbstractDynamicMetadataProvider.cpp
@@ -367,7 +367,7 @@ time_t  AbstractDynamicMetadataProvider::cacheEntity(EntityDescriptor* entity, b
     if (!writeLocked) {
         m_lock->wrlock();
     }
-    Locker locker(writeLocked ? nullptr : const_cast<AbstractDynamicMetadataProvider*>(this));
+    Locker locker(writeLocked ? nullptr : const_cast<AbstractDynamicMetadataProvider*>(this), false);
 
     // Compute the smaller of the validUntil / cacheDuration constraints.
     time_t cacheExp = (entity->getValidUntil() ? entity->getValidUntilEpoch() : SAMLTIME_MAX) - now;

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-shibboleth/opensaml2.git



More information about the Pkg-shibboleth-devel mailing list