例子一
堆栈信息
image.png
根据堆栈分析：
1，野指针
2，有对应的堆栈
查看堆栈代码，看那些有可能野指针：
+ (NSURLSessionDataTask *)httpAsyncPostWithUrl:(NSString *)urlHost
dictionary:(NSDictionary *)dictionary
userAgent:(NSString *)userAgent
completionBlock:(BDHKNetworkServiceResponse)block {
[self setupNetworkService];
// 检查网络
if ([BDHKNetworkInfoManager currentNetworkStatus] == BDHKNetworkInfoStatusNotReachable) {
if (block) {
NSError *error = [NSError errorWithDomain:kBDHKNetworkingErrorDomain
code:kBDHKNetworkingUnreachableCode
userInfo:nil];
// [BDHKCoreDataCheck apiErrorCheckApi:urlHost tab:@"haokan" tag:@"haokan" videoType:@"video" vid:@"" error:error extraDict:@{}];
block(nil, error);
}
return nil;
}
if ([Pyramid.bdhk_commonParams boolUploadZid]) {
[Pyramid.bdhk_commonParams zid]; //更新zid
}
NSTimeInterval startTime = [[NSDate date] timeIntervalSince1970];
__block NSDictionary *timeDic = nil;
BDHKNetworkingResult resultBlock = ^(NSURLSessionTask * _Nullable task,
NSDictionary * _Nullable response,
NSError * _Nullable error) {
// passposrt下沉后已修改
if (!HK_isEmptyDictionary(response)) {
[[BDHKCorePassportService sharedInstance] updatePassportStatusWithLoginInfo:response];
}
// if (!HK_isEmptyDictionary(response)) {
// @try {
// id obj = [NSClassFromString(@"BDHKPassportService") performSelector:@selector(sharedInstance)];
// [obj performSelector:@selector(updatePassportStatusWithLoginInfo:) withObject:response afterDelay:0];
// } @catch (NSException *exception) {
//
// }
// }
if (error.code == kBDHKNetworkingUntrustCerErrorCode) {
bdhk_toast(@"检测到代理连接，关闭代理后可正常使用");
NSDictionary *errorInfo = @{@"k": @"hk_stability",
@"v": @"untrust_user_cer",
@"url": urlHost};
[BDHKNewLogService send760PackLogWithLogExtra:@{@"extra": errorInfo}];
NSString * apiName = [dictionary.allKeys objectAtIndexCheck:0];
[BDHKCoreDataCheck apiErrorCheckApi:apiName?:urlHost tab:@"haokan" tag:@"haokan" videoType:@"video" vid:@"" error:error extraDict:errorInfo];
}
if (block) {
block(response, error);
}
// 打点
NSTimeInterval responseTime = [[NSDate date] timeIntervalSince1970];
BOOL isTurbo = [BDHKNetworking isTurboSession:task.bdhkSession];
NSInteger errorCode = error ? error.code : 0;
[BDHKNetWorkService apiRequestTime:startTime
type:isTurbo ? @"turbo" : @"native"
code:errorCode
message:error.description
response:task.response
dictionary:dictionary
responseObject:response
timeDic:timeDic
url:urlHost
responseTime:responseTime];
if (error && error.code != NSURLErrorCancelled) {
BOOL isEmpty = HK_isEmptyDictionary(response);
[BDHKNetWorkService pageErrorLogDic:dictionary
type:isEmpty ? @"2" : @"0"
error:error];
NSString * apiName = [dictionary.allKeys objectAtIndexCheck:0];
[BDHKCoreDataCheck apiErrorCheckApi:apiName?:urlHost tab:@"haokan" tag:@"haokan" videoType:@"video" vid:@"" error:error extraDict:@{}];
}
};
NSURLSessionDataTask *dataTask = nil;
if (@available(iOS 10.0, *)) {
dataTask = [BDHKNetworking postHost:urlHost
apiAndParameters:dictionary
userAgent:userAgent
completionBlock:[resultBlock copy]
metricsBlock:^(NSURLSessionTask * _Nullable task,
NSURLSessionTaskMetrics * _Nullable metrics) {
timeDic = [self dicFromMetrics:metrics];
}];
} else {
dataTask = [BDHKNetworking postHost:urlHost
apiAndParameters:dictionary
userAgent:userAgent
completionBlock:[resultBlock copy]];
}
if ([dictionary objectForKey:@"feed"] != nil) {
dataTask.priority = NSURLSessionTaskPriorityHigh; // 将feed请求调为最高优先级
}
return dataTask;
}
分析所有参数：
urlHost，dictionary，userAgent， block， startTime， timeDic，dataTask
以上所有参数，除了timeDic，其他的都不存在多线程读写安全的问题，因为他们要不就是个真正的局部变量，或者不存在一个线程在读另一个线程在写的可能
只有timeDic，在当前函数的下方有赋值的写操作，但是在block里面有读操作，并且不能保证当前函数的线程和block回调的线程是在同一个线程，所以是有造成多线程读写crash的
解决方案
对timeDic 读写进行加锁操作
@synchronized (self) {
timeDic = [self dicFromMetrics:metrics];
}
读操作这里用了copy，因为这个方法调用用到了timeDic，没法直接加锁
如果强行加锁只能把整个方法加锁
@synchronized (self) {
[BDHKNetWorkService apiRequestTime:startTime
type:isTurbo ? @"turbo" : @"native"
code:errorCode
message:error.description
response:task.response
dictionary:dictionary
responseObject:response
timeDic:tmpTimeDic
url:urlHost
responseTime:responseTime];
}
这种加锁方式风险极高，因为你不知道这个方法实现内部是否有耗时操作，如果有，这里就会造成阻塞，所以把timeDic直接加锁拷贝，因为方法内部没有对timeDic修改，只是获取内容，没有修改的操作，所以copy也不影响，最终修改方案如下：
NSDictionary *tmpTimeDic = nil;
@synchronized (self) {
tmpTimeDic = [timeDic copy];
}
[BDHKNetWorkService apiRequestTime:startTime
type:isTurbo ? @"turbo" : @"native"
code:errorCode
message:error.description
response:task.response
dictionary:dictionary
responseObject:response
timeDic:tmpTimeDic
url:urlHost
responseTime:responseTime];
Q&A
多线程同时读写为什么会造成crash:
1，基本数据类型（非指针类型）：不会造成crash，就是读取到的数据可能不对，不一致
2，指针类型：由于指针类型是直接赋值的操作，多线程多写，a线程读的时候，b线程可能刚好给重新赋值，因此a线程拿到的指针就指向了一个异常内存，造成crash
