Use of uninitialised value in libbacktrace/pecoff.c

[ranma42]

In coff_add(), str_size is declared as a size_t, but if there is a symbol table, only 4 bytes of it are initialised:

memcpy (&str_size, syms_view.data + syms_size, 4);

str_size should probably be declared as a uint32_t.

The bug at rust-lang/rust#28447 was caused by this.
It was fixed in the Rust local copy of libbacktrace: rust-lang/rust@55e2b7e

This was reported at https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69314

[dscho]

Is this not addressed by #6? That Pull Request sits unanswered for months already, and it looks pretty good from my side, so I have no idea why you don't just click "Merge" @ianlancetaylor...