From patchwork Wed Nov 22 21:50:28 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Joseph Myers X-Patchwork-Id: 24453 Received: (qmail 66165 invoked by alias); 22 Nov 2017 21:50:37 -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 66156 invoked by uid 89); 22 Nov 2017 21:50:37 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-24.3 required=5.0 tests=AWL, BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, KB_WAM_FROM_NAME_SINGLEWORD, RCVD_IN_DNSWL_NONE, SPF_PASS, URIBL_RED autolearn=ham version=3.3.2 spammy= X-HELO: relay1.mentorg.com Date: Wed, 22 Nov 2017 21:50:28 +0000 From: Joseph Myers To: Subject: Obsolete p_secstodate Message-ID: User-Agent: Alpine 2.20 (DEB 67 2015-01-07) MIME-Version: 1.0 X-ClientProxiedBy: svr-ies-mbx-01.mgc.mentorg.com (139.181.222.1) To SVR-IES-MBX-03.mgc.mentorg.com (139.181.222.3) This patch, relative to a tree with (pending review) applied, obsoletes p_secstodate, making the underlying function __p_secstodate into a compat symbol not available for new binaries or ports. The calls in ns_print.c (part of incomplete handling of TKEY) are changed to use %lu to print times instead of trying to pretty-print the times any more. Tested for x86_64. 2017-11-22 Joseph Myers * resolv/res_debug.c (p_secstodate): Condition definition on [SHLIB_COMPAT (libresolv, GLIBC_2_0, GLIBC_2_27)]. Define directly as __p_secstodate, and as a compat symbol. Do not use libresolv_hidden_def. * resolv/resolv.h (p_secstodate): Remove macro and function declaration. * resolv/ns_print.c (ns_sprintrrf): Print times with %lu, not using p_secstodate. * include/resolv.h (__p_secstodate): Do not use libresolv_hidden_proto. * resolv/Makefile (tests): Move tst-p_secstodate to .... (tests-internal): ... here. * resolv/tst-p_secstodate.c: Include . Condition all contents on [TEST_COMPAT (libresolv, GLIBC_2_0, GLIBC_2_27)] and declare and use __p_secstodate and use compat_symbol_reference in that case. [!TEST_COMPAT (libresolv, GLIBC_2_0, GLIBC_2_27)] (do_test): Add implementation returning 77. diff --git a/NEWS b/NEWS index 1c5da21..30ff04e 100644 --- a/NEWS +++ b/NEWS @@ -66,6 +66,9 @@ Deprecated and removed features, and other changes affecting compatibility: * In the malloc_info output, the element may contain another element, "subheaps", which contains the number of sub-heaps. +* The libresolv function p_secstodate is no longer supported for new + programs. + Changes to build and runtime requirements: [Add changes to build and runtime requirements here] diff --git a/include/resolv.h b/include/resolv.h index 634f552..daf4a74 100644 --- a/include/resolv.h +++ b/include/resolv.h @@ -65,7 +65,6 @@ libresolv_hidden_proto (__res_nameinquery) libresolv_hidden_proto (__res_queriesmatch) libresolv_hidden_proto (__b64_ntop) libresolv_hidden_proto (__dn_count_labels) -libresolv_hidden_proto (__p_secstodate) # endif /* _RESOLV_H_ && !_ISOMAC */ #endif diff --git a/resolv/Makefile b/resolv/Makefile index aff6710..a98a84f 100644 --- a/resolv/Makefile +++ b/resolv/Makefile @@ -55,7 +55,6 @@ tests += \ tst-resolv-network \ tst-resolv-res_init-multi \ tst-resolv-search \ - tst-p_secstodate \ # These tests need libdl. ifeq (yes,$(build-shared)) @@ -77,6 +76,9 @@ endif # This test accesses __inet_ntop_length, an internal libc function. tests-internal += tst-inet_pton +# This test accesses the __p_secstodate compat symbol. +tests-internal += tst-p_secstodate + # This test sends millions of packets and is rather slow. xtests += tst-resolv-qtypes diff --git a/resolv/ns_print.c b/resolv/ns_print.c index f55680c..d61f504 100644 --- a/resolv/ns_print.c +++ b/resolv/ns_print.c @@ -488,12 +488,12 @@ ns_sprintrrf(const u_char *msg, size_t msglen, /* Inception. */ t = ns_get32(rdata); rdata += NS_INT32SZ; - len = SPRINTF((tmp, "%s ", p_secstodate(t))); + len = SPRINTF((tmp, "%lu ", t)); T(addstr(tmp, len, &buf, &buflen)); /* Experation. */ t = ns_get32(rdata); rdata += NS_INT32SZ; - len = SPRINTF((tmp, "%s ", p_secstodate(t))); + len = SPRINTF((tmp, "%lu ", t)); T(addstr(tmp, len, &buf, &buflen)); /* Mode , Error, Key Size. */ diff --git a/resolv/res_debug.c b/resolv/res_debug.c index e319774..4446030 100644 --- a/resolv/res_debug.c +++ b/resolv/res_debug.c @@ -1052,6 +1052,7 @@ dn_count_labels(const char *name) { libresolv_hidden_def (__dn_count_labels) +#if SHLIB_COMPAT (libresolv, GLIBC_2_0, GLIBC_2_27) /* * Make dates expressed in seconds-since-Jan-1-1970 easy to read. * SIG records are required to be printed like this, by the Secure DNS RFC. @@ -1059,7 +1060,7 @@ libresolv_hidden_def (__dn_count_labels) * signed 32-bit range. */ char * -p_secstodate (u_long secs) { +__p_secstodate (u_long secs) { /* XXX nonreentrant */ static char output[15]; /* YYYYMMDDHHMMSS and null */ time_t clock = secs; @@ -1080,13 +1081,14 @@ p_secstodate (u_long secs) { that, even given range checks on all fields with __builtin_unreachable called for out-of-range values. */ DIAG_PUSH_NEEDS_COMMENT; -#if __GNUC_PREREQ (7, 0) +# if __GNUC_PREREQ (7, 0) DIAG_IGNORE_NEEDS_COMMENT (8, "-Wformat-overflow="); -#endif +# endif sprintf(output, "%04d%02d%02d%02d%02d%02d", time->tm_year, time->tm_mon, time->tm_mday, time->tm_hour, time->tm_min, time->tm_sec); DIAG_POP_NEEDS_COMMENT; return (output); } -libresolv_hidden_def (__p_secstodate) +compat_symbol (libresolv, __p_secstodate, __p_secstodate, GLIBC_2_0); +#endif diff --git a/resolv/resolv.h b/resolv/resolv.h index e8c581c..80a523e 100644 --- a/resolv/resolv.h +++ b/resolv/resolv.h @@ -213,7 +213,6 @@ __END_DECLS #define p_fqname __p_fqname #define p_fqnname __p_fqnname #define p_option __p_option -#define p_secstodate __p_secstodate #define p_time __p_time #define p_type __p_type #define p_rcode __p_rcode @@ -268,7 +267,6 @@ const unsigned char * p_fqnname (const unsigned char *__cp, const unsigned char * p_fqname (const unsigned char *, const unsigned char *, FILE *) __THROW; const char * p_option (unsigned long __option) __THROW; -char * p_secstodate (unsigned long) __THROW; int dn_count_labels (const char *) __THROW; int dn_comp (const char *, unsigned char *, int, unsigned char **, unsigned char **) __THROW; diff --git a/resolv/tst-p_secstodate.c b/resolv/tst-p_secstodate.c index 9dac1ad..c310dbb 100644 --- a/resolv/tst-p_secstodate.c +++ b/resolv/tst-p_secstodate.c @@ -1,4 +1,4 @@ -/* Test p_secstodate. +/* Test __p_secstodate compat symbol. Copyright (C) 2017 Free Software Foundation, Inc. This file is part of the GNU C Library. @@ -23,9 +23,16 @@ #include #include +#include + +#if TEST_COMPAT (libresolv, GLIBC_2_0, GLIBC_2_27) + +char *__p_secstodate (unsigned long int); +compat_symbol_reference (libresolv, __p_secstodate, __p_secstodate, GLIBC_2_0); + struct test { - /* Argument to p_secstodate. */ + /* Argument to __p_secstodate. */ unsigned long int in; /* Expected output. */ const char *out; @@ -53,7 +60,7 @@ do_test (void) int ret = 0; for (size_t i = 0; i < array_length (tests); i++) { - char *p = p_secstodate (tests[i].in); + char *p = __p_secstodate (tests[i].in); printf ("Test %zu: %lu -> %s\n", i, tests[i].in, p); if (strcmp (p, tests[i].out) != 0) { @@ -64,4 +71,14 @@ do_test (void) return ret; } +#else + +static int +do_test (void) +{ + return 77; +} + +#endif + #include