qiuzijian / google-breakpad

Automatically exported from code.google.com/p/google-breakpad
0 stars 0 forks source link

dump_syms dead loop #355

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
tools/linux/dump_syms dead loop

**maybe** because in common/linux/stabs_reader.cc
bool StabsReader::Process() {
  symbol_ = symbols_;
  while (symbol_ < symbols_end_) {
    if (symbol_->n_type == N_SO) {
      if (! ProcessCompilationUnit())
        return false;
// ---------- added begin -----------------
      else
        symbol_++;
// ---------- added end -----------------
    } else
      symbol_++;
  }
  return true;
}

first, I changed dump_syms Makefile, removed -m32
and execute ./dump_syms myapp*
then dead loop.

*myapp is build with -gstabs+

Original issue reported on code.google.com by ken.s...@gmail.com on 7 Dec 2009 at 4:32

GoogleCodeExporter commented 9 years ago
I think I hit this same problem. You're running on x86-64?

Jim: I guess this is your code now. Latest trunk builds on x86-64, but 
dump_syms hangs.

Original comment by ted.mielczarek on 9 Dec 2009 at 8:49

GoogleCodeExporter commented 9 years ago
I submitted a patch for that, it's been waiting for review:

http://breakpad.appspot.com/25003/show

Original comment by jimbla...@gmail.com on 10 Dec 2009 at 6:33

GoogleCodeExporter commented 9 years ago

Original comment by jimbla...@gmail.com on 10 Dec 2009 at 6:34

GoogleCodeExporter commented 9 years ago
This should be fixed in r443.

Original comment by jimbla...@gmail.com on 15 Dec 2009 at 4:34