用于防止崩溃的 Objective-C 线程安全代码

Question

我收到了一份带有 SIGSEGV 的 iPhone 崩溃报告，我想我已经缩小了可能的原因并找到了解决方案。 由于线程导致的崩溃很难调试，因此我无法重现此问题，但可以对我的假设提供一些帮助 - 是否合理？

我的代码使用 ASIHttpRequest 通过 ASINetWorkQueue 下载一组文件。这是一个简化的示例

//initialize download queue and do this code block in a loop for each file

NSURL *fileURL = [NSURL URLWithString:...
__block ASIHTTPRequest *fileRequest = [ASIHTTPRequest requestWithURL:fileURL];

[fileRequest setCompletionBlock:^{
   //do some stuff   
}];
[fileRequest setFailedBlock:^{
    NSString *someError = [NSString stringWithFormat:...
    [self someErrorMethod:someError];       
}];

[downloadQueue addOperation:...

---

-(void)someErrorMethod(NSString *errorMessage) {
    DDLogWarn(errorMessage);

    if ([self downloadQueue]) {
        for (ASIHTTPRequest *request in [[self downloadQueue] operations]) {
            [request clearDelegatesAndCancel];
        }
        [[self downloadQueue] reset];
    }
 }

---

崩溃报告的前两行是

• libobjc.A.dylib 0x31846fbc objc_msgSend + 15
• MyApp 0x0002cab5 -[Myapp someErrorMethod:] (MyApp.m:)

---

我对为什么会发生这种情况的想法

• 文件下载失败，调用失败的块
• 它遍历每个请求并清除委托并取消它们，然后重置队列
• 但是，在运行前，另一个文件下载失败，进入失败块回调
• 但是，由于它现在已被取消，其失败的块已被释放
• 当代码尝试记录错误消息时，它的内存已被释放，并且会出现不可预知的结果

这有意义吗？由于我是 Objective-C 的新手，我的分析是正确的还是我遗漏了一些明显的东西？

我正在考虑使用锁来使 errorMethod 线程安全，希望它能解决这个问题。根据上面的代码，这听起来像是正确的解决方案吗？

谢谢

Answer 1

这听起来不太可能。 ASIHttpRequest 可能在同一个线程上执行所有回调（我对此相当肯定）。

如果我不得不猜测，您的错误更有可能出现在这一行：

DDLogWarn(errorMessage);

DDLogWarn 的第一个参数是格式，而不是字符串。在 errorMessage 包含 % 的任何情况下，这都可能会崩溃。你的意思是：

DDLogWarn(@"%@", errorMessage);

---

由于DDLogWarn() 是一个可变参数方法，它将开始用它在堆栈中找到的（随机）值替换字符串中的任何 % 替换值。它将读取堆栈，直到您用完 % 替换。如果任何 % 替换是基于指针的（如 %s 或 %@），那么它将跟随指针指向随机位置。

SEG_ACCERR 表示您请求了一块您不拥有的内存。 SEG_MAPERR 表示您请求了一块未映射的内存。要么是遵循完全随机指针的预期结果。