Re: rtld dropping core on recent -current

On Thu, Aug 02, 2012 at 09:55:36PM -0400, Alexander Kabaev wrote:
> It is weird that program tries to dlopen what appears to be the binary
> (itself?), but that did uncover the issue. Please try attached patch,
> I only very lightly tested it here.
> 
> Also available here:
> http://people.freebsd.org/~kan/rtld-digest-notes.diff
> 
> -- 
> Alexander Kabaev

diff --git a/libexec/rtld-elf/map_object.c b/libexec/rtld-elf/map_object.c
index 509a64f..350d437 100644
--- a/libexec/rtld-elf/map_object.c
+++ b/libexec/rtld-elf/map_object.c
_at__at_ -153,7 +153,6 _at__at_ map_object(int fd, const char *path, const struct stat *sb)
 		break;
 	    note_start = (Elf_Addr)(char *)hdr + phdr->p_offset;
 	    note_end = note_start + phdr->p_filesz;
-	    digest_notes(obj, note_start, note_end);
 	    break;
 	}
 
_at__at_ -292,6 +291,11 _at__at_ map_object(int fd, const char *path, const struct stat *sb)
     obj->relro_page = obj->relocbase + trunc_page(relro_page);
     obj->relro_size = round_page(relro_size);
 
+    if (note_start < note_end)
+    {
+	digest_notes(obj, note_start, note_end);
+    }
+
     munmap(hdr, PAGE_SIZE);
     return (obj);
 
This is the right fix.

Why do you need the '{}' there ?