NSMutableDictionary внутри JSONModel - EXC_BAD_ACCESS KERN_INVALID_ADDRESS

Crashlytics сообщила об этом сбое в одном из моих приложений, и я не могу воспроизвести его вообще, независимо от того, что я делаю. Это происходит примерно с 5% пользователей, поэтому это довольно большое дело. Я публикую скриншоты с сообщением о сбоях, а также методы, упомянутые в отчете о сбое. Любая идея, как это решить?

Отчет о сбоях

Здесь произошло сбой приложения:

#pragma mark - custom transformations
-(BOOL)__customSetValue:(id<NSObject>)value forProperty:(JSONModelClassProperty*)property
{
    if (!property.customSetters)
        property.customSetters = [NSMutableDictionary new];

    NSString *className = NSStringFromClass([JSONValueTransformer classByResolvingClusterClasses:[value class]]);

    if (!property.customSetters[className]) {
        //check for a custom property setter method
        NSString* ucfirstName = [property.name stringByReplacingCharactersInRange:NSMakeRange(0,1)
                                                                       withString:[[property.name substringToIndex:1] uppercaseString]];
        NSString* selectorName = [NSString stringWithFormat:@"set%@With%@:", ucfirstName, className];

        SEL customPropertySetter = NSSelectorFromString(selectorName);

        //check if there a custom selector like this
        if (![self respondsToSelector: customPropertySetter]) {
            property.customSetters[className] = [NSNull null]; // this is line 855
            return NO;
        }

        //cache the custom setter selector
        property.customSetters[className] = selectorName;
    }

    if (property.customSetters[className] != [NSNull null]) {
        //call the custom setter
        //https://github.com/steipete
        SEL selector = NSSelectorFromString(property.customSetters[className]);
        ((void (*) (id, SEL, id))objc_msgSend)(self, selector, value);
        return YES;
    }

    return NO;
}

Это исходный метод:

-(void)reloadUserInfoWithCompletion:(void (^) (LoginObject *response))handler andFailure:(void (^)(NSError *err))failureHandler {
    NSString *lat;
    NSString *lon;

    lat = [NSString stringWithFormat:@"%.6f",[[LocationManager sharedInstance] getPosition].coordinate.latitude];
    lon = [NSString stringWithFormat:@"%.6f",[[LocationManager sharedInstance] getPosition].coordinate.longitude];

    NSMutableDictionary *params = [NSMutableDictionary new];
    [params setObject:lat forKey:@"latitude"];
    [params setObject:lon forKey:@"longitude"];

    [[LoginHandler sharedInstance] getLoginToken:^(NSString *response) {

        NSDictionary *headers;
        if (response) {
            headers = @{@"Login-Token":response};
        }
        GETRequest *req = [GETRequest new];
        [req setCompletionHandler:^(NSString *response) {
            dispatch_async(dispatch_get_global_queue(DISPATCH_QUEUE_PRIORITY_DEFAULT, 0), ^{
                NSLog(@"response: %@",response);
                NSError *err = nil;
                self.loginObject.userDetails = [[User alloc] initWithString:response error:&err]; // <- this is the line reported in the crash
                [self storeLoginObject];
                NSLog(@"%@",self.loginObject.userDetails);
//                [Utils updateFiltersFullAccessIfAll]; 
                dispatch_async(dispatch_get_main_queue(), ^{
                    if (handler) {
                        handler(self.loginObject);
                    }
                });
            });
        }];
        [req setFailedHandler:^(NSError *err) {
            if (failureHandler) {
                failureHandler(err);
            }
        }];
        NSLog(@"%@",params);
        [req requestWithLinkString:USER_DETAILS parameters:nil andHeaders:headers];
    }];

}

Ответы

Ответ 1

Итак, setObject:forKey: может вызвать проблемы двумя способами. 1. Если object - nil или 2. key - nil. Оба могут привести к сбою, который вы видите. Учитывая, что вы устанавливаете значение object на [NSNull null], вероятно, можно с уверенностью предположить, что это проблема key (строка 855).

Отход оттуда, который показал бы, что className - nil. Если вы посмотрите, ваш код не защитит от этого. Вы делаете предположение, что NSStringFromClass (пара строк до) возвращает вам допустимую строку, которая предполагает, что value, первоначально переданный в метод, не имеет значения nil. Если он nil, он пропустит все ваши проверки, включая !property.customSetters[className], так как это будет !nil, позволяя ему ввести if.

Если я читаю ваш код правильно (немного сложно, так как я не могу проверить какие-либо мои предположения) NSLog(@"response: %@",response); будет печатать ответ nil.

Попробуйте увидеть, как ваш код обрабатывает эти неожиданные nil, и дайте мне знать в комментариях, как это происходит.

Ответ 2

Если вы не пользуетесь настраиваемыми настройками модели, вы можете заменить JSONModel __customSetValue: forProperty: с помощью swizzling или Aspects library

#import "JSONModel+Aspects.h"
#import "JSONModel.h"
#import "Aspects.h"

@implementation JSONModel (Aspects)

+(void)load {
    static dispatch_once_t onceToken;
    dispatch_once(&onceToken, ^{
        [JSONModel aspect_hookSelector:@selector(__customSetValue:forProperty:) withOptions:AspectPositionInstead usingBlock:^(id<AspectInfo> aspectInfo) {
            return NO;
        } error:NULL];
    });
}

@end