commit 036612b0fc076ea0f85e051cae41be5b3adbc15f
author Adrien Béraud <adrien.beraud@savoirfairelinux.com> Fri Dec 08 14:19:50 2023 -0500
committer Adrien Béraud <adrien.beraud@savoirfairelinux.com> Mon Dec 11 16:24:49 2023 -0500
tree 202ae2794b26e8a19e331cfadaae8fe3e4311633
parent b23278d216dfedf1cc8a7c489b98a5ca24f87381

certstore: cleanup

Change-Id: Ic9bd9a2b570b334cd84652cae97c9a1ffe3350b5

src/security/certstore.cpp

diff --git a/src/security/certstore.cpp b/src/security/certstore.cpp
index 8bd9236..cea117a 100644
--- a/src/security/certstore.cpp
+++ b/src/security/certstore.cpp
@@ -148,9 +148,7 @@
 {
     auto getCertificate_ = [this](const std::string& k) -> std::shared_ptr<crypto::Certificate> {
         auto cit = certs_.find(k);
-        if (cit == certs_.cend())
-            return {};
-        return cit->second;
+        return cit != certs_.cend() ? cit->second : std::shared_ptr<crypto::Certificate>{};
     };
     std::unique_lock<std::mutex> l(lock_);
     auto crt = getCertificate_(k);
@@ -633,7 +631,7 @@
         cert = cert->issuer? cert->issuer : certStore_.getCertificate(cert->getIssuerUID());
     }
 
-    return allowed? PermissionStatus::ALLOWED : PermissionStatus::UNDEFINED;
+    return allowed ? PermissionStatus::ALLOWED : PermissionStatus::UNDEFINED;
 }
 
 std::vector<std::string>