random crash at iOS 13.x AFNetworking解决 - 多线程争抢资源

时间：2021-05-24 09:00:05 阅读：0 评论：0 收藏：0 [点我收藏+]

标签：and 一个 target 方式无法框架相关 mamicode sleep

问题描述

线上版本从ios 13.x 之后出现很多afn相关的crash，在 AFNetworking 的 github 上对应的issue#4591，我在这issue下也添加我的评论的解决demo
下面是crash堆栈，有些crash并不只是下面这种最终crash在objc_retain，有些则是objc_release和objc_msgSend之类的
技术图片

我使用的网络框架

我项目中使用的是XMNetworking，它内部也是基于AFN的封装。

问题分析

从观察来看，所有这个crash都是在子线程，且objc_retain objc_release都属于内存方面问题，objc_retain极有可能是引用了一个已释放的对象，objc_release像是对一个已释放的对象进行了多次释放，objc_msgSend像是发送消息的对象和创建时匹配不上。所以要想方式找到复现的方式，然后再考虑怎么利用加锁进行解决，这一步也是经历了很久。

问题复现与解决

1.尝试加锁：修改XMEngine.m 文件

- (void)xm_processResponse:(NSURLResponse *)response
                    object:(id)responseObject
                     error:(NSError *)error
                   request:(XMRequest *)request
         completionHandler:(XMCompletionHandler)completionHandler {
    NSError *serializationError = nil;
    if (request.responseSerializerType != kXMResponseSerializerRAW) {
        XMLock();
        AFHTTPResponseSerializer *responseSerializer = [self xm_getResponseSerializer:request];
        responseObject = [responseSerializer responseObjectForResponse:response data:responseObject error:&serializationError];
        XMUnlock();
    }
    
    if (completionHandler) {
        if (serializationError) {
            completionHandler(nil, serializationError);
        } else {
            completionHandler(responseObject, error);
        }
    }
}

可以解决这个问题，但是还是没有定位到哪个具体问题，也就是加锁的代码太多了，也影响性能。
2.考虑复现问题，加入测试代码，XMEngine 的 - (void)xm_dataTaskWithRequest:(XMRequest *)request completionHandler:(XMCompletionHandler)completionHandler 方法中加入处理

    dataTask = [sessionManager dataTaskWithRequest:urlRequest
                                    uploadProgress:nil
                                  downloadProgress:nil
                                 completionHandler:^(NSURLResponse *response, id responseObject, NSError *error) {
                                     __strong __typeof(weakSelf)strongSelf = weakSelf;
        
#warning test code Simulate network multithreading return
                                    dispatch_queue_t queue = dispatch_queue_create(@"test_queue", DISPATCH_QUEUE_CONCURRENT);
                                    for (NSInteger i = 0; i < 300; i ++) {
                                        dispatch_async(queue, ^{
                                            [strongSelf xm_processResponse:response
                                                                    object:responseObject
                                                                     error:error
                                                                   request:request
                                                         completionHandler:completionHandler];
                                        });
                                    }
                                     
                                 }];

同时考虑到对象创建问题，在懒加载时加入延时处理，以让多线程进入。

- (AFJSONRequestSerializer *)afJSONRequestSerializer {
    if (!_afJSONRequestSerializer) {
#warning test code
        [NSThread sleepForTimeInterval:0.2];
        
        _afJSONRequestSerializer = [AFJSONRequestSerializer serializer];
        
    }
    return _afJSONRequestSerializer;
}

- (AFJSONResponseSerializer *)afJSONResponseSerializer {
    if (!_afJSONResponseSerializer) {
#warning test code
        [NSThread sleepForTimeInterval:0.2];
        
        
        _afJSONResponseSerializer = [AFJSONResponseSerializer serializer];
        // Append more other commonly-used types to the JSON responses accepted MIME types.
        //_afJSONResponseSerializer.acceptableContentTypes = [NSSet setWithObjects:@"application/json", @"text/json", @"text/javascript", @"text/html", @"text/plain", nil];
    }
    return _afJSONResponseSerializer;
}

看到了对应的问题日志
*** -[AFJSONResponseSerializer release]: message sent to deallocated instance 0x6000035132d0

优化解决

XMEngine 里的所有懒加载加锁处理，下面这样处理之后，再用上面方式进行测试，发现无法复现了，且我线上版本也使用这种方式之后，再也没有看到类似的bug.

- (AFHTTPRequestSerializer *)afHTTPRequestSerializer {
    static dispatch_once_t onceToken;
    dispatch_once(&onceToken, ^{
        _afHTTPRequestSerializer = [AFHTTPRequestSerializer serializer];
    });
    return _afHTTPRequestSerializer;
}

- (AFJSONRequestSerializer *)afJSONRequestSerializer {
    static dispatch_once_t onceToken;
    dispatch_once(&onceToken, ^{
        _afJSONRequestSerializer = [AFJSONRequestSerializer serializer];
    });
    return _afJSONRequestSerializer;
}

总结

其实最后来看这个Bug其实和AFN可能没有关系，是我们在使用单例创建afn的时候需要注意线程安全，但是很奇怪ios 13之前的系统没有这个问题，所以不知道系统发生了什么变化。