https://sourceware.org/bugzilla/show_bug.cgi?id=30354
Nick Clifton <nickc at redhat dot com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |RESOLVED Resolution|--- |FIXED --- Comment #11 from Nick Clifton <nickc at redhat dot com> --- (In reply to Torbjörn SVENSSON from comment #9) Hi Torbjörn > > if (!cmse_sec->gc_mark > > && !_bfd_elf_gc_mark (info, cmse_sec, > > gc_mark_hook)) > > return false; > > /* The debug sections related to these secure entry > > functions are marked on enabling below flag. */ > > debug_sec_need_to_be_marked = true; > > break; > Is it correct to break the for-loop here? > When the break statement is there, then only the first entry in the > sym_hashes array that matches the CMSE_PREFIX criteria will passed on to the > _bfd_elf_gc_mark function. Isn't it required to call this function for every > cmse_sec value or would it be enough with just the first value? You are correct. That was another mistake. I have fixed that too and checked in the patch. Cheers Nick -- You are receiving this mail because: You are on the CC list for the bug.