iOS crash分析实践

背景

分享一些过去两个月遇到的crash。

正文

一、运行时错误

1、UICollectionView的调用顺序

从堆栈可以看出是indexPath无效,通常是indexPath的section或者row超过了数据的大小;

根据堆栈信息和日志信息,可以找到用户操作路径,是通过scheme进入分类;
但是直接用真机复现,相同的操作并不会导致crash;
通过分析crash出现的机型和系统特征,发现都是iOS 13以下系统,而刚刚尝试的是iOS 13的真机;
于是用iOS 12模拟器尝试同样的路径,可以成功复现。
分析原因后,定位到是先调用scrollToItemAtIndexPath,再调用reloadData导致的异常。
解决方案也很简单,调整为正确的顺序即可。

寻找复现路径的时候,要尽量从两个方向去复现:
1、用户设备条件,包括iOS系统版本、iphone机型、网络环境等要保持一致;
2、App运行上下文,包括App版本、操作路径、运行环境等;

2、HTML转码NSAttributedString耗时过长

业务需要把html格式的字符串转成NSAttributedString,原来HTML转码成NSAttributedString使用的是系统自带的方法:

- (nullable instancetype)initWithData:(NSData *)data options:(NSDictionary<NSAttributedStringDocumentReadingOptionKey, id> *)options documentAttributes:(NSDictionary<NSAttributedStringDocumentAttributeKey, id> * __nullable * __nullable)dict error:(NSError **)error;

举一个实际的例子:

- (void)viewDidLoad {
    [super viewDidLoad];
    // Do any additional setup after loading the view, typically from a nib.

    NSString *pathStr = [[NSBundle mainBundle] pathForResource:@"test" ofType:@"data"];
    NSString *htmlStr = [NSString stringWithContentsOfURL:[NSURL fileURLWithPath:pathStr] encoding:NSUTF8StringEncoding error:nil];

    NSLog(@"load string:%@", htmlStr);
    NSData *htmlData = [NSData dataWithBytes:htmlStr.UTF8String length:[htmlStr lengthOfBytesUsingEncoding:NSUTF8StringEncoding]];
    NSDictionary *dic = @{NSDocumentTypeDocumentAttribute:NSHTMLTextDocumentType,
                          NSCharacterEncodingDocumentAttribute:[NSNumber numberWithInt:NSUTF8StringEncoding]};
    NSDate *date = [NSDate date];
    NSAttributedString *attrStr = [[NSAttributedString alloc] initWithData:htmlData options:dic documentAttributes:nil error:nil];

    NSLog(@"transform time:%f", [[NSDate date] timeIntervalSinceDate:date]);
}

这个逻辑测试阶段一切正常, 直到遇到一段HTML文本,其中带有img标签。
这段HTML文本在转码的时候会同步对图片资源进行加载,导致线程阻塞,如果阻塞时间过长,还会引发crash。
堆栈如下:


此时,再回顾系统api的解释,才有了更深刻的理解:

It will try to synchronize with the main thread, fail, and time out. Calling it from the main thread works (but can still time out if the HTML contains references to external resources, which should be avoided at all costs).

解决方案1、转码前,手动过滤掉<img>的标签;
解决方案2、改用DTCoreText的html转NSAttributedString;

使用一个不熟悉的系统API接口,最好花时间阅读下接口说明;

二、多线程

1、dispatch_once引起死锁

dispatch_once常被用于iOS单例实现方法,也会被用来处理某些只需要执行一次的场景,比如下文的获取某些tag的方法:

+ (NSInteger)coolCommentTag { 
    static NSInteger ret = 5; // 实验写死默认值,客户端不关心,透传给后台 
    static dispatch_once_t onceToken; 
    dispatch_once(&onceToken, ^{ 
        NSDictionary *abDict = [GET_SERVICE(SSTestManager) getExperimentValueForKey:@"social_comment_tag" withExposure:YES]; 
        if (SSIsValidatedDict(abDict)) { 
            ret = [abDict bda_integerValueForKey:@"comment_tag"]; 
        } 
    });      
    return ret; 
} 

