diff options
| author | Nils Maier <maierman@web.de> | 2013-08-21 06:06:10 +0200 |
|---|---|---|
| committer | Nils Maier <maierman@web.de> | 2013-08-21 07:32:01 +0200 |
| commit | cf6f58ceece4c85760c82b78c4872dba331ddfc6 (patch) | |
| tree | c0a279e8384dc413697320ff1740707bb9d9efa2 /src/AppleTLSContext.cc | |
| parent | 8526ceeb452d23d7aa73d01c6cb0fc8ae5e0377b (diff) | |
More code cleanups
Diffstat (limited to 'src/AppleTLSContext.cc')
| -rw-r--r-- | src/AppleTLSContext.cc | 26 |
1 files changed, 13 insertions, 13 deletions
diff --git a/src/AppleTLSContext.cc b/src/AppleTLSContext.cc index e4a45768..0c9fe999 100644 --- a/src/AppleTLSContext.cc +++ b/src/AppleTLSContext.cc @@ -58,11 +58,11 @@ namespace { }; #endif // defined(__MAC_10_7) - class cfrelease { + class CFReleaser { const void *ptr_; public: - inline cfrelease(const void *ptr) : ptr_(ptr) {} - inline ~cfrelease() { if (ptr_) CFRelease(ptr_); } + inline CFReleaser(const void *ptr) : ptr_(ptr) {} + inline ~CFReleaser() { if (ptr_) CFRelease(ptr_); } }; static inline bool isWhitespace(char c) @@ -123,13 +123,13 @@ namespace { A2_LOG_ERROR("Failed to get a certref!"); return false; } - cfrelease del_ref(ref); + CFReleaser del_ref(ref); CFDataRef data = SecCertificateCopyData(ref); if (!data) { A2_LOG_ERROR("Failed to get a data!"); return false; } - cfrelease del_data(data); + CFReleaser del_data(data); // Do try all supported hash algorithms. // Usually the fingerprint would be sha1 or md5, however this is more @@ -183,12 +183,12 @@ SecIdentityRef AppleTLSContext::getCredentials() bool AppleTLSContext::tryAsFingerprint(const std::string& fingerprint) { - std::string fp = stripWhitespace(fingerprint); + auto fp = stripWhitespace(fingerprint); // Verify this is a valid hex representation and normalize. fp = util::toHex(util::fromHex(fp.begin(), fp.end())); // Verify this can represent a hash - std::vector<std::string> ht = MessageDigest::getSupportedHashTypes(); + auto ht = MessageDigest::getSupportedHashTypes(); if (std::find_if(ht.begin(), ht.end(), hash_validator(fp)) == ht.end()) { A2_LOG_INFO(fmt("%s is not a fingerprint, invalid hash representation", fingerprint.c_str())); return false; @@ -198,25 +198,25 @@ bool AppleTLSContext::tryAsFingerprint(const std::string& fingerprint) A2_LOG_DEBUG(fmt("Looking for cert with fingerprint %s", fp.c_str())); // Build and run the KeyChain the query. - SecPolicyRef policy = SecPolicyCreateSSL(true, nullptr); + auto policy = SecPolicyCreateSSL(true, nullptr); if (!policy) { A2_LOG_ERROR("Failed to create SecPolicy"); return false; } - cfrelease del_policy(policy); + CFReleaser del_policy(policy); const void *query_values[] = { kSecClassIdentity, kCFBooleanTrue, policy, kSecMatchLimitAll }; - CFDictionaryRef query = CFDictionaryCreate(nullptr, query_keys, query_values, - 4, nullptr, nullptr); + auto query = CFDictionaryCreate(nullptr, query_keys, query_values, 4, + nullptr, nullptr); if (!query) { A2_LOG_ERROR("Failed to create identity query"); return false; } - cfrelease del_query(query); + CFReleaser del_query(query); CFArrayRef identities; OSStatus err = SecItemCopyMatching(query, (CFTypeRef*)&identities); if (err != errSecSuccess) { @@ -254,7 +254,7 @@ bool AppleTLSContext::tryAsFingerprint(const std::string& fingerprint) if (err != errSecSuccess) { A2_LOG_ERROR("Certificate search failed: " + errToString(err)); } - cfrelease del_search(search); + CFReleaser del_search(search); SecIdentityRef id; while (SecIdentitySearchCopyNext(search, &id) == errSecSuccess) { |
