From patchwork Thu Jul 15 09:10:17 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Florian Weimer X-Patchwork-Id: 44361 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 D44063AAA0E6 for ; Thu, 15 Jul 2021 09:22:52 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org D44063AAA0E6 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1626340972; bh=QWJie6ly8Y5VO74DD9qZysU9l6M75rSZta81dVTgW4Y=; h=To:Subject:In-Reply-To:References:Date:List-Id:List-Unsubscribe: List-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To: From; b=N8fvY5pyKTkjFFjXfgWuuP2z1AWWYyvY4ozz+7+iZbABzECq5myjnWr32s+gFL86/ 4+V6zD2oN5Qz58xnZ8Adoq+iA9MlCif5vze6gKGPSjDUujzdfZySUQxV6iqndMzSKx ugEjBDbQO0uYYoBfKOIO8xUmzJ5sGAFTp8zZF+/k= X-Original-To: libc-alpha@sourceware.org Delivered-To: libc-alpha@sourceware.org Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by sourceware.org (Postfix) with ESMTP id 237E33AAA08B for ; Thu, 15 Jul 2021 09:10:23 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 237E33AAA08B Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-352-T2spEPEjMdqDLndiQFkh7Q-1; Thu, 15 Jul 2021 05:10:21 -0400 X-MC-Unique: T2spEPEjMdqDLndiQFkh7Q-1 Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id ACB4F1084F4B for ; Thu, 15 Jul 2021 09:10:20 +0000 (UTC) Received: from oldenburg.str.redhat.com (ovpn-112-73.phx2.redhat.com [10.3.112.73]) by smtp.corp.redhat.com (Postfix) with ESMTPS id D70B860871 for ; Thu, 15 Jul 2021 09:10:19 +0000 (UTC) To: libc-alpha@sourceware.org Subject: [PATCH 15/24] resolv: Move ns_samename into its own file, and into libc In-Reply-To: References: X-From-Line: 7c77769931e9fc7e987aad32002ee8fbe00e59e4 Mon Sep 17 00:00:00 2001 Message-Id: <7c77769931e9fc7e987aad32002ee8fbe00e59e4.1626339931.git.fweimer@redhat.com> Date: Thu, 15 Jul 2021 11:10:17 +0200 User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux) MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.13 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com X-Spam-Status: No, score=-12.9 required=5.0 tests=BAYES_00, DKIMWL_WL_HIGH, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, GIT_PATCH_0, RCVD_IN_DNSWL_LOW, RCVD_IN_MSPIKE_H4, RCVD_IN_MSPIKE_WL, SPF_HELO_NONE, SPF_NONE, 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: Florian Weimer via Libc-alpha From: Florian Weimer Reply-To: Florian Weimer Errors-To: libc-alpha-bounces+patchwork=sourceware.org@sourceware.org Sender: "Libc-alpha" But only as an internal symbol, __libc_ns_samename. The libresolv ABI is preserved. This is because the function is deprecated, and it does not make sense to add new symbol versions for deprecated functions. Also reformat the implementation to GNU style. Reviewed-by: Carlos O'Donell Tested-by: Carlos O'Donell --- include/arpa/nameser.h | 3 ++- resolv/Makefile | 1 + resolv/Versions | 1 + resolv/ns_print.c | 6 +++--- resolv/ns_samedomain.c | 28 +++------------------------- resolv/ns_samename.c | 38 ++++++++++++++++++++++++++++++++++++++ resolv/res_query.c | 2 +- resolv/res_send.c | 4 ++-- resolv/resolv-deprecated.c | 6 ++++++ 9 files changed, 57 insertions(+), 32 deletions(-) create mode 100644 resolv/ns_samename.c diff --git a/include/arpa/nameser.h b/include/arpa/nameser.h index a529cc86d3..53f1dbc7c3 100644 --- a/include/arpa/nameser.h +++ b/include/arpa/nameser.h @@ -68,7 +68,6 @@ libresolv_hidden_proto (ns_parserr) libresolv_hidden_proto (ns_sprintrr) libresolv_hidden_proto (ns_sprintrrf) libresolv_hidden_proto (ns_samedomain) -libresolv_hidden_proto (ns_samename) libresolv_hidden_proto (ns_format_ttl) extern __typeof (ns_makecanon) __libc_ns_makecanon; @@ -87,6 +86,8 @@ extern __typeof (ns_name_uncompress) __ns_name_uncompress; libc_hidden_proto (__ns_name_uncompress) extern __typeof (ns_name_unpack) __ns_name_unpack; libc_hidden_proto (__ns_name_unpack) +extern __typeof (ns_samename) __libc_ns_samename; +libc_hidden_proto (__libc_ns_samename) # endif /* !_ISOMAC */ #endif diff --git a/resolv/Makefile b/resolv/Makefile index d1ad5e4cba..744da951ae 100644 --- a/resolv/Makefile +++ b/resolv/Makefile @@ -43,6 +43,7 @@ routines := \ ns_name_skip \ ns_name_uncompress \ ns_name_unpack \ + ns_samename \ nsap_addr \ res-close \ res-name-checking \ diff --git a/resolv/Versions b/resolv/Versions index 93078de25c..2201d4224c 100644 --- a/resolv/Versions +++ b/resolv/Versions @@ -73,6 +73,7 @@ libc { __libc_dn_expand; __libc_dn_skipname; __libc_ns_makecanon; + __libc_ns_samename; __libc_res_dnok; __libc_res_hnok; __ns_name_compress; diff --git a/resolv/ns_print.c b/resolv/ns_print.c index 9a9602d74c..43f39edf61 100644 --- a/resolv/ns_print.c +++ b/resolv/ns_print.c @@ -103,9 +103,9 @@ ns_sprintrrf(const u_char *msg, size_t msglen, /* * Owner. */ - if (name_ctx != NULL && ns_samename(name_ctx, name) == 1) { + if (name_ctx != NULL && __libc_ns_samename (name_ctx, name) == 1) T(addstr("\t\t\t", 3, &buf, &buflen)); - } else { + else { len = prune_origin(name, origin); if (*name == '\0') { goto root; @@ -621,7 +621,7 @@ prune_origin(const char *name, const char *origin) { const char *oname = name; while (*name != '\0') { - if (origin != NULL && ns_samename(name, origin) == 1) + if (origin != NULL && __libc_ns_samename (name, origin) == 1) return (name - oname - (name > oname)); while (*name != '\0') { if (*name == '\\') { diff --git a/resolv/ns_samedomain.c b/resolv/ns_samedomain.c index cfff2516b0..46cb8f5fb2 100644 --- a/resolv/ns_samedomain.c +++ b/resolv/ns_samedomain.c @@ -139,31 +139,9 @@ libresolv_hidden_def (ns_samedomain) * is "a" a subdomain of "b"? */ int -ns_subdomain(const char *a, const char *b) { - return (ns_samename(a, b) != 1 && ns_samedomain(a, b)); +ns_subdomain (const char *a, const char *b) +{ + return __libc_ns_samename (a, b) != 1 && ns_samedomain (a, b); } -/*% - * determine whether domain name "a" is the same as domain name "b" - * - * return: - *\li -1 on error - *\li 0 if names differ - *\li 1 if names are the same - */ - -int -ns_samename(const char *a, const char *b) { - char ta[NS_MAXDNAME], tb[NS_MAXDNAME]; - - if (__libc_ns_makecanon(a, ta, sizeof ta) < 0 || - __libc_ns_makecanon(b, tb, sizeof tb) < 0) - return (-1); - if (strcasecmp(ta, tb) == 0) - return (1); - else - return (0); -} -libresolv_hidden_def (ns_samename) - /*! \file */ diff --git a/resolv/ns_samename.c b/resolv/ns_samename.c new file mode 100644 index 0000000000..61fe3cab43 --- /dev/null +++ b/resolv/ns_samename.c @@ -0,0 +1,38 @@ +/* Check if two domain names are equal after trailing dot normalization. + * Copyright (c) 2004 by Internet Systems Consortium, Inc. ("ISC") + * Copyright (c) 1995,1999 by Internet Software Consortium. + * + * Permission to use, copy, modify, and distribute this software for any + * purpose with or without fee is hereby granted, provided that the above + * copyright notice and this permission notice appear in all copies. + * + * THE SOFTWARE IS PROVIDED "AS IS" AND INTERNET SOFTWARE CONSORTIUM DISCLAIMS + * ALL WARRANTIES WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES + * OF MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL INTERNET SOFTWARE + * CONSORTIUM BE LIABLE FOR ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL + * DAMAGES OR ANY DAMAGES WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR + * PROFITS, WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS + * ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS + * SOFTWARE. + */ + +#include +#include + +/* Determines whether domain name A is the same as domain name B. + Returns -1 on error, 0 if names differ, 1 if names are the + same. */ +int +__libc_ns_samename (const char *a, const char *b) +{ + char ta[NS_MAXDNAME], tb[NS_MAXDNAME]; + + if (__libc_ns_makecanon (a, ta, sizeof ta) < 0 || + __libc_ns_makecanon (b, tb, sizeof tb) < 0) + return -1; + if (__strcasecmp (ta, tb) == 0) + return 1; + else + return 0; +} +libc_hidden_def (__libc_ns_samename) diff --git a/resolv/res_query.c b/resolv/res_query.c index ebbe5a6a4e..5b51af8451 100644 --- a/resolv/res_query.c +++ b/resolv/res_query.c @@ -656,7 +656,7 @@ __res_context_hostalias (struct resolv_context *ctx, if (!*cp1) break; *cp1 = '\0'; - if (ns_samename(buf, name) == 1) { + if (__libc_ns_samename(buf, name) == 1) { while (isspace(*++cp1)) ; if (!*cp1) diff --git a/resolv/res_send.c b/resolv/res_send.c index 915fc6d2c6..dfea9fc7fa 100644 --- a/resolv/res_send.c +++ b/resolv/res_send.c @@ -224,8 +224,8 @@ res_nameinquery(const char *name, int type, int class, return (-1); NS_GET16(ttype, cp); NS_GET16(tclass, cp); - if (ttype == type && tclass == class && - ns_samename(tname, name) == 1) + if (ttype == type && tclass == class + && __libc_ns_samename (tname, name) == 1) return (1); } return (0); diff --git a/resolv/resolv-deprecated.c b/resolv/resolv-deprecated.c index cbd1078dc2..d8344f8f39 100644 --- a/resolv/resolv-deprecated.c +++ b/resolv/resolv-deprecated.c @@ -33,3 +33,9 @@ ns_makecanon (const char *src, char *dst, size_t dstsize) { return __libc_ns_makecanon (src, dst, dstsize); } + +int +ns_samename (const char *a, const char *b) +{ + return __libc_ns_samename (a, b); +}