From 9064157fc21cc8d83dcc9ea5087fa9a5d8a201db Mon Sep 17 00:00:00 2001 From: nwhitehorn Date: Sat, 17 Dec 2011 16:20:27 +0000 Subject: [PATCH 114/175] Additional icache paranoia: non-PLT relocations can modify the text segment. It is then important to make sure the icache is synchronized again to prevent (rare) random seg faults and illegal instructions. MFC after: 3 days git-svn-id: http://svn.freebsd.org/base/head@228646 ccf9f872-aa2e-dd11-9fc8-001c23d0bc1f (cherry picked from commit 0aa7470d844d047a47dfb7bfe1cc1c697786ffbe) Signed-off-by: Xin Li --- libexec/rtld-elf/powerpc/reloc.c | 4 ++++ libexec/rtld-elf/powerpc64/reloc.c | 7 +++++-- 2 files changed, 9 insertions(+), 2 deletions(-) diff --git a/libexec/rtld-elf/powerpc/reloc.c b/libexec/rtld-elf/powerpc/reloc.c index c8967b6..d0637f4 100644 --- a/libexec/rtld-elf/powerpc/reloc.c +++ b/libexec/rtld-elf/powerpc/reloc.c @@ -317,6 +317,10 @@ reloc_non_plt(Obj_Entry *obj, Obj_Entry *obj_rtld, RtldLockState *lockstate) done: if (cache != NULL) free(cache); + + /* Synchronize icache for text seg in case we made any changes */ + __syncicache(obj->mapbase, obj->textsize); + return (r); } diff --git a/libexec/rtld-elf/powerpc64/reloc.c b/libexec/rtld-elf/powerpc64/reloc.c index 7476a1e..47368a8 100644 --- a/libexec/rtld-elf/powerpc64/reloc.c +++ b/libexec/rtld-elf/powerpc64/reloc.c @@ -313,9 +313,12 @@ reloc_non_plt(Obj_Entry *obj, Obj_Entry *obj_rtld, RtldLockState *lockstate) } r = 0; done: - if (cache) { + if (cache) munmap(cache, bytes); - } + + /* Synchronize icache for text seg in case we made any changes */ + __syncicache(obj->mapbase, obj->textsize); + return (r); } -- 1.7.9.4