diff --git a/Podfile.lock b/Podfile.lock index f9479ae58c..e284a4cd98 100644 --- a/Podfile.lock +++ b/Podfile.lock @@ -246,22 +246,22 @@ EXTERNAL SOURCES: CHECKOUT OPTIONS: AxolotlKit: - :commit: f43a65b456c43d1f5cad621ef108b4e18a70c17f + :commit: 66b38e23b4e980cbddc94980f5bd0d1211d9184a :git: https://github.com/signalapp/SignalProtocolKit.git Curve25519Kit: - :commit: 14effe344be35f5d9f5597a2fd56824f78a4067d + :commit: 9e5663baf4c2c7cb68ac4c0beb0423383c19a7fe :git: https://github.com/signalapp/Curve25519Kit GRKOpenSSLFramework: :commit: b799c27e7927e5304ec1e4ad53c6d33c6fd1cae7 :git: https://github.com/signalapp/GRKOpenSSLFramework HKDFKit: - :commit: 8b8326cd50bc488663a3d3743f1a92b90f4d85b4 + :commit: 3e0c2371d125f2d3db26daa498d5d436961b1795 :git: https://github.com/signalapp/HKDFKit.git SignalCoreKit: - :commit: ff0b95770520133b83a4bd7b26bc2c90b51abc4d + :commit: b60dc7d58dfc93ca6eafbb3ea5300c6d67ebc69a :git: https://github.com/signalapp/SignalCoreKit.git SignalMetadataKit: - :commit: 90f3dee7122ff13061770c496001135ba90b71e3 + :commit: d6b0186cc5b15019e85c375bce3bf78e7a7fc162 :git: https://github.com/signalapp/SignalMetadataKit SocketRocket: :commit: 9f9563a83cd8960503074aa8de72206f83fb7a69 @@ -277,7 +277,7 @@ SPEC CHECKSUMS: AFNetworking: b6f891fdfaed196b46c7a83cf209e09697b94057 AxolotlKit: bb02741a93400b915df1be3365ad13d3768b0103 CocoaLumberjack: db7cc9e464771f12054c22ff6947c5a58d43a0fd - Curve25519Kit: 2b2e55cfa2a72f1152d2733a3058d94cc87f37c3 + Curve25519Kit: b3e77b7152ebe95fee2b3fb6c955449492bc14f7 GRKOpenSSLFramework: 8a3735ad41e7dc1daff460467bccd32ca5d6ae3e HKDFKit: 3b6dbbb9d59c221cc6c52c3aa915700cbf24e376 libPhoneNumber-iOS: e444379ac18bbfbdefad571da735b2cd7e096caa diff --git a/Pods b/Pods index 3b101143e5..5821eb8e9f 160000 --- a/Pods +++ b/Pods @@ -1 +1 @@ -Subproject commit 3b101143e57b34fbef613c365c9ee2e29bd25d16 +Subproject commit 5821eb8e9fd9f76fbdbedd3402892c185e65c48e diff --git a/Signal/src/Models/OWSDeviceProvisioningURLParser.h b/Signal/src/Models/OWSDeviceProvisioningURLParser.h index 319a717cd0..22758bbc85 100644 --- a/Signal/src/Models/OWSDeviceProvisioningURLParser.h +++ b/Signal/src/Models/OWSDeviceProvisioningURLParser.h @@ -1,12 +1,14 @@ -// Copyright © 2016 Open Whisper Systems. All rights reserved. +// +// Copyright (c) 2018 Open Whisper Systems. All rights reserved. +// NS_ASSUME_NONNULL_BEGIN @interface OWSDeviceProvisioningURLParser : NSObject @property (readonly, getter=isValid) BOOL valid; -@property (nonatomic, readonly) NSString *ephemeralDeviceId; -@property (nonatomic, readonly) NSData *publicKey; +@property (nonatomic, readonly, nullable) NSString *ephemeralDeviceId; +@property (nonatomic, readonly, nullable) NSData *publicKey; - (instancetype)initWithProvisioningURL:(NSString *)provisioningURL; diff --git a/Signal/src/Models/OWSDeviceProvisioningURLParser.m b/Signal/src/Models/OWSDeviceProvisioningURLParser.m index 41aebc6ab0..3ae5d86655 100644 --- a/Signal/src/Models/OWSDeviceProvisioningURLParser.m +++ b/Signal/src/Models/OWSDeviceProvisioningURLParser.m @@ -26,7 +26,11 @@ NSString *const OWSQueryItemNameEncodedPublicKeyKey = @"pub_key"; _ephemeralDeviceId = queryItem.value; } else if ([queryItem.name isEqualToString:OWSQueryItemNameEncodedPublicKeyKey]) { NSString *encodedPublicKey = queryItem.value; - _publicKey = [[NSData dataFromBase64String:encodedPublicKey] removeKeyType]; + @try { + _publicKey = [[NSData dataFromBase64String:encodedPublicKey] throws_removeKeyType]; + } @catch (NSException *exception) { + OWSFailDebug(@"exception: %@", exception); + } } else { OWSLogWarn(@"Unkown query item in provisioning string: %@", queryItem.name); } diff --git a/Signal/src/ViewControllers/ConversationView/ConversationViewController.m b/Signal/src/ViewControllers/ConversationView/ConversationViewController.m index 15d8d3cc9f..142ba0cadc 100644 --- a/Signal/src/ViewControllers/ConversationView/ConversationViewController.m +++ b/Signal/src/ViewControllers/ConversationView/ConversationViewController.m @@ -1901,7 +1901,9 @@ typedef enum : NSUInteger { // but there will be some legacy ones in the wild, behind which await // as-of-yet-undecrypted messages if ([errorMessage isKindOfClass:[TSInvalidIdentityKeyReceivingErrorMessage class]]) { - [errorMessage acceptNewIdentityKey]; + // Deliberately crash if the user fails to explicitly accept the new identity + // key. In practice we haven't been creating these messages in over a year. + [errorMessage throws_acceptNewIdentityKey]; } }]; [actionSheetController addAction:acceptSafetyNumberAction]; diff --git a/SignalMessaging/environment/migrations/OWSDatabaseMigration.m b/SignalMessaging/environment/migrations/OWSDatabaseMigration.m index 1a7c7702fe..75fc94aed2 100644 --- a/SignalMessaging/environment/migrations/OWSDatabaseMigration.m +++ b/SignalMessaging/environment/migrations/OWSDatabaseMigration.m @@ -40,7 +40,8 @@ NS_ASSUME_NONNULL_BEGIN + (NSString *)migrationId { - OWSRaiseException(NSInternalInconsistencyException, @"Must override %@ in subclass", NSStringFromSelector(_cmd)); + OWSAbstractMethod(); + return @""; } + (NSString *)collection @@ -51,7 +52,7 @@ NS_ASSUME_NONNULL_BEGIN - (void)runUpWithTransaction:(YapDatabaseReadWriteTransaction *)transaction { - OWSRaiseException(NSInternalInconsistencyException, @"Must override %@ in subclass", NSStringFromSelector(_cmd)); + OWSAbstractMethod(); } - (void)runUpWithCompletion:(OWSDatabaseMigrationCompletion)completion diff --git a/SignalMessaging/utils/ThreadUtil.m b/SignalMessaging/utils/ThreadUtil.m index 4d9029050b..a25363a294 100644 --- a/SignalMessaging/utils/ThreadUtil.m +++ b/SignalMessaging/utils/ThreadUtil.m @@ -631,13 +631,17 @@ NS_ASSUME_NONNULL_BEGIN continue; } - NSData *_Nullable newIdentityKey = safetyNumberChange.newIdentityKey; - if (newIdentityKey == nil) { - OWSFailDebug(@"Safety number change was missing it's new identity key."); - continue; - } + @try { + NSData *_Nullable newIdentityKey = [safetyNumberChange throws_newIdentityKey]; + if (newIdentityKey == nil) { + OWSFailDebug(@"Safety number change was missing it's new identity key."); + continue; + } - [missingUnseenSafetyNumberChanges addObject:newIdentityKey]; + [missingUnseenSafetyNumberChanges addObject:newIdentityKey]; + } @catch (NSException *exception) { + OWSFailDebug(@"exception: %@", exception); + } } // Count the de-duplicated "blocking" safety number changes and all diff --git a/SignalServiceKit/src/Account/TSAccountManager.m b/SignalServiceKit/src/Account/TSAccountManager.m index 5b1af082be..01b1243392 100644 --- a/SignalServiceKit/src/Account/TSAccountManager.m +++ b/SignalServiceKit/src/Account/TSAccountManager.m @@ -159,7 +159,7 @@ NSString *const TSAccountManager_NeedsAccountAttributesUpdateKey = @"TSAccountMa NSString *phoneNumber = self.phoneNumberAwaitingVerification; if (!phoneNumber) { - OWSRaiseException(@"RegistrationFail", @"Internal Corrupted State"); + OWSFail(@"phoneNumber was unexpectedly nil"); } [self storeLocalNumber:phoneNumber]; diff --git a/SignalServiceKit/src/Contacts/CDSSigningCertificate.m b/SignalServiceKit/src/Contacts/CDSSigningCertificate.m index 16e78c3fd9..448b4d92cc 100644 --- a/SignalServiceKit/src/Contacts/CDSSigningCertificate.m +++ b/SignalServiceKit/src/Contacts/CDSSigningCertificate.m @@ -213,8 +213,7 @@ NS_ASSUME_NONNULL_BEGIN // We need to use an Intel certificate as the anchor for IAS verification. NSData *_Nullable anchorCertificate = [self certificateDataForService:@"ias-root"]; if (!anchorCertificate) { - OWSFailDebug(@"could not load anchor certificate."); - OWSRaiseException(@"OWSSignalService_CouldNotLoadCertificate", @"%s", __PRETTY_FUNCTION__); + OWSFail(@"could not load anchor certificate."); } else { anchorCertificates = @[ anchorCertificate ]; } diff --git a/SignalServiceKit/src/Contacts/ContactDiscoveryService.m b/SignalServiceKit/src/Contacts/ContactDiscoveryService.m index 922ecbdf6e..757fb1dd1e 100644 --- a/SignalServiceKit/src/Contacts/ContactDiscoveryService.m +++ b/SignalServiceKit/src/Contacts/ContactDiscoveryService.m @@ -83,9 +83,9 @@ NSErrorDomain const ContactDiscoveryServiceErrorDomain = @"SignalServiceKit.Cont NSData *ephemeralToStatic; @try { ephemeralToEphemeral = - [Curve25519 generateSharedSecretFromPublicKey:self.serverEphemeralPublic andKeyPair:self.keyPair]; + [Curve25519 throws_generateSharedSecretFromPublicKey:self.serverEphemeralPublic andKeyPair:self.keyPair]; ephemeralToStatic = - [Curve25519 generateSharedSecretFromPublicKey:self.serverStaticPublic andKeyPair:self.keyPair]; + [Curve25519 throws_generateSharedSecretFromPublicKey:self.serverStaticPublic andKeyPair:self.keyPair]; } @catch (NSException *exception) { OWSFailDebug(@"could not generate shared secrets: %@", exception); return NO; @@ -101,7 +101,7 @@ NSErrorDomain const ContactDiscoveryServiceErrorDomain = @"SignalServiceKit.Cont NSData *_Nullable derivedMaterial; @try { derivedMaterial = - [HKDFKit deriveKey:masterSecret info:nil salt:publicKeys outputSize:(int)kAES256_KeyByteLength * 2]; + [HKDFKit throws_deriveKey:masterSecret info:nil salt:publicKeys outputSize:(int)kAES256_KeyByteLength * 2]; } @catch (NSException *exception) { OWSFailDebug(@"could not derive service key: %@", exception); return NO; diff --git a/SignalServiceKit/src/Contacts/TSThread.m b/SignalServiceKit/src/Contacts/TSThread.m index fe839afb28..4d046f132b 100644 --- a/SignalServiceKit/src/Contacts/TSThread.m +++ b/SignalServiceKit/src/Contacts/TSThread.m @@ -248,8 +248,12 @@ ConversationColorName const kConversationColorName_Default = ConversationColorNa [self enumerateInteractionsUsingBlock:^(TSInteraction *interaction) { if ([interaction isKindOfClass:[TSInvalidIdentityKeyReceivingErrorMessage class]]) { TSInvalidIdentityKeyReceivingErrorMessage *error = (TSInvalidIdentityKeyReceivingErrorMessage *)interaction; - if ([[error newIdentityKey] isEqualToData:key]) { - [errorMessages addObject:(TSInvalidIdentityKeyReceivingErrorMessage *)interaction]; + @try { + if ([[error throws_newIdentityKey] isEqualToData:key]) { + [errorMessages addObject:(TSInvalidIdentityKeyReceivingErrorMessage *)interaction]; + } + } @catch (NSException *exception) { + OWSFailDebug(@"exception: %@", exception); } } }]; diff --git a/SignalServiceKit/src/Devices/OWSProvisioningCipher.m b/SignalServiceKit/src/Devices/OWSProvisioningCipher.m index 2541e4fb97..aac5b64997 100644 --- a/SignalServiceKit/src/Devices/OWSProvisioningCipher.m +++ b/SignalServiceKit/src/Devices/OWSProvisioningCipher.m @@ -54,11 +54,7 @@ NS_ASSUME_NONNULL_BEGIN - (nullable NSData *)encrypt:(NSData *)dataToEncrypt { @try { - // Exceptions can be thrown in a number of places in encryptUnsafe, e.g.: - // - // * Curve25519's generateSharedSecretFromPublicKey. - // * [HKDFKit deriveKey]. - return [self encryptUnsafe:dataToEncrypt]; + return [self throws_encryptWithData:dataToEncrypt]; } @catch (NSException *exception) { OWSFailDebug(@"exception: %@ of type: %@ with reason: %@, user info: %@.", exception.description, @@ -69,14 +65,14 @@ NS_ASSUME_NONNULL_BEGIN } } -- (nullable NSData *)encryptUnsafe:(NSData *)dataToEncrypt +- (nullable NSData *)throws_encryptWithData:(NSData *)dataToEncrypt { NSData *sharedSecret = - [Curve25519 generateSharedSecretFromPublicKey:self.theirPublicKey andKeyPair:self.ourKeyPair]; + [Curve25519 throws_generateSharedSecretFromPublicKey:self.theirPublicKey andKeyPair:self.ourKeyPair]; NSData *infoData = [@"TextSecure Provisioning Message" dataUsingEncoding:NSASCIIStringEncoding]; NSData *nullSalt = [[NSMutableData dataWithLength:32] copy]; - NSData *derivedSecret = [HKDFKit deriveKey:sharedSecret info:infoData salt:nullSalt outputSize:64]; + NSData *derivedSecret = [HKDFKit throws_deriveKey:sharedSecret info:infoData salt:nullSalt outputSize:64]; NSData *cipherKey = [derivedSecret subdataWithRange:NSMakeRange(0, 32)]; NSData *macKey = [derivedSecret subdataWithRange:NSMakeRange(32, 32)]; if (cipherKey.length != 32) { diff --git a/SignalServiceKit/src/Messages/InvalidKeyMessages/TSInvalidIdentityKeyErrorMessage.h b/SignalServiceKit/src/Messages/InvalidKeyMessages/TSInvalidIdentityKeyErrorMessage.h index 2ef76e8adc..2f4f5ff085 100644 --- a/SignalServiceKit/src/Messages/InvalidKeyMessages/TSInvalidIdentityKeyErrorMessage.h +++ b/SignalServiceKit/src/Messages/InvalidKeyMessages/TSInvalidIdentityKeyErrorMessage.h @@ -1,5 +1,5 @@ // -// Copyright (c) 2017 Open Whisper Systems. All rights reserved. +// Copyright (c) 2018 Open Whisper Systems. All rights reserved. // #import "TSErrorMessage.h" @@ -10,8 +10,8 @@ NS_ASSUME_NONNULL_BEGIN @interface TSInvalidIdentityKeyErrorMessage : TSErrorMessage -- (void)acceptNewIdentityKey; -- (nullable NSData *)newIdentityKey; +- (void)throws_acceptNewIdentityKey NS_SWIFT_UNAVAILABLE("throws objc exceptions"); +- (nullable NSData *)throws_newIdentityKey NS_SWIFT_UNAVAILABLE("throws objc exceptions"); - (NSString *)theirSignalId; @end diff --git a/SignalServiceKit/src/Messages/InvalidKeyMessages/TSInvalidIdentityKeyErrorMessage.m b/SignalServiceKit/src/Messages/InvalidKeyMessages/TSInvalidIdentityKeyErrorMessage.m index 78fbbcc30e..2394618757 100644 --- a/SignalServiceKit/src/Messages/InvalidKeyMessages/TSInvalidIdentityKeyErrorMessage.m +++ b/SignalServiceKit/src/Messages/InvalidKeyMessages/TSInvalidIdentityKeyErrorMessage.m @@ -8,12 +8,12 @@ NS_ASSUME_NONNULL_BEGIN @implementation TSInvalidIdentityKeyErrorMessage -- (void)acceptNewIdentityKey +- (void)throws_acceptNewIdentityKey { OWSAbstractMethod(); } -- (nullable NSData *)newIdentityKey +- (nullable NSData *)throws_newIdentityKey { OWSAbstractMethod(); return nil; diff --git a/SignalServiceKit/src/Messages/InvalidKeyMessages/TSInvalidIdentityKeyReceivingErrorMessage.m b/SignalServiceKit/src/Messages/InvalidKeyMessages/TSInvalidIdentityKeyReceivingErrorMessage.m index fac35fbdd1..7fa210fa90 100644 --- a/SignalServiceKit/src/Messages/InvalidKeyMessages/TSInvalidIdentityKeyReceivingErrorMessage.m +++ b/SignalServiceKit/src/Messages/InvalidKeyMessages/TSInvalidIdentityKeyReceivingErrorMessage.m @@ -81,7 +81,7 @@ NS_ASSUME_NONNULL_BEGIN return _envelope; } -- (void)acceptNewIdentityKey +- (void)throws_acceptNewIdentityKey { OWSAssertIsOnMainThread(); @@ -90,7 +90,7 @@ NS_ASSUME_NONNULL_BEGIN return; } - NSData *_Nullable newKey = [self newIdentityKey]; + NSData *_Nullable newKey = [self throws_newIdentityKey]; if (!newKey) { OWSFailDebug(@"Couldn't extract identity key to accept"); return; @@ -112,7 +112,7 @@ NS_ASSUME_NONNULL_BEGIN } } -- (nullable NSData *)newIdentityKey +- (nullable NSData *)throws_newIdentityKey { if (!self.envelope) { OWSLogError(@"Error message had no envelope data to extract key from"); @@ -130,8 +130,8 @@ NS_ASSUME_NONNULL_BEGIN return nil; } - PreKeyWhisperMessage *message = [[PreKeyWhisperMessage alloc] initWithData:pkwmData]; - return [message.identityKey removeKeyType]; + PreKeyWhisperMessage *message = [[PreKeyWhisperMessage alloc] init_throws_withData:pkwmData]; + return [message.identityKey throws_removeKeyType]; } - (NSString *)theirSignalId diff --git a/SignalServiceKit/src/Messages/InvalidKeyMessages/TSInvalidIdentityKeySendingErrorMessage.m b/SignalServiceKit/src/Messages/InvalidKeyMessages/TSInvalidIdentityKeySendingErrorMessage.m index 01e4d179c9..3b5a47b2dd 100644 --- a/SignalServiceKit/src/Messages/InvalidKeyMessages/TSInvalidIdentityKeySendingErrorMessage.m +++ b/SignalServiceKit/src/Messages/InvalidKeyMessages/TSInvalidIdentityKeySendingErrorMessage.m @@ -45,13 +45,13 @@ NSString *TSInvalidRecipientKey = @"TSInvalidRecipientKey"; return self; } -- (void)acceptNewIdentityKey +- (void)throws_acceptNewIdentityKey { // Shouldn't really get here, since we're no longer creating blocking SN changes. // But there may still be some old unaccepted SN errors in the wild that need to be accepted. OWSFailDebug(@"accepting new identity key is deprecated."); - NSData *_Nullable newIdentityKey = self.newIdentityKey; + NSData *_Nullable newIdentityKey = [self throws_newIdentityKey]; if (!newIdentityKey) { OWSFailDebug(@"newIdentityKey is unexpectedly nil. Bad Prekey bundle?: %@", self.preKeyBundle); return; @@ -60,9 +60,9 @@ NSString *TSInvalidRecipientKey = @"TSInvalidRecipientKey"; [[OWSIdentityManager sharedManager] saveRemoteIdentity:newIdentityKey recipientId:self.recipientId]; } -- (nullable NSData *)newIdentityKey +- (nullable NSData *)throws_newIdentityKey { - return [self.preKeyBundle.identityKey removeKeyType]; + return [self.preKeyBundle.identityKey throws_removeKeyType]; } - (NSString *)theirSignalId diff --git a/SignalServiceKit/src/Messages/OWSBatchMessageProcessor.m b/SignalServiceKit/src/Messages/OWSBatchMessageProcessor.m index df04ac7ffb..49f05fc881 100644 --- a/SignalServiceKit/src/Messages/OWSBatchMessageProcessor.m +++ b/SignalServiceKit/src/Messages/OWSBatchMessageProcessor.m @@ -403,9 +403,9 @@ NSString *const OWSMessageContentJobFinderExtensionGroup = @"OWSMessageContentJo if (!envelope) { reportFailure(transaction); } else { - [self.messageManager processEnvelope:envelope - plaintextData:job.plaintextData - transaction:transaction]; + [self.messageManager throws_processEnvelope:envelope + plaintextData:job.plaintextData + transaction:transaction]; } } @catch (NSException *exception) { OWSFailDebug(@"Received an invalid envelope: %@", exception.debugDescription); diff --git a/SignalServiceKit/src/Messages/OWSIdentityManager.h b/SignalServiceKit/src/Messages/OWSIdentityManager.h index 838de61d9f..2d487487d1 100644 --- a/SignalServiceKit/src/Messages/OWSIdentityManager.h +++ b/SignalServiceKit/src/Messages/OWSIdentityManager.h @@ -60,8 +60,8 @@ extern const NSUInteger kStoredIdentityKeyLength; - (nullable OWSRecipientIdentity *)untrustedIdentityForSendingToRecipientId:(NSString *)recipientId; // This method can be called from any thread. -- (void)processIncomingSyncMessage:(SSKProtoVerified *)verified - transaction:(YapDatabaseReadWriteTransaction *)transaction; +- (void)throws_processIncomingSyncMessage:(SSKProtoVerified *)verified + transaction:(YapDatabaseReadWriteTransaction *)transaction; - (BOOL)saveRemoteIdentity:(NSData *)identityKey recipientId:(NSString *)recipientId; diff --git a/SignalServiceKit/src/Messages/OWSIdentityManager.m b/SignalServiceKit/src/Messages/OWSIdentityManager.m index 590209bdf8..ece948c886 100644 --- a/SignalServiceKit/src/Messages/OWSIdentityManager.m +++ b/SignalServiceKit/src/Messages/OWSIdentityManager.m @@ -678,8 +678,8 @@ NSString *const kNSNotificationName_IdentityStateDidChange = @"kNSNotificationNa [transaction removeObjectForKey:recipientId inCollection:OWSIdentityManager_QueuedVerificationStateSyncMessages]; } -- (void)processIncomingSyncMessage:(SSKProtoVerified *)verified - transaction:(YapDatabaseReadWriteTransaction *)transaction +- (void)throws_processIncomingSyncMessage:(SSKProtoVerified *)verified + transaction:(YapDatabaseReadWriteTransaction *)transaction { OWSAssertDebug(verified); OWSAssertDebug(transaction); @@ -696,7 +696,7 @@ NSString *const kNSNotificationName_IdentityStateDidChange = @"kNSNotificationNa rawIdentityKey); return; } - NSData *identityKey = [rawIdentityKey removeKeyType]; + NSData *identityKey = [rawIdentityKey throws_removeKeyType]; switch (verified.state) { case SSKProtoVerifiedStateDefault: diff --git a/SignalServiceKit/src/Messages/OWSMessageDecrypter.m b/SignalServiceKit/src/Messages/OWSMessageDecrypter.m index 7a956ceaab..960069cc77 100644 --- a/SignalServiceKit/src/Messages/OWSMessageDecrypter.m +++ b/SignalServiceKit/src/Messages/OWSMessageDecrypter.m @@ -26,6 +26,7 @@ #import #import #import +#import #import #import @@ -211,7 +212,7 @@ NSError *EnsureDecryptError(NSError *_Nullable error, NSString *fallbackErrorDes switch (envelope.type) { case SSKProtoEnvelopeTypeCiphertext: { - [self decryptSecureMessage:envelope + [self throws_decryptSecureMessage:envelope envelopeData:envelopeData successBlock:^(OWSMessageDecryptResult *result, YapDatabaseReadWriteTransaction *transaction) { OWSLogDebug(@"decrypted secure message."); @@ -228,7 +229,7 @@ NSError *EnsureDecryptError(NSError *_Nullable error, NSString *fallbackErrorDes return; } case SSKProtoEnvelopeTypePrekeyBundle: { - [self decryptPreKeyBundle:envelope + [self throws_decryptPreKeyBundle:envelope envelopeData:envelopeData successBlock:^(OWSMessageDecryptResult *result, YapDatabaseReadWriteTransaction *transaction) { OWSLogDebug(@"decrypted pre-key whisper message"); @@ -297,10 +298,10 @@ NSError *EnsureDecryptError(NSError *_Nullable error, NSString *fallbackErrorDes failureBlock(); } -- (void)decryptSecureMessage:(SSKProtoEnvelope *)envelope - envelopeData:(NSData *)envelopeData - successBlock:(DecryptSuccessBlock)successBlock - failureBlock:(void (^)(NSError *_Nullable error))failureBlock +- (void)throws_decryptSecureMessage:(SSKProtoEnvelope *)envelope + envelopeData:(NSData *)envelopeData + successBlock:(DecryptSuccessBlock)successBlock + failureBlock:(void (^)(NSError *_Nullable error))failureBlock { OWSAssertDebug(envelope); OWSAssertDebug(envelopeData); @@ -311,16 +312,16 @@ NSError *EnsureDecryptError(NSError *_Nullable error, NSString *fallbackErrorDes envelopeData:envelopeData cipherTypeName:@"Secure Message" cipherMessageBlock:^(NSData *encryptedData) { - return [[WhisperMessage alloc] initWithData:encryptedData]; + return [[WhisperMessage alloc] init_throws_withData:encryptedData]; } successBlock:successBlock failureBlock:failureBlock]; } -- (void)decryptPreKeyBundle:(SSKProtoEnvelope *)envelope - envelopeData:(NSData *)envelopeData - successBlock:(DecryptSuccessBlock)successBlock - failureBlock:(void (^)(NSError *_Nullable error))failureBlock +- (void)throws_decryptPreKeyBundle:(SSKProtoEnvelope *)envelope + envelopeData:(NSData *)envelopeData + successBlock:(DecryptSuccessBlock)successBlock + failureBlock:(void (^)(NSError *_Nullable error))failureBlock { OWSAssertDebug(envelope); OWSAssertDebug(envelopeData); @@ -334,7 +335,7 @@ NSError *EnsureDecryptError(NSError *_Nullable error, NSString *fallbackErrorDes envelopeData:envelopeData cipherTypeName:@"PreKey Bundle" cipherMessageBlock:^(NSData *encryptedData) { - return [[PreKeyWhisperMessage alloc] initWithData:encryptedData]; + return [[PreKeyWhisperMessage alloc] init_throws_withData:encryptedData]; } successBlock:successBlock failureBlock:failureBlock]; @@ -378,7 +379,7 @@ NSError *EnsureDecryptError(NSError *_Nullable error, NSString *fallbackErrorDes // plaintextData may be nil for some envelope types. NSData *_Nullable plaintextData = - [[cipher decrypt:cipherMessage protocolContext:transaction] removePadding]; + [[cipher throws_decrypt:cipherMessage protocolContext:transaction] removePadding]; OWSMessageDecryptResult *result = [OWSMessageDecryptResult resultWithEnvelopeData:envelopeData plaintextData:plaintextData source:envelope.source @@ -448,13 +449,15 @@ NSError *EnsureDecryptError(NSError *_Nullable error, NSString *fallbackErrorDes } SMKDecryptResult *_Nullable decryptResult = - [cipher decryptMessageWithCertificateValidator:certificateValidator - cipherTextData:encryptedData - timestamp:serverTimestamp - localRecipientId:localRecipientId - localDeviceId:localDeviceId - protocolContext:transaction - error:&error]; + [cipher throwswrapped_decryptMessageWithCertificateValidator:certificateValidator + cipherTextData:encryptedData + timestamp:serverTimestamp + localRecipientId:localRecipientId + localDeviceId:localDeviceId + protocolContext:transaction + error:&error]; + SCKRaiseIfExceptionWrapperError(error); + if (error || !decryptResult) { if ([error.domain isEqualToString:@"SignalMetadataKit.SMKSecretSessionCipherError"] && error.code == SMKSecretSessionCipherErrorSelfSentMessage) { diff --git a/SignalServiceKit/src/Messages/OWSMessageManager.h b/SignalServiceKit/src/Messages/OWSMessageManager.h index 0e7cd51e43..fb274d0603 100644 --- a/SignalServiceKit/src/Messages/OWSMessageManager.h +++ b/SignalServiceKit/src/Messages/OWSMessageManager.h @@ -19,9 +19,9 @@ NS_ASSUME_NONNULL_BEGIN - (instancetype)initWithPrimaryStorage:(OWSPrimaryStorage *)primaryStorage NS_DESIGNATED_INITIALIZER; // processEnvelope: can be called from any thread. -- (void)processEnvelope:(SSKProtoEnvelope *)envelope - plaintextData:(NSData *_Nullable)plaintextData - transaction:(YapDatabaseReadWriteTransaction *)transaction; +- (void)throws_processEnvelope:(SSKProtoEnvelope *)envelope + plaintextData:(NSData *_Nullable)plaintextData + transaction:(YapDatabaseReadWriteTransaction *)transaction; // This should be invoked by the main app when the app is ready. - (void)startObserving; diff --git a/SignalServiceKit/src/Messages/OWSMessageManager.m b/SignalServiceKit/src/Messages/OWSMessageManager.m index 514d5c3d2f..06a2a52134 100644 --- a/SignalServiceKit/src/Messages/OWSMessageManager.m +++ b/SignalServiceKit/src/Messages/OWSMessageManager.m @@ -212,9 +212,9 @@ NS_ASSUME_NONNULL_BEGIN #pragma mark - message handling -- (void)processEnvelope:(SSKProtoEnvelope *)envelope - plaintextData:(NSData *_Nullable)plaintextData - transaction:(YapDatabaseReadWriteTransaction *)transaction +- (void)throws_processEnvelope:(SSKProtoEnvelope *)envelope + plaintextData:(NSData *_Nullable)plaintextData + transaction:(YapDatabaseReadWriteTransaction *)transaction { if (!envelope) { OWSFailDebug(@"Missing envelope."); @@ -256,7 +256,7 @@ NS_ASSUME_NONNULL_BEGIN OWSFailDebug(@"missing decrypted data for envelope: %@", [self descriptionForEnvelope:envelope]); return; } - [self handleEnvelope:envelope plaintextData:plaintextData transaction:transaction]; + [self throws_handleEnvelope:envelope plaintextData:plaintextData transaction:transaction]; break; case SSKProtoEnvelopeTypeReceipt: OWSAssertDebug(!plaintextData); @@ -345,9 +345,9 @@ NS_ASSUME_NONNULL_BEGIN } } -- (void)handleEnvelope:(SSKProtoEnvelope *)envelope - plaintextData:(NSData *)plaintextData - transaction:(YapDatabaseReadWriteTransaction *)transaction +- (void)throws_handleEnvelope:(SSKProtoEnvelope *)envelope + plaintextData:(NSData *)plaintextData + transaction:(YapDatabaseReadWriteTransaction *)transaction { if (!envelope) { OWSFailDebug(@"Missing envelope."); @@ -395,7 +395,9 @@ NS_ASSUME_NONNULL_BEGIN OWSLogInfo(@"handling content: ", [self descriptionForContent:contentProto]); if (contentProto.syncMessage) { - [self handleIncomingEnvelope:envelope withSyncMessage:contentProto.syncMessage transaction:transaction]; + [self throws_handleIncomingEnvelope:envelope + withSyncMessage:contentProto.syncMessage + transaction:transaction]; [[OWSDeviceManager sharedManager] setHasReceivedSyncMessage]; } else if (contentProto.dataMessage) { @@ -745,9 +747,9 @@ NS_ASSUME_NONNULL_BEGIN }]; } -- (void)handleIncomingEnvelope:(SSKProtoEnvelope *)envelope - withSyncMessage:(SSKProtoSyncMessage *)syncMessage - transaction:(YapDatabaseReadWriteTransaction *)transaction +- (void)throws_handleIncomingEnvelope:(SSKProtoEnvelope *)envelope + withSyncMessage:(SSKProtoSyncMessage *)syncMessage + transaction:(YapDatabaseReadWriteTransaction *)transaction { if (!envelope) { OWSFailDebug(@"Missing envelope."); @@ -862,7 +864,7 @@ NS_ASSUME_NONNULL_BEGIN transaction:transaction]; } else if (syncMessage.verified) { OWSLogInfo(@"Received verification state for %@", syncMessage.verified.destination); - [self.identityManager processIncomingSyncMessage:syncMessage.verified transaction:transaction]; + [self.identityManager throws_processIncomingSyncMessage:syncMessage.verified transaction:transaction]; } else { OWSLogWarn(@"Ignoring unsupported sync message."); } diff --git a/SignalServiceKit/src/Messages/OWSMessageSender.m b/SignalServiceKit/src/Messages/OWSMessageSender.m index 9f8769425d..81b6953f38 100644 --- a/SignalServiceKit/src/Messages/OWSMessageSender.m +++ b/SignalServiceKit/src/Messages/OWSMessageSender.m @@ -50,6 +50,7 @@ #import #import #import +#import #import #import #import @@ -478,7 +479,7 @@ NSString *const OWSMessageSenderRateLimitedException = @"RateLimitedException"; failure:(RetryableFailureHandler)failure { [self.udManager - ensureSenderCertificateObjCWithSuccess:^(SMKSenderCertificate *senderCertificate) { + ensureSenderCertificateWithSuccess:^(SMKSenderCertificate *senderCertificate) { dispatch_async([OWSDispatch sendingQueue], ^{ [self sendMessageToService:message senderCertificate:senderCertificate success:success failure:failure]; }); @@ -838,7 +839,7 @@ NSString *const OWSMessageSenderRateLimitedException = @"RateLimitedException"; NSArray *deviceMessages; @try { - deviceMessages = [self deviceMessagesForMessageSendUnsafe:messageSend]; + deviceMessages = [self throws_deviceMessagesForMessageSendUnsafe:messageSend]; } @catch (NSException *exception) { if ([exception.name isEqualToString:UntrustedIdentityKeyException]) { // This *can* happen under normal usage, but it should happen relatively rarely. @@ -887,7 +888,7 @@ NSString *const OWSMessageSenderRateLimitedException = @"RateLimitedException"; return nil; } - NSData *newIdentityKey = [newIdentityKeyWithVersion removeKeyType]; + NSData *newIdentityKey = [newIdentityKeyWithVersion throws_removeKeyType]; [self.identityManager saveRemoteIdentity:newIdentityKey recipientId:recipient.recipientId]; return nil; @@ -1406,8 +1407,7 @@ NSString *const OWSMessageSenderRateLimitedException = @"RateLimitedException"; [self sendMessageToRecipient:messageSend]; } -// NOTE: This method uses exceptions for control flow. -- (NSArray *)deviceMessagesForMessageSendUnsafe:(OWSMessageSend *)messageSend +- (NSArray *)throws_deviceMessagesForMessageSendUnsafe:(OWSMessageSend *)messageSend { OWSAssertDebug(messageSend.message); OWSAssertDebug(messageSend.recipient); @@ -1443,17 +1443,17 @@ NSString *const OWSMessageSenderRateLimitedException = @"RateLimitedException"; @try { // This may involve blocking network requests, so we do it _before_ // we open a transaction. - [self ensureRecipientHasSessionForMessageSend:messageSend deviceId:deviceId]; + [self throws_ensureRecipientHasSessionForMessageSend:messageSend deviceId:deviceId]; __block NSDictionary *messageDict; __block NSException *encryptionException; [self.dbConnection readWriteWithBlock:^(YapDatabaseReadWriteTransaction *transaction) { @try { - messageDict = [self encryptedMessageForMessageSend:messageSend - deviceId:deviceId - plainText:plainText - transaction:transaction]; + messageDict = [self throws_encryptedMessageForMessageSend:messageSend + deviceId:deviceId + plainText:plainText + transaction:transaction]; } @catch (NSException *exception) { encryptionException = exception; } @@ -1485,9 +1485,7 @@ NSString *const OWSMessageSenderRateLimitedException = @"RateLimitedException"; return [messagesArray copy]; } -// NOTE: This method uses exceptions for control flow. -- (void)ensureRecipientHasSessionForMessageSend:(OWSMessageSend *)messageSend - deviceId:(NSNumber *)deviceId +- (void)throws_ensureRecipientHasSessionForMessageSend:(OWSMessageSend *)messageSend deviceId:(NSNumber *)deviceId { OWSAssertDebug(messageSend); OWSAssertDebug(deviceId); @@ -1546,7 +1544,7 @@ NSString *const OWSMessageSenderRateLimitedException = @"RateLimitedException"; deviceId:[deviceId intValue]]; [self.dbConnection readWriteWithBlock:^(YapDatabaseReadWriteTransaction *transaction) { @try { - [builder processPrekeyBundle:bundle protocolContext:transaction]; + [builder throws_processPrekeyBundle:bundle protocolContext:transaction]; } @catch (NSException *caughtException) { exception = caughtException; } @@ -1611,11 +1609,10 @@ NSString *const OWSMessageSenderRateLimitedException = @"RateLimitedException"; }) retainUntilComplete]; } -// NOTE: This method uses exceptions for control flow. -- (NSDictionary *)encryptedMessageForMessageSend:(OWSMessageSend *)messageSend - deviceId:(NSNumber *)deviceId - plainText:(NSData *)plainText - transaction:(YapDatabaseReadWriteTransaction *)transaction +- (NSDictionary *)throws_encryptedMessageForMessageSend:(OWSMessageSend *)messageSend + deviceId:(NSNumber *)deviceId + plainText:(NSData *)plainText + transaction:(YapDatabaseReadWriteTransaction *)transaction { OWSAssertDebug(messageSend); OWSAssertDebug(deviceId); @@ -1658,17 +1655,18 @@ NSString *const OWSMessageSenderRateLimitedException = @"RateLimitedException"; OWSRaiseException(@"SecretSessionCipherFailure", @"Can't create secret session cipher."); } - serializedMessage = [secretCipher encryptMessageWithRecipientId:recipientId - deviceId:deviceId.intValue - paddedPlaintext:[plainText paddedMessageBody] - senderCertificate:messageSend.senderCertificate - protocolContext:transaction - error:&error]; + serializedMessage = [secretCipher throwswrapped_encryptMessageWithRecipientId:recipientId + deviceId:deviceId.intValue + paddedPlaintext:[plainText paddedMessageBody] + senderCertificate:messageSend.senderCertificate + protocolContext:transaction + error:&error]; + SCKRaiseIfExceptionWrapperError(error); messageType = TSUnidentifiedSenderMessageType; } else { // This may throw an exception. id encryptedMessage = - [cipher encryptMessage:[plainText paddedMessageBody] protocolContext:transaction]; + [cipher throws_encryptMessage:[plainText paddedMessageBody] protocolContext:transaction]; serializedMessage = encryptedMessage.serialized; messageType = [self messageTypeForCipherMessage:encryptedMessage]; } @@ -1680,7 +1678,7 @@ NSString *const OWSMessageSenderRateLimitedException = @"RateLimitedException"; device:[deviceId intValue] content:serializedMessage isSilent:isSilent - registrationId:[cipher remoteRegistrationId:transaction]]; + registrationId:[cipher throws_remoteRegistrationId:transaction]]; NSError *error; NSDictionary *jsonDict = [MTLJSONAdapter JSONDictionaryFromModel:messageParams error:&error]; diff --git a/SignalServiceKit/src/Messages/UD/OWSUDManager.swift b/SignalServiceKit/src/Messages/UD/OWSUDManager.swift index 22ab6a428c..ce6686121d 100644 --- a/SignalServiceKit/src/Messages/UD/OWSUDManager.swift +++ b/SignalServiceKit/src/Messages/UD/OWSUDManager.swift @@ -65,8 +65,8 @@ private func string(forUnidentifiedAccessMode mode: UnidentifiedAccessMode) -> S // We use completion handlers instead of a promise so that message sending // logic can access the strongly typed certificate data. @objc - func ensureSenderCertificateObjC(success:@escaping (SMKSenderCertificate) -> Void, - failure:@escaping (Error) -> Void) + func ensureSenderCertificate(success:@escaping (SMKSenderCertificate) -> Void, + failure:@escaping (Error) -> Void) // MARK: Unrestricted Access @@ -109,6 +109,8 @@ public class OWSUDManagerImpl: NSObject, OWSUDManager { guard TSAccountManager.isRegistered() else { return } + + // Any error is silently ignored on startup. self.ensureSenderCertificate().retainUntilComplete() } NotificationCenter.default.addObserver(self, @@ -121,6 +123,7 @@ public class OWSUDManagerImpl: NSObject, OWSUDManager { func registrationStateDidChange() { AssertIsOnMainThread() + // Any error is silently ignored ensureSenderCertificate().retainUntilComplete() } @@ -297,8 +300,8 @@ public class OWSUDManagerImpl: NSObject, OWSUDManager { } @objc - public func ensureSenderCertificateObjC(success:@escaping (SMKSenderCertificate) -> Void, - failure:@escaping (Error) -> Void) { + public func ensureSenderCertificate(success:@escaping (SMKSenderCertificate) -> Void, + failure:@escaping (Error) -> Void) { firstly { ensureSenderCertificate() }.map { certificate in @@ -348,7 +351,7 @@ public class OWSUDManagerImpl: NSObject, OWSUDManager { let anHourFromNowMs = nowMs + kHourInMs do { - try certificateValidator.validate(senderCertificate: certificate, validationTime: anHourFromNowMs) + try certificateValidator.throwswrapped_validate(senderCertificate: certificate, validationTime: anHourFromNowMs) return true } catch { OWSLogger.error("Invalid certificate") diff --git a/SignalServiceKit/src/Network/API/Requests/TSRequest.m b/SignalServiceKit/src/Network/API/Requests/TSRequest.m index b3efd07233..e253706f78 100644 --- a/SignalServiceKit/src/Network/API/Requests/TSRequest.m +++ b/SignalServiceKit/src/Network/API/Requests/TSRequest.m @@ -30,7 +30,7 @@ - (instancetype)init { - OWSRaiseException(NSInternalInconsistencyException, @"You must use the initWithURL: method"); + OWSFail(@"You must use the initWithURL: method"); return nil; } @@ -41,7 +41,7 @@ cachePolicy:(NSURLRequestCachePolicy)cachePolicy timeoutInterval:(NSTimeInterval)timeoutInterval { - OWSRaiseException(NSInternalInconsistencyException, @"You must use the initWithURL method"); + OWSFail(@"You must use the initWithURL: method"); return nil; } diff --git a/SignalServiceKit/src/Network/API/SignalServiceProfile.swift b/SignalServiceKit/src/Network/API/SignalServiceProfile.swift index 4d79540c4c..e1175884f8 100644 --- a/SignalServiceKit/src/Network/API/SignalServiceProfile.swift +++ b/SignalServiceKit/src/Network/API/SignalServiceProfile.swift @@ -32,8 +32,15 @@ public class SignalServiceProfile: NSObject { guard identityKeyWithType.count == kIdentityKeyLength else { throw ValidationError.invalidIdentityKey(description: "malformed identity key \(identityKeyWithType.hexadecimalString) with decoded length: \(identityKeyWithType.count)") } - // `removeKeyType` is an objc category method only on NSData, so temporarily cast. - self.identityKey = (identityKeyWithType as NSData).removeKeyType() as Data + do { + // `removeKeyType` is an objc category method only on NSData, so temporarily cast. + self.identityKey = try (identityKeyWithType as NSData).removeKeyType() as Data + } catch { + // `removeKeyType` throws an SCKExceptionWrapperError, which, typically should + // be unwrapped by any objc code calling this method. + owsFailDebug("identify key had unexpected format") + throw ValidationError.invalidIdentityKey(description: "malformed identity key \(identityKeyWithType.hexadecimalString) with data: \(identityKeyWithType)") + } self.profileNameEncrypted = try params.optionalBase64EncodedData(key: "name") diff --git a/SignalServiceKit/src/Network/OWSCensorshipConfiguration.m b/SignalServiceKit/src/Network/OWSCensorshipConfiguration.m index 943ff7a871..3ddbea908a 100644 --- a/SignalServiceKit/src/Network/OWSCensorshipConfiguration.m +++ b/SignalServiceKit/src/Network/OWSCensorshipConfiguration.m @@ -152,13 +152,11 @@ NSString *const OWSCensorshipConfiguration_DefaultFrontingHost = OWSCensorshipCo NSError *error; NSData *certData = [self certificateDataWithName:certName error:&error]; if (error) { - OWSLogError(@"reading data for certificate: %@ failed with error: %@", certName, error); - OWSRaiseException(@"OWSSignalService_UnableToReadCertificate", @"%@", error.description); + OWSFail(@"reading data for certificate: %@ failed with error: %@", certName, error); } if (!certData) { - OWSLogError(@"No data for certificate: %@", certName); - OWSRaiseException(@"OWSSignalService_UnableToReadCertificate", @"%@", error.description); + OWSFail(@"reading data for certificate: %@ failed with error: %@", certName, error); } [certificates addObject:certData]; } diff --git a/SignalServiceKit/src/Security/OWSHTTPSecurityPolicy.m b/SignalServiceKit/src/Security/OWSHTTPSecurityPolicy.m index ad358de61d..40a5187052 100644 --- a/SignalServiceKit/src/Security/OWSHTTPSecurityPolicy.m +++ b/SignalServiceKit/src/Security/OWSHTTPSecurityPolicy.m @@ -43,7 +43,7 @@ NSString *path = [bundle pathForResource:service ofType:@"cer"]; if (![[NSFileManager defaultManager] fileExistsAtPath:path]) { - OWSRaiseException(@"Missing server certificate", @"Missing signing certificate for service %@", service); + OWSFail(@"Missing signing certificate for service %@", service); } NSData *certificateData = [NSData dataWithContentsOfFile:path]; diff --git a/SignalServiceKit/src/Storage/AxolotlStore/OWSPrimaryStorage+PreKeyStore.m b/SignalServiceKit/src/Storage/AxolotlStore/OWSPrimaryStorage+PreKeyStore.m index b5622c48f8..5035b43839 100644 --- a/SignalServiceKit/src/Storage/AxolotlStore/OWSPrimaryStorage+PreKeyStore.m +++ b/SignalServiceKit/src/Storage/AxolotlStore/OWSPrimaryStorage+PreKeyStore.m @@ -49,7 +49,7 @@ } } -- (PreKeyRecord *)loadPreKey:(int)preKeyId +- (PreKeyRecord *)throws_loadPreKey:(int)preKeyId { PreKeyRecord *preKeyRecord = [self.dbReadConnection preKeyRecordForKey:[self keyFromInt:preKeyId] inCollection:OWSPrimaryStoragePreKeyStoreCollection]; diff --git a/SignalServiceKit/src/Storage/AxolotlStore/OWSPrimaryStorage+SignedPreKeyStore.m b/SignalServiceKit/src/Storage/AxolotlStore/OWSPrimaryStorage+SignedPreKeyStore.m index 48eef0a750..bc9be27514 100644 --- a/SignalServiceKit/src/Storage/AxolotlStore/OWSPrimaryStorage+SignedPreKeyStore.m +++ b/SignalServiceKit/src/Storage/AxolotlStore/OWSPrimaryStorage+SignedPreKeyStore.m @@ -29,14 +29,23 @@ NSString *const OWSPrimaryStorageKeyPrekeyCurrentSignedPrekeyId = @"currentSigne // Signed prekey ids must be > 0. int preKeyId = 1 + arc4random_uniform(INT32_MAX - 1); ECKeyPair *_Nullable identityKeyPair = [[OWSIdentityManager sharedManager] identityKeyPair]; - return [[SignedPreKeyRecord alloc] - initWithId:preKeyId - keyPair:keyPair - signature:[Ed25519 sign:keyPair.publicKey.prependKeyType withKeyPair:identityKeyPair] - generatedAt:[NSDate date]]; + OWSAssert(identityKeyPair); + + @try { + NSData *signature = [Ed25519 throws_sign:keyPair.publicKey.prependKeyType withKeyPair:identityKeyPair]; + return [[SignedPreKeyRecord alloc] initWithId:preKeyId + keyPair:keyPair + signature:signature + generatedAt:[NSDate date]]; + } @catch (NSException *exception) { + // throws_sign only throws when the data to sign is empty or `keyPair` is nil. + // Neither of which should happen. + OWSFail(@"exception: %@", exception); + return nil; + } } -- (SignedPreKeyRecord *)loadSignedPrekey:(int)signedPreKeyId +- (SignedPreKeyRecord *)throws_loadSignedPrekey:(int)signedPreKeyId { SignedPreKeyRecord *preKeyRecord = [self.dbReadConnection signedPreKeyRecordForKey:[self keyFromInt:signedPreKeyId] diff --git a/SignalServiceKit/src/Storage/OWSPrimaryStorage.m b/SignalServiceKit/src/Storage/OWSPrimaryStorage.m index b7000f0f7c..982a721af6 100644 --- a/SignalServiceKit/src/Storage/OWSPrimaryStorage.m +++ b/SignalServiceKit/src/Storage/OWSPrimaryStorage.m @@ -28,8 +28,6 @@ NSString *const OWSUIDatabaseConnectionWillUpdateExternallyNotification = @"OWSU NSString *const OWSUIDatabaseConnectionDidUpdateExternallyNotification = @"OWSUIDatabaseConnectionDidUpdateExternallyNotification"; NSString *const OWSUIDatabaseConnectionNotificationsKey = @"OWSUIDatabaseConnectionNotificationsKey"; -NSString *const OWSPrimaryStorageExceptionName_CouldNotCreateDatabaseDirectory - = @"TSStorageManagerExceptionName_CouldNotCreateDatabaseDirectory"; void VerifyRegistrationsForPrimaryStorage(OWSStorage *storage) { @@ -252,8 +250,7 @@ void VerifyRegistrationsForPrimaryStorage(OWSStorage *storage) NSString *databaseDirPath = [[OWSFileSystem appSharedDataDirectoryPath] stringByAppendingPathComponent:@"database"]; if (![OWSFileSystem ensureDirectoryExists:databaseDirPath]) { - OWSRaiseException( - OWSPrimaryStorageExceptionName_CouldNotCreateDatabaseDirectory, @"Could not create new database directory"); + OWSFail(@"Could not create new database directory"); } return databaseDirPath; } diff --git a/SignalServiceKit/src/Storage/OWSStorage.m b/SignalServiceKit/src/Storage/OWSStorage.m index 285a0f2103..597e51d454 100644 --- a/SignalServiceKit/src/Storage/OWSStorage.m +++ b/SignalServiceKit/src/Storage/OWSStorage.m @@ -26,12 +26,6 @@ NS_ASSUME_NONNULL_BEGIN NSString *const StorageIsReadyNotification = @"StorageIsReadyNotification"; - -NSString *const OWSStorageExceptionName_DatabasePasswordInaccessibleWhileBackgrounded - = @"OWSStorageExceptionName_DatabasePasswordInaccessibleWhileBackgrounded"; -NSString *const OWSStorageExceptionName_DatabasePasswordUnwritable - = @"OWSStorageExceptionName_DatabasePasswordUnwritable"; -NSString *const OWSStorageExceptionName_NoDatabase = @"OWSStorageExceptionName_NoDatabase"; NSString *const OWSResetStorageNotification = @"OWSResetStorageNotification"; static NSString *keychainService = @"TSKeyChainService"; @@ -330,7 +324,7 @@ NSString *const kNSUserDefaults_DatabaseExtensionVersionMap = @"kNSUserDefaults_ // Sleep to give analytics events time to be delivered. [NSThread sleepForTimeInterval:15.0f]; - OWSRaiseException(OWSStorageExceptionName_NoDatabase, @"Failed to initialize database."); + OWSFail(@"Failed to initialize database."); } } } @@ -503,8 +497,7 @@ NSString *const kNSUserDefaults_DatabaseExtensionVersionMap = @"kNSUserDefaults_ { YapDatabaseConnection *dbConnection = self.database.newConnection; if (!dbConnection) { - OWSRaiseException( - @"OWSStorageExceptionName_CouldNotOpenConnection", @"Storage could not open new database connection."); + OWSFail(@"Storage could not open new database connection."); } return dbConnection; } @@ -862,7 +855,7 @@ NSString *const kNSUserDefaults_DatabaseExtensionVersionMap = @"kNSUserDefaults_ // Presumably this happened in response to a push notification. It's possible that the keychain is corrupted // but it could also just be that the user hasn't yet unlocked their device since our password is // kSecAttrAccessibleAfterFirstUnlockThisDeviceOnly - OWSRaiseException(OWSStorageExceptionName_DatabasePasswordInaccessibleWhileBackgrounded, @"%@", errorDescription); + OWSFail(@"%@", errorDescription); } + (void)deleteDBKeys @@ -925,8 +918,7 @@ NSString *const kNSUserDefaults_DatabaseExtensionVersionMap = @"kNSUserDefaults_ // Sleep to give analytics events time to be delivered. [NSThread sleepForTimeInterval:15.0f]; - OWSRaiseException( - OWSStorageExceptionName_DatabasePasswordUnwritable, @"Setting keychain value failed with error: %@", error); + OWSFail(@"Setting keychain value failed with error: %@", error); } else { OWSLogWarn(@"Successfully set new keychain value."); } diff --git a/SignalServiceKit/src/Util/UIImage+OWS.m b/SignalServiceKit/src/Util/UIImage+OWS.m index 8619cbeedf..69c78784fe 100644 --- a/SignalServiceKit/src/Util/UIImage+OWS.m +++ b/SignalServiceKit/src/Util/UIImage+OWS.m @@ -138,7 +138,8 @@ break; default: - OWSRaiseException(NSInternalInconsistencyException, @"Invalid image orientation"); + OWSFailDebug(@"Invalid image orientation"); + return nil; } ///////////////////////////////////////////////////////////////////////////// diff --git a/SignalServiceKit/tests/Storage/TSStoragePreKeyStoreTests.m b/SignalServiceKit/tests/Storage/TSStoragePreKeyStoreTests.m index ad98a0a335..d87c0f4758 100644 --- a/SignalServiceKit/tests/Storage/TSStoragePreKeyStoreTests.m +++ b/SignalServiceKit/tests/Storage/TSStoragePreKeyStoreTests.m @@ -35,10 +35,10 @@ PreKeyRecord *lastPreKeyRecord = [generatedKeys lastObject]; PreKeyRecord *firstPreKeyRecord = [generatedKeys firstObject]; - XCTAssert([[[OWSPrimaryStorage sharedManager] loadPreKey:lastPreKeyRecord.Id].keyPair.publicKey + XCTAssert([[[OWSPrimaryStorage sharedManager] throws_loadPreKey:lastPreKeyRecord.Id].keyPair.publicKey isEqualToData:lastPreKeyRecord.keyPair.publicKey]); - XCTAssert([[[OWSPrimaryStorage sharedManager] loadPreKey:firstPreKeyRecord.Id].keyPair.publicKey + XCTAssert([[[OWSPrimaryStorage sharedManager] throws_loadPreKey:firstPreKeyRecord.Id].keyPair.publicKey isEqualToData:firstPreKeyRecord.keyPair.publicKey]); } @@ -56,8 +56,8 @@ [[OWSPrimaryStorage sharedManager] removePreKey:lastPreKeyRecord.Id]; - XCTAssertThrows([[OWSPrimaryStorage sharedManager] loadPreKey:lastPreKeyRecord.Id]); - XCTAssertNoThrow([[OWSPrimaryStorage sharedManager] loadPreKey:firstPreKeyRecord.Id]); + XCTAssertThrows([[OWSPrimaryStorage sharedManager] throws_loadPreKey:lastPreKeyRecord.Id]); + XCTAssertNoThrow([[OWSPrimaryStorage sharedManager] throws_loadPreKey:firstPreKeyRecord.Id]); } @end