diff options
author | wtc%google.com <devnull@localhost> | 2011-10-18 19:03:31 +0000 |
---|---|---|
committer | wtc%google.com <devnull@localhost> | 2011-10-18 19:03:31 +0000 |
commit | 9ad7d2177090754e997ba2e2a37521814d5596c5 (patch) | |
tree | 34d70936bce527fa18fbb5c636fdab30cecf1349 /security/nss/lib | |
parent | 97cca8772eab69e764ec01a9e58293bf080e5af2 (diff) | |
download | nss-hg-9ad7d2177090754e997ba2e2a37521814d5596c5.tar.gz |
Bug 693228: code cleanup: nss_doLockInit and PR_CallOnce return PRStatus,
not SECStatus. r=rrelyea.
Diffstat (limited to 'security/nss/lib')
-rw-r--r-- | security/nss/lib/nss/nssinit.c | 15 |
1 files changed, 7 insertions, 8 deletions
diff --git a/security/nss/lib/nss/nssinit.c b/security/nss/lib/nss/nssinit.c index 50bc44ae2..14ff5ec79 100644 --- a/security/nss/lib/nss/nssinit.c +++ b/security/nss/lib/nss/nssinit.c @@ -540,13 +540,13 @@ nss_doLockInit(void) { nssInitLock = PZ_NewLock(nssILockOther); if (nssInitLock == NULL) { - return (PRStatus) SECFailure; + return PR_FAILURE; } nssInitCondition = PZ_NewCondVar(nssInitLock); if (nssInitCondition == NULL) { - return (PRStatus) SECFailure; + return PR_FAILURE; } - return (PRStatus) SECSuccess; + return PR_SUCCESS; } @@ -577,10 +577,10 @@ nss_Init(const char *configdir, const char *certPrefix, const char *keyPrefix, return SECSuccess; } - /* make sure our locks are initialized one and only one time */ - rv = PR_CallOnce(&nssInitOnce, nss_doLockInit); - if (rv != SECSuccess) { - return rv; + /* make sure our lock and condition variable are initialized one and only + * one time */ + if (PR_CallOnce(&nssInitOnce, nss_doLockInit) != PR_SUCCESS) { + return SECFailure; } /* @@ -597,7 +597,6 @@ nss_Init(const char *configdir, const char *certPrefix, const char *keyPrefix, /* once we've completed basic initialization, we can allow more than * one process initialize NSS at a time. */ } - /* get the current value */ nssIsInInit++; PZ_Unlock(nssInitLock); |