这段代码会造成一个crash:

堆栈关键信息:dispatch_gate_wait_slow;
注意到上图,crash的是子线程,这时候要习惯性看看主线程在处理什么逻辑

从这两个堆栈信息,我们基本可以确定是因为多线程死锁导致的卡死。
场景分析:
1、主线程:did注册 =》TTNet初始化 =》发起网络请求 =》 获取公共参数 =》 dispatch_once ;
2、子线程:RPC(网络层)初始化 =》初始化 =》 获取公共参数 =》 dispatch_once ;
然后发生死锁

修复方式:去掉dispatch_once,改用线程安全的方式;

经验总结:
网络层相关接口有可能是子线程发起;
dispatch_once内的代码要尽可能简单;
子线程发生crash时,要习惯性看看主线程;

三、内存相关

1、dealloc访问weak指针

weak指针经常被应用到在代理、block避免循环引用,避免有一个特性就是对象dealloc的时候,指向对象的weak指针会被置为nil。

但是使用不当的时候,weak指针也很容易造成crash,如下图:


场景举例:
1、书架某个按钮,使用getter的方式获取;getter方法中用block去响应按钮点击,block因为持有了外部self,所以会用weak-strong-dance;如果在dealloc方法访问到该按钮,则会发生crash;
2、详情页的某个view,使用getter的方式获取,在dealloc方法时访问了该getter;

getter实现

修复方式

经验总结:
1、getter实现应该简单化,尽量少的去设置很多属性和创建block等,仅仅作为懒加载去创建对象;
2、dealloc方法不要访问getter和setter的方法;

思考题,为什么对象dealloc创建对象的weak指针会crash?

2、子线程释放对象

block是常见的回调方式,当我们调用某个异步方法,想在回调时继续保持运行上下文,就会传入一个回调block,等到就绪时再执行block。
比如说常见的rpc请求:

[SSBookApiService getBookToneInfo:^(SSBookToneInfoRequest *request) {
        request.bookId = bookId;
    } completion:^(BDRpcError *error, SSBookToneInfoResponse *response) {
        XXX.XXX
    }
}];

block保持上下文的方式,就是持有block中所有访问到的外部对象。
这种持有特性可能会导致一些意想不到的情况,比如说在子线程销毁一个对象

如图,子线程的堆栈都是在dealloc方法,一层层递归。

通过代码分析,可以知道是BDRpcAsyncOperation持有了某个block;该block持有了其他对象(因为block中访问了该对象)。
结果BDRpcAsyncOperation在子线程销毁时,dealloc方法层层递进,最终触发了某个对象在子线程销毁,而该对象会在dealloc方法处理一些UI相关的逻辑。

修复方法:
1、block访问到的外部对象,非局部变量尽可能使用weak-strong的方式来声明;用weak指针来声明外部的变量,如果该对象在block回调前被释放,则会变为nil;
2、如果需要block持有该对象,则可以在dealloc的时候使用dispatch_async的方式,放到主线程去执行;(不能访问self相关)

总结

1、寻找复现路径的时候,要尽可能还原现场,才能更好复现;
2、使用一个不熟悉的系统API接口,最好花时间阅读下接口说明;
3、子线程发生crash时,要习惯性看看主线程;

©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 211,948评论 6 492
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 90,371评论 3 385
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 157,490评论 0 348
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 56,521评论 1 284
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 65,627评论 6 386
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 49,842评论 1 290
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 38,997评论 3 408
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 37,741评论 0 268
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 44,203评论 1 303
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 36,534评论 2 327
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 38,673评论 1 341
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 34,339评论 4 330
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 39,955评论 3 313
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 30,770评论 0 21
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 32,000评论 1 266
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 46,394评论 2 360
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 43,562评论 2 349