From patchwork Fri Aug 18 15:57:16 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "H.J. Lu" X-Patchwork-Id: 22219 Received: (qmail 120348 invoked by alias); 18 Aug 2017 15:57:21 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-alpha-owner@sourceware.org Delivered-To: mailing list libc-alpha@sourceware.org Received: (qmail 120328 invoked by uid 89); 18 Aug 2017 15:57:20 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-25.0 required=5.0 tests=AWL, BAYES_00, FREEMAIL_FROM, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, RCVD_IN_DNSWL_NONE, RCVD_IN_SORBS_SPAM, SPF_PASS, URIBL_RED autolearn=ham version=3.3.2 spammy=BEFORE X-HELO: mail-oi0-f66.google.com X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=YvI2W98oz1vPR9f5EIsdbRox3BiV/2fabitfbpOAC74=; b=f9+W5JNzylXcaF7FLNGMUN6Bo4w019LF7Cv0cAghlgrrGjkaita7vBHW2owGT+xHA4 YFcINE/ZJq6SkT9bWaIFAtbDsyaDmNGNUPJixTxOlGL4mET0KJvGBzSeDbrhM57ytL0D 18aIi+eyHX/98dQjSyUJ8dlxFW1Cjqj02cFH5QLsHM7J7aANdfH5EwgZDroDIMOOrccJ EVImPRetUAQxB4XtqwWpTaLsICDyXdwZzq0UHzauPHUYD82IGVajdQLpdOWx1gtDnzHe o+404tQlEe3xuqZ/eyccl1wNPnGG/mBRyQICmKJYZBgAXBVtiSSh1jWQY8kbG3lZWmAy 1o6w== X-Gm-Message-State: AHYfb5j8HZtkUtkf03ASW1cAun4lFJh7sJk0frK4XS7NVy6VncQ6KOzF /srxAx0JT67E3yOQvjDsewvnB9LSBfmP X-Received: by 10.202.81.68 with SMTP id f65mr12438477oib.2.1503071837299; Fri, 18 Aug 2017 08:57:17 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: References: <20170818130141.GA4463@gmail.com> From: "H.J. Lu" Date: Fri, 18 Aug 2017 08:57:16 -0700 Message-ID: Subject: Re: [PATCH 1/2] Mark internal argz functions with attribute_hidden [BZ #18822] To: Joseph Myers Cc: GNU C Library On Fri, Aug 18, 2017 at 8:50 AM, Joseph Myers wrote: > On Fri, 18 Aug 2017, H.J. Lu wrote: > >> > In this case, the problem presumably is that the internal declarations are >> > in the installed argz.h header. So move them from the installed header to >> > the include/ one. At that point, using typeof *does* make sense to copy >> > the type from the public function, e.g. >> > >> > extern __typeof (argz_add) __argz_add __THROW attribute_hidden; >> > >> > (argz.h is shared with gnulib, but in gnulib it doesn't have those >> > internal declarations anyway. So moving them to the include/ header helps >> > bring the public one closer to the gnulib version.) >> > >> >> Done. >> >> Here is the updated patch. OK for master? > > __argz_replace was not previously __THROW, and neither is argz_replace. > If you have done an analysis that establishes that use of __THROW is safe > there, it should be added to argz_replace rather than just to the internal > declaration. Here is the updated patch with __THROW removed from __argz_replace. OK for master? Thanks. From 8093c83073f568ee0a23d13767d8d567bfc0e98a Mon Sep 17 00:00:00 2001 From: "H.J. Lu" Date: Fri, 18 Aug 2017 05:19:36 -0700 Subject: [PATCH] Mark internal argz functions with attribute_hidden [BZ #18822] Move internal argz function prototypes to include/argz.h and mark them with attribute_hidden to allow direct access within libc.so and libc.a without using GOT nor PLT. This also brings string/argz.h closer to the gnulib version. [BZ #18822] * include/argz.h (__argz_create_sep): New function prototype. (__argz_append): Likewise. (__argz_add): Likewise. (__argz_add_sep): Likewise. (__argz_delete): Likewise. (__argz_insert): Likewise. (__argz_replace): Likewise. * string/argz.h (__argz_create_sep): Removed. (__argz_append): Likewise. (__argz_add): Likewise. (__argz_add_sep): Likewise. (__argz_delete): Likewise. (__argz_insert): Likewise. (__argz_replace): Likewise. --- include/argz.h | 32 ++++++++++++++++++++++++++++++++ string/argz.h | 26 -------------------------- 2 files changed, 32 insertions(+), 26 deletions(-) diff --git a/include/argz.h b/include/argz.h index 0388c23277..770f02eefb 100644 --- a/include/argz.h +++ b/include/argz.h @@ -4,6 +4,38 @@ # ifndef _ISOMAC +extern error_t __argz_create_sep (const char *__restrict __string, + int __sep, char **__restrict __argz, + size_t *__restrict __len) + __THROW attribute_hidden; +extern error_t __argz_append (char **__restrict __argz, + size_t *__restrict __argz_len, + const char *__restrict __buf, size_t __buf_len) + __THROW attribute_hidden; +extern error_t __argz_add (char **__restrict __argz, + size_t *__restrict __argz_len, + const char *__restrict __str) + __THROW attribute_hidden; +extern error_t __argz_add_sep (char **__restrict __argz, + size_t *__restrict __argz_len, + const char *__restrict __string, int __delim) + __THROW attribute_hidden; +extern void __argz_delete (char **__restrict __argz, + size_t *__restrict __argz_len, + char *__restrict __entry) + __THROW attribute_hidden; +extern error_t __argz_insert (char **__restrict __argz, + size_t *__restrict __argz_len, + char *__restrict __before, + const char *__restrict __entry) + __THROW attribute_hidden; +extern error_t __argz_replace (char **__restrict __argz, + size_t *__restrict __argz_len, + const char *__restrict __str, + const char *__restrict __with, + unsigned int *__restrict __replace_count) + attribute_hidden; + libc_hidden_proto (argz_delete) libc_hidden_proto (__argz_count) libc_hidden_proto (__argz_stringify) diff --git a/string/argz.h b/string/argz.h index e07d74208a..269016d7ff 100644 --- a/string/argz.h +++ b/string/argz.h @@ -44,9 +44,6 @@ extern error_t argz_create (char *const __argv[], char **__restrict __argz, STRING, returning it in ARGZ, and the total length in LEN. If a memory allocation error occurs, ENOMEM is returned, otherwise 0. The result can be destroyed using free. */ -extern error_t __argz_create_sep (const char *__restrict __string, - int __sep, char **__restrict __argz, - size_t *__restrict __len) __THROW; extern error_t argz_create_sep (const char *__restrict __string, int __sep, char **__restrict __argz, size_t *__restrict __len) __THROW; @@ -70,38 +67,24 @@ extern void __argz_stringify (char *__argz, size_t __len, int __sep) __THROW; extern void argz_stringify (char *__argz, size_t __len, int __sep) __THROW; /* Append BUF, of length BUF_LEN to the argz vector in ARGZ & ARGZ_LEN. */ -extern error_t __argz_append (char **__restrict __argz, - size_t *__restrict __argz_len, - const char *__restrict __buf, size_t __buf_len) - __THROW; extern error_t argz_append (char **__restrict __argz, size_t *__restrict __argz_len, const char *__restrict __buf, size_t __buf_len) __THROW; /* Append STR to the argz vector in ARGZ & ARGZ_LEN. */ -extern error_t __argz_add (char **__restrict __argz, - size_t *__restrict __argz_len, - const char *__restrict __str) __THROW; extern error_t argz_add (char **__restrict __argz, size_t *__restrict __argz_len, const char *__restrict __str) __THROW; /* Append SEP separated list in STRING to the argz vector in ARGZ & ARGZ_LEN. */ -extern error_t __argz_add_sep (char **__restrict __argz, - size_t *__restrict __argz_len, - const char *__restrict __string, int __delim) - __THROW; extern error_t argz_add_sep (char **__restrict __argz, size_t *__restrict __argz_len, const char *__restrict __string, int __delim) __THROW; /* Delete ENTRY from ARGZ & ARGZ_LEN, if it appears there. */ -extern void __argz_delete (char **__restrict __argz, - size_t *__restrict __argz_len, - char *__restrict __entry) __THROW; extern void argz_delete (char **__restrict __argz, size_t *__restrict __argz_len, char *__restrict __entry) __THROW; @@ -112,10 +95,6 @@ extern void argz_delete (char **__restrict __argz, ARGZ, ENTRY) will insert ENTRY at the beginning of ARGZ. If BEFORE is not in ARGZ, EINVAL is returned, else if memory can't be allocated for the new ARGZ, ENOMEM is returned, else 0. */ -extern error_t __argz_insert (char **__restrict __argz, - size_t *__restrict __argz_len, - char *__restrict __before, - const char *__restrict __entry) __THROW; extern error_t argz_insert (char **__restrict __argz, size_t *__restrict __argz_len, char *__restrict __before, @@ -124,11 +103,6 @@ extern error_t argz_insert (char **__restrict __argz, /* Replace any occurrences of the string STR in ARGZ with WITH, reallocating ARGZ as necessary. If REPLACE_COUNT is non-zero, *REPLACE_COUNT will be incremented by number of replacements performed. */ -extern error_t __argz_replace (char **__restrict __argz, - size_t *__restrict __argz_len, - const char *__restrict __str, - const char *__restrict __with, - unsigned int *__restrict __replace_count); extern error_t argz_replace (char **__restrict __argz, size_t *__restrict __argz_len, const char *__restrict __str, -- 2.13.5