On Sun, Jul 31, 2016 at 10:00 AM, Stefan Kempf <[email protected]> wrote: > clang errors out about use of undefined variables when building > binutils-2.17. > > In elf.c, do not increment `s' before it is initialized. At the time > of the increment, `s' is otherwise unused anyway.
Indeed, it's dead there. > In elflink.c, initialize sec_contents and l_sec_contents to make > sure that the free(sec_contents) and free(l_sec_contents) are called > on valid pointers. clang only complained about l_sec_contents, right? sec_contents is always set by bfd_malloc_and_get_section() ...but initializing both is clearer, yes. ok guenther@
