Bug 1041730: Remove unused parameters from PublicKeyPinningService, r=cviecco

--HG--
extra : rebase_source : 6a091a32ab5178899fc1f9ceb57614f4a87f25c3
This commit is contained in:
Brian Smith 2014-07-21 19:18:31 -07:00
parent 826703fd5a
commit 61f8d7b98e

View File

@ -151,7 +151,7 @@ TransportSecurityPreloadCompare(const void *key, const void *entry) {
*/
static bool
CheckPinsForHostname(const CERTCertList *certList, const char *hostname,
const PRTime time, bool enforceTestMode)
bool enforceTestMode)
{
if (!certList) {
return false;
@ -230,8 +230,8 @@ CheckPinsForHostname(const CERTCertList *certList, const char *hostname,
* evaluating at the first OK pin).
*/
static bool
CheckChainAgainstAllNames(const CERTCertList* certList, const PRTime time,
bool enforceTestMode) {
CheckChainAgainstAllNames(const CERTCertList* certList, bool enforceTestMode)
{
PR_LOG(gPublicKeyPinningLog, PR_LOG_DEBUG,
("pkpin: top of checkChainAgainstAllNames"));
CERTCertListNode* node = CERT_LIST_HEAD(certList);
@ -275,7 +275,7 @@ CheckChainAgainstAllNames(const CERTCertList* certList, const PRTime time,
// cannot call CheckPinsForHostname on empty or null hostname
break;
}
if (CheckPinsForHostname(certList, hostName, time, enforceTestMode)) {
if (CheckPinsForHostname(certList, hostName, enforceTestMode)) {
hasValidPins = true;
break;
}
@ -299,7 +299,7 @@ PublicKeyPinningService::ChainHasValidPins(const CERTCertList* certList,
return true;
}
if (!hostname || hostname[0] == 0) {
return CheckChainAgainstAllNames(certList, time, enforceTestMode);
return CheckChainAgainstAllNames(certList, enforceTestMode);
}
return CheckPinsForHostname(certList, hostname, time, enforceTestMode);
return CheckPinsForHostname(certList, hostname, enforceTestMode);
}