• 微信公众号:美女很有趣。 工作之余,放松一下,关注即送10G+美女照片!

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

开发技术 开发技术 4周前 (05-14) 10次浏览

问题描述

线上版本从ios 13.x 之后出现很多afn相关的crash,在 AFNetworking 的 github 上对应的issue#4591,我在这issue下也添加我的评论的解决demo
下面是crash堆栈,有些crash并不只是下面这种最终crash在objc_retain,有些则是objc_releaseobjc_msgSend之类的
random crash at iOS 13.x AFNetworking解决 - 多线程争抢资源

我使用的网络框架

我项目中使用的是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之前的系统没有这个问题,所以不知道系统发生了什么变化。

相关Demo

https://github.com/LiJinShi/NetWorkingBug


程序员灯塔
转载请注明原文链接:random crash at iOS 13.x AFNetworking解决 – 多线程争抢资源
喜欢 (0)