From patchwork Tue Dec 28 11:50:52 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Adhemerval Zanella X-Patchwork-Id: 49329 Return-Path: X-Original-To: patchwork@sourceware.org Delivered-To: patchwork@sourceware.org Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id E40CD3858000 for ; Tue, 28 Dec 2021 11:51:22 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org E40CD3858000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1640692282; bh=8YgPorUBHlYAWnuq0+14zny3z4cZn4PDJCgbgiTI06I=; h=To:Subject:Date:List-Id:List-Unsubscribe:List-Archive:List-Post: List-Help:List-Subscribe:From:Reply-To:Cc:From; b=Bnt/Di7zyXhovkamnPxw/EuJ2gCWGHWk/qcY1/bG30ZWqqtwzntz4ShgSeRyeX94p 0Wb3LFQLa6bjP1uGhW8z4bFG94SWTAr0qktKRsydr/c8fsvF/F7KgqrYUkMDfz+lfk nqVdIS/5tOGTEoHaTCvGsoZoZS04dsknA42lJd/0= X-Original-To: libc-alpha@sourceware.org Delivered-To: libc-alpha@sourceware.org Received: from mail-qk1-x731.google.com (mail-qk1-x731.google.com [IPv6:2607:f8b0:4864:20::731]) by sourceware.org (Postfix) with ESMTPS id 024023858C60 for ; Tue, 28 Dec 2021 11:51:01 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 024023858C60 Received: by mail-qk1-x731.google.com with SMTP id r139so15966749qke.9 for ; Tue, 28 Dec 2021 03:51:00 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=8YgPorUBHlYAWnuq0+14zny3z4cZn4PDJCgbgiTI06I=; b=VeTlz968ZLHM4FNeRn3Cuz0+sYrhdor6wQgJqoNqvrl5zmKsa4LXVNS362sMCxcXCQ 74sPN9cRKdjcizCgEPQrMR6armBy7KvDYkGk39MEn3Gzs3gespWUB/eWp56RyW361b6x aFnshcyDwCWbaHaPBTsKo8YW9CGU5YtWe0GXkqvDIFlvY+y72lGkqAFZ+9FKDDBvAeCx sE5+JxBwi9/3LjwNX4mG1X3qM8fwptltOpwc4SsLccVg/iGs4+edKIzfJ8QV7JWsGd5F SDhh5enz3El6b9F+GKyCGxcVqtJAho6wY4DQe9Ndlxd38Dtd6qt+r6BOPaNKCmJhQDqI H3Uw== X-Gm-Message-State: AOAM530U0LbL9Qzh732Fq5iD5Zu1uKko9wbkgHRXX2BxZmh0id4U0wVQ AsnAKAhIJN/hpOsovJmjiDtcbkLCzPpURw== X-Google-Smtp-Source: ABdhPJyu8BkCVTmtdMoo6PR4j22BqRrGsr07bG6OlZcG8VjH4oWvFsf9jhjhztJG6rYDWq0rIPEOag== X-Received: by 2002:a05:620a:3193:: with SMTP id bi19mr14796438qkb.296.1640692260317; Tue, 28 Dec 2021 03:51:00 -0800 (PST) Received: from birita.. ([2804:431:c7ca:a350:f453:adcd:c68e:4e0d]) by smtp.gmail.com with ESMTPSA id ay32sm14741885qkb.63.2021.12.28.03.50.58 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 28 Dec 2021 03:51:00 -0800 (PST) To: libc-alpha@sourceware.org, jma14 Subject: [PATCH v8 0/4] Multiple rtld-audit fixes Date: Tue, 28 Dec 2021 08:50:52 -0300 Message-Id: <20211228115056.3613468-1-adhemerval.zanella@linaro.org> X-Mailer: git-send-email 2.32.0 MIME-Version: 1.0 X-Spam-Status: No, score=-5.5 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, RCVD_IN_DNSWL_NONE, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on server2.sourceware.org X-BeenThere: libc-alpha@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Libc-alpha mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-Patchwork-Original-From: Adhemerval Zanella via Libc-alpha From: Adhemerval Zanella Reply-To: Adhemerval Zanella Cc: John Mellor-Crummey Errors-To: libc-alpha-bounces+patchwork=sourceware.org@sourceware.org Sender: "Libc-alpha" This patches fixes some of remaining issues brought by John Mellor-Crummey [1] while trying to use it along with the HPCToolkit. This should cover all the issues listed as 'Tier 1' [2] (the aarch64 SVE requires additional work, so it is postpone to 2.36) and also most of the 'Tier2' issue (BZ#28096 inclusive) which prevents the use of some glibc function that uses TLS internally on the audit module. I have checked the patches on x86_64, i686, aarch64, armv7, powerpc64, powerpc64le, and powerpc. [1] https://sourceware.org/pipermail/libc-alpha/2021-June/127636.html [2] https://docs.google.com/document/d/1dVaDBdzySecxQqD6hLLzDrEF18M1UtjDna9gL5BWWI0/edit# [3] https://sourceware.org/git/?p=glibc.git;a=shortlog;h=refs/heads/azanella/ld-audit-fixes Changes from v7: - Added la_activity tests during application exit. - Refactor _dl_allocate_tls_init to not initialize static TLS. - Changed sotruss to warn instead of error for bind-now. - Added NEWS and changed commit message for aarch64. Changes from v6: - Dropped SVE, main application on main_map l_name, and Run constructors if executable has a soname of a dependency patches. - Bumped LAV_VERSION to 2 on la_symbind bind-now support. - Added extension pointer on aarch64 fix. - Moved the refactor patch at the start of the set. - Changed _dl_audit_objsearch interface. Changes from v5: - Fixed build with --enable-profiling=yes. - Moved la_activity (LA_ACT_ADD) *after* _dl_add_to_namespace_list() for BZ#28062 fix. - Fixed powerpc64 ELFv1 OPD toc setup for bind-now. - Fixed testsuite issues for ia64. - Removed LA_SYMB_BINDNOW now that LA_SYMB_NOPLTENTER and LA_SYMB_NOPLTEXIT is passed for bind-now. Changes from v4: - Added a fix for constructors if executable has a soname of a dependency - Rebased against master. Changes from v3 - Added a aarch64 SVE RFC patch. - Fixed an issue with bind-now fix on powerpc64 ELFv1. - Rebased against master. Changes from v2 - Refactored rtld-audit code to move common come to dl-audit.c. - Issue audit la_objopen() for vDSO. - Isseu la_activity during application exit. - Issue la_symbind() for bind-now (BZ #23734). - Fix runtime linker auditing on aarch64 (BZ #26643) Changes from v1 - Fixed -fstack-protector-all tst-auditmod17. - Simplify the _dl_call_libc_early_init call the 'Fix audit regression' patch. - Remove symbind check fr BZ#15333. - Added the BZ#28096 fix. Adhemerval Zanella (3): elf: Add la_activity during application exit elf: Fix initial-exec TLS access on audit modules (BZ #28096) elf: Issue la_symbind for bind-now (BZ #23734) Ben Woodard (1): elf: Fix runtime linker auditing on aarch64 (BZ #26643) NEWS | 8 ++ bits/link_lavcurrent.h | 2 +- elf/Makefile | 105 ++++++++++++++- elf/dl-audit.c | 58 +++++---- elf/dl-fini.c | 10 ++ elf/dl-tls.c | 16 ++- elf/do-rel.h | 57 ++++++-- elf/rtld.c | 3 +- elf/sotruss-lib.c | 7 + elf/tst-audit21.c | 42 ++++++ elf/tst-audit23.c | 214 +++++++++++++++++++++++++++++++ elf/tst-audit23mod.c | 23 ++++ elf/tst-audit24a.c | 36 ++++++ elf/tst-audit24amod1.c | 31 +++++ elf/tst-audit24amod2.c | 25 ++++ elf/tst-audit24b.c | 37 ++++++ elf/tst-audit24bmod1.c | 31 +++++ elf/tst-audit24bmod2.c | 23 ++++ elf/tst-audit24c.c | 2 + elf/tst-audit24d.c | 36 ++++++ elf/tst-audit24dmod1.c | 33 +++++ elf/tst-audit24dmod2.c | 28 ++++ elf/tst-audit24dmod3.c | 31 +++++ elf/tst-audit24dmod4.c | 25 ++++ elf/tst-audit25a.c | 127 ++++++++++++++++++ elf/tst-audit25b.c | 128 ++++++++++++++++++ elf/tst-audit25mod1.c | 30 +++++ elf/tst-audit25mod2.c | 30 +++++ elf/tst-audit25mod3.c | 22 ++++ elf/tst-audit25mod4.c | 22 ++++ elf/tst-auditmod21a.c | 80 ++++++++++++ elf/tst-auditmod21b.c | 22 ++++ elf/tst-auditmod23.c | 74 +++++++++++ elf/tst-auditmod24.h | 29 +++++ elf/tst-auditmod24a.c | 114 ++++++++++++++++ elf/tst-auditmod24b.c | 104 +++++++++++++++ elf/tst-auditmod24c.c | 3 + elf/tst-auditmod24d.c | 120 +++++++++++++++++ elf/tst-auditmod25.c | 79 ++++++++++++ sysdeps/aarch64/Makefile | 20 +++ sysdeps/aarch64/bits/link.h | 26 ++-- sysdeps/aarch64/dl-audit-check.h | 28 ++++ sysdeps/aarch64/dl-link.sym | 6 +- sysdeps/aarch64/dl-trampoline.S | 97 +++++++++----- sysdeps/aarch64/tst-audit26.c | 37 ++++++ sysdeps/aarch64/tst-audit26mod.c | 33 +++++ sysdeps/aarch64/tst-audit26mod.h | 50 ++++++++ sysdeps/aarch64/tst-audit27.c | 64 +++++++++ sysdeps/aarch64/tst-audit27mod.c | 95 ++++++++++++++ sysdeps/aarch64/tst-audit27mod.h | 67 ++++++++++ sysdeps/aarch64/tst-auditmod26.c | 103 +++++++++++++++ sysdeps/aarch64/tst-auditmod27.c | 180 ++++++++++++++++++++++++++ sysdeps/generic/dl-audit-check.h | 23 ++++ sysdeps/generic/dl-lookupcfg.h | 3 + sysdeps/generic/ldsodefs.h | 5 +- sysdeps/hppa/dl-lookupcfg.h | 3 + sysdeps/ia64/dl-lookupcfg.h | 3 + sysdeps/powerpc/dl-lookupcfg.h | 39 ++++++ 58 files changed, 2658 insertions(+), 91 deletions(-) create mode 100644 elf/tst-audit21.c create mode 100644 elf/tst-audit23.c create mode 100644 elf/tst-audit23mod.c create mode 100644 elf/tst-audit24a.c create mode 100644 elf/tst-audit24amod1.c create mode 100644 elf/tst-audit24amod2.c create mode 100644 elf/tst-audit24b.c create mode 100644 elf/tst-audit24bmod1.c create mode 100644 elf/tst-audit24bmod2.c create mode 100644 elf/tst-audit24c.c create mode 100644 elf/tst-audit24d.c create mode 100644 elf/tst-audit24dmod1.c create mode 100644 elf/tst-audit24dmod2.c create mode 100644 elf/tst-audit24dmod3.c create mode 100644 elf/tst-audit24dmod4.c create mode 100644 elf/tst-audit25a.c create mode 100644 elf/tst-audit25b.c create mode 100644 elf/tst-audit25mod1.c create mode 100644 elf/tst-audit25mod2.c create mode 100644 elf/tst-audit25mod3.c create mode 100644 elf/tst-audit25mod4.c create mode 100644 elf/tst-auditmod21a.c create mode 100644 elf/tst-auditmod21b.c create mode 100644 elf/tst-auditmod23.c create mode 100644 elf/tst-auditmod24.h create mode 100644 elf/tst-auditmod24a.c create mode 100644 elf/tst-auditmod24b.c create mode 100644 elf/tst-auditmod24c.c create mode 100644 elf/tst-auditmod24d.c create mode 100644 elf/tst-auditmod25.c create mode 100644 sysdeps/aarch64/dl-audit-check.h create mode 100644 sysdeps/aarch64/tst-audit26.c create mode 100644 sysdeps/aarch64/tst-audit26mod.c create mode 100644 sysdeps/aarch64/tst-audit26mod.h create mode 100644 sysdeps/aarch64/tst-audit27.c create mode 100644 sysdeps/aarch64/tst-audit27mod.c create mode 100644 sysdeps/aarch64/tst-audit27mod.h create mode 100644 sysdeps/aarch64/tst-auditmod26.c create mode 100644 sysdeps/aarch64/tst-auditmod27.c create mode 100644 sysdeps/generic/dl-audit-check.h create mode 100644 sysdeps/powerpc/dl-lookupcfg.h