Fix some mostly harmless warnings from LLVM static analysis

This commit is contained in:
Cameron Gutman 2014-10-21 00:36:30 -04:00
parent 127c53e341
commit 97f968fb0d
3 changed files with 9 additions and 21 deletions

View File

@ -119,13 +119,12 @@ static NSData* p12 = nil;
return NULL; return NULL;
} }
int ret = -1;
EVP_MD_CTX *mdctx = NULL; EVP_MD_CTX *mdctx = NULL;
mdctx = EVP_MD_CTX_create(); mdctx = EVP_MD_CTX_create();
ret = EVP_DigestSignInit(mdctx, NULL, EVP_sha256(), NULL, pkey); EVP_DigestSignInit(mdctx, NULL, EVP_sha256(), NULL, pkey);
ret = EVP_DigestSignUpdate(mdctx, [data bytes], [data length]); EVP_DigestSignUpdate(mdctx, [data bytes], [data length]);
size_t slen; size_t slen;
ret = EVP_DigestSignFinal(mdctx, NULL, &slen); EVP_DigestSignFinal(mdctx, NULL, &slen);
unsigned char* signature = malloc(slen); unsigned char* signature = malloc(slen);
int result = EVP_DigestSignFinal(mdctx, signature, &slen); int result = EVP_DigestSignFinal(mdctx, signature, &slen);

View File

@ -184,9 +184,8 @@ static const NSString* PORT = @"47984";
} }
- (void)connection:(NSURLConnection *)connection willSendRequestForAuthenticationChallenge:(NSURLAuthenticationChallenge *)challenge { - (void)connection:(NSURLConnection *)connection willSendRequestForAuthenticationChallenge:(NSURLAuthenticationChallenge *)challenge {
SecIdentityRef identity = [self getClientCertificate]; // Go get a SecIdentityRef SecIdentityRef identity = [self getClientCertificate];
CFArrayRef certs = [self getCertificate:identity]; // Get an array of certificates NSArray *certArray = [self getCertificate:identity];
NSArray *certArray = (__bridge NSArray *)certs;
NSURLCredential *newCredential = [NSURLCredential credentialWithIdentity:identity certificates:certArray persistence:NSURLCredentialPersistencePermanent]; NSURLCredential *newCredential = [NSURLCredential credentialWithIdentity:identity certificates:certArray persistence:NSURLCredentialPersistencePermanent];
@ -194,22 +193,12 @@ static const NSString* PORT = @"47984";
} }
// Returns an array containing the certificate // Returns an array containing the certificate
- (CFArrayRef)getCertificate:(SecIdentityRef) identity { - (NSArray*)getCertificate:(SecIdentityRef) identity {
SecCertificateRef certificate = nil; SecCertificateRef certificate = nil;
SecIdentityCopyCertificate(identity, &certificate); SecIdentityCopyCertificate(identity, &certificate);
SecCertificateRef certs[1] = { certificate };
CFArrayRef certArray = CFArrayCreate(NULL, (const void **) certs, 1, NULL); return [[NSArray alloc] initWithObjects:(__bridge id)certificate, nil];
SecPolicyRef policyRef = SecPolicyCreateBasicX509();
SecTrustRef trustRef;
OSStatus status = SecTrustCreateWithCertificates(certArray, policyRef, &trustRef);
if (status != noErr) {
NSLog(@"Error Creating certificate");
}
return certArray;
} }
// Returns the identity // Returns the identity

View File

@ -83,7 +83,7 @@ int mkcert(X509 **x509p, EVP_PKEY **pkeyp, int bits, int serial, int years) {
RSA *rsa; RSA *rsa;
X509_NAME *name = NULL; X509_NAME *name = NULL;
if ((pkeyp == NULL) || (*pkeyp == NULL)) { if (*pkeyp == NULL) {
if ((pk=EVP_PKEY_new()) == NULL) { if ((pk=EVP_PKEY_new()) == NULL) {
abort(); abort();
return(0); return(0);
@ -92,7 +92,7 @@ int mkcert(X509 **x509p, EVP_PKEY **pkeyp, int bits, int serial, int years) {
pk = *pkeyp; pk = *pkeyp;
} }
if ((x509p == NULL) || (*x509p == NULL)) { if (*x509p == NULL) {
if ((x = X509_new()) == NULL) { if ((x = X509_new()) == NULL) {
goto err; goto err;
} }