静态音频分析功能中的间歇性EXC_BAD_ACCESS

Question

我正在编写一个使用实时音频分析的iOS应用程序。它有一个间歇性的崩溃（大约5分钟后，仅在模拟器上。虽然这意味着它不受运输应用程序的关注，但它确实是一个开发过程中的痛苦，此外，如果我追踪它，我晚上会睡得更好。崩溃总是发生在同一个地方，在我的音频分析代码中的一个静态函数中：崩溃发生在这里：

struct Tone {
// (other stuff)
// THIS is the problem function:
static bool dbCompare(Tone const& l, Tone const& r) { return l.db < r.db; }
  }
};

还有这里：

Tone const* findTone(double minfreq = 70.0, double maxfreq = 700.0) const {
    if (m_tones.empty()) { m_oldfreq = 0.0; return NULL; }
    double db = std::max_element(m_tones.begin(), m_tones.end(), Tone::dbCompare)->db;
    Tone const* best = NULL;
    double bestscore = 0;
    for (tones_t::const_iterator it = m_tones.begin(); it != m_tones.end(); ++it) {
    // ON THIS LINE. Is my iterator somehow becoming invalid?:
        if (it->db < db - 20.0 || it->freq < minfreq || it->age < Tone::MINAGE) continue;
        if (it->freq > maxfreq) break;
        double score = it->db - std::max(180.0, std::abs(it->freq - 300.0)) / 10.0;
        if (m_oldfreq != 0.0 && std::fabs(it->freq/m_oldfreq - 1.0) < 0.05) score += 10.0;
        if (best && bestscore > score) break;
        best = &*it;
        bestscore = score;
    }
    m_oldfreq = (best ? best->freq : 0.0);
    return best;
}

（好吧，下面的这个修复程序是错误的，@Justin。我把它留在这里是为了完整，至少现在是这样）。为了在可能传入的任何空指针中幸存下来，我添加了一些错误处理检查和尝试：

struct Tone {
    static bool dbCompare(Tone const& l, Tone const& r) {
    try {
      if (&l == NULL || &r == NULL) throw 1;
      return l.db < r.db;
      throw 1;
    }
    catch(int e) {
      NSLog(@"AUDIO ERROR: Pointer invalid. %d", e);
    }
    return 0;
  }
};

没有骰子，它仍然在同一条线上失败。我是不是误解了接球投掷？错误只是在管道的后面发生吗？

这段代码来自开源摇滚乐队风格的游戏Performous，他们的游戏没有这个问题。（这也不适用于iPhone。）所以这给了我希望，我可以消除模拟器崩溃。。。

第2版Per@Justin我和GuardMalloc一起跑步。模拟器5.0会立即崩溃——对此似乎有一些SO问题，但目前我正在运行4.3模拟器，它以正常的方式给出崩溃，但我不知道如何使用：

#0  0x94f38c97 in malloc_error_break ()
#1  0x94efa4ce in szone_error ()
#2  0x94efc35f in allocate_pages ()
#3  0x94f013cc in allocate_pages_securely ()
#4  0x94f019b8 in szone_malloc_should_clear ()
#5  0x94f0266b in szone_malloc ()
#6  0x00331cc3 in GMmalloc_zone_malloc ()
#7  0x94f38edb in malloc_set_zone_name ()
#8  0x94f394a8 in _malloc_initialize ()
#9  0x94f3986b in malloc ()
#10 0x00002ef0 in start ()
#11 0x00013f1d in std::_List_const_iterator<Tone> std::max_element<std::_List_const_iterator<Tone>, bool (*)(Tone const&, Tone const&)>(std::_List_const_iterator<Tone>, std::_List_const_iterator<Tone>, bool (*)(Tone const&, Tone const&)) at /Developer/of_007_iphone/apps/cwi007/SingXO/SingXO/Pitch/pitch.hh:25
#12 0x000139b2 in Analyzer::findTone(double, double) const ()
#13 0x00011950 in -[AppDelegate timerCallback:] ()
#14 0x00115308 in -[CCTimer update:] ()
#15 0x0011e124 in -[CCScheduler tick:] ()
#16 0x0014993a in -[CCDirectorIOS drawScene] ()
#17 0x0014bda4 in -[CCDirectorDisplayLink mainLoop:] ()
#18 0x007baa88 in CA::Display::DisplayLink::dispatch(unsigned long long, unsigned long long) ()
#19 0x007babcd in CA::Display::EmulatorDisplayLink::callback(__CFRunLoopTimer*, void*) ()
#20 0x019c88c3 in __CFRUNLOOP_IS_CALLING_OUT_TO_A_TIMER_CALLBACK_FUNCTION__ ()
#21 0x019c9e74 in __CFRunLoopDoTimer ()
#22 0x019262c9 in __CFRunLoopRun ()
#23 0x01925840 in CFRunLoopRunSpecific ()
#24 0x01925761 in CFRunLoopRunInMode ()
#25 0x020bb1c4 in GSEventRunModal ()
#26 0x020bb289 in GSEventRun ()
#27 0x00beac93 in UIApplicationMain ()
#28 0x00002fb6 in main ()

我也在控制台上得到了这个：

GuardMalloc[MyApp-723]: *** mmap(size=2097152) failed (error code=12)
*** error: can't allocate region
*** set a breakpoint in malloc_error_break to debug

哦，当然，这是NOW似乎是罪魁祸首的功能：

- (void)timerCallback:(NSTimer *)timer {
  Tone const * tone = audio->analyzer->findTone();
  if (tone && tone->db > kToneMinDB) {
    self.currentPitch = tone->freq;
    self.currentVolume = tone->stabledb;
    self.currentNoteName = [NSString stringWithCString:scale->getNoteStr(tone->freq).c_str() encoding:NSUTF8StringEncoding];
//    NSLog(@"Current Note Name in timerCallback: %@", currentNoteName);
    self.currentNoteFunction = [pitchFilter getPitchFunctionFromPitchName:currentNoteName];
  } else {
    self.currentNoteName = @"--";
    self.currentNoteFunction = -1;
  }

}

我不明白程序是如何在没有有效音调的情况下进入if (tone)块的，或者相反，如果音调有效，它会如何崩溃。帮助

（这越来越长了。我是否应该删除其中的一些，以服务于一个更集中的问题？）

我在想：1.这是一个线程问题：：不寒而栗：一个线程正在更改音频数据，而另一个线程在读取它。2.timerCallback函数中的某些东西正在损坏堆栈，我不知道，因为它的分配太频繁了。这个函数被调用了很多，但不是一个极端的数量；它被安排为每150毫秒。

不确定为什么这两种情况在模拟器中都会出现问题，但在设备上却不会。

Answer 1

C++引用永远不能是0/NULL。如果是的话，你已经进入了未定义的领域。如果参数可能是0，则必须在执行链中向上移动测试。

也就是说，唯一形式良好的dbCompare版本看起来是这样的：

struct Tone {
    static bool dbCompare(Tone const& l, Tone const& r) {
        return l.db < r.db;
    }
};

更新

如果在迭代时调整迭代器的大小，那么它将是无效的——我认为在这个实现中不会发生这种情况。你是从另一个线程调整容器的大小吗？

此外，试着用GuardMalloc运行它——它旨在让这些问题对你来说显而易见。通过更快地失败，它将更加本地化（在许多情况下）。

Answer 2

最佳=&it行在我看来很奇怪。容器是通过引用存储还是通过值存储？看起来您可能正在返回临时存储的地址。这可能会带来问题。