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

我收到了一份带有 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

---

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

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

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

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

谢谢

---

Best Answer-推荐答案

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

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

DDLogWarn(errorMessage);

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

DDLogWarn(@"%@", errorMessage);

---

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

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

关于用于防止崩溃的 Objective-C 线程安全代码，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/9364539/