From nobody Fri Feb 04 14:58:23 2022 X-Original-To: dev-commits-src-branches@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id B8840196EE43; Fri, 4 Feb 2022 14:58:23 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4JqzDv4dnhz4j8Z; Fri, 4 Feb 2022 14:58:23 +0000 (UTC) (envelope-from git@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1643986703; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=RziNZ7aUSZtGjJvdDwvWneD74R4zFpXAlJj+621fKWo=; b=MmtK6gkSzV86QcKSDNksMHYxs4hZEB7gSUiyHgFiVIDnG+QbTJAKY2sRZ3ZZzCpoXjmU3V O+BRPOQgdEdHnlzH/WEPbUR7Apj3+xtBj9y9WNStm6VO4gdi29ih72zEpTi5W8Isgb4404 I+Amo5+6LXncu/BHlEBQ3kNxIC32z332/Rn+syO/Ql8xyk1rSoGteATXbSkCEoxRDSqFQW PQcJZ5jpw3UEYks6WlfdXYDn54HheVXHfcftaI2ex2o31QBhQfKN4hS4eiMcsakb6yExC9 A1g+k1DQLB5V+AKAsrfWu5+ePcNlJpNkzkL5eYYoW/svKgB/Yz8Pz/70bv4C0w== Received: from gitrepo.freebsd.org (gitrepo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:5]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) by mxrelay.nyi.freebsd.org (Postfix) with ESMTPS id 803A1583D; Fri, 4 Feb 2022 14:58:23 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from gitrepo.freebsd.org ([127.0.1.44]) by gitrepo.freebsd.org (8.16.1/8.16.1) with ESMTP id 214EwNYb091258; Fri, 4 Feb 2022 14:58:23 GMT (envelope-from git@gitrepo.freebsd.org) Received: (from git@localhost) by gitrepo.freebsd.org (8.16.1/8.16.1/Submit) id 214EwNM6091257; Fri, 4 Feb 2022 14:58:23 GMT (envelope-from git) Date: Fri, 4 Feb 2022 14:58:23 GMT Message-Id: <202202041458.214EwNM6091257@gitrepo.freebsd.org> To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org From: Mark Johnston Subject: git: 1a233fd31769 - stable/13 - sort: Fix message catalogue usage List-Id: Commits to the stable branches of the FreeBSD src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-branches List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-src-branches@freebsd.org X-BeenThere: dev-commits-src-branches@freebsd.org MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Git-Committer: markj X-Git-Repository: src X-Git-Refname: refs/heads/stable/13 X-Git-Reftype: branch X-Git-Commit: 1a233fd317690e5eaed1476756ac3e991ca03ff2 Auto-Submitted: auto-generated ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1643986703; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=RziNZ7aUSZtGjJvdDwvWneD74R4zFpXAlJj+621fKWo=; b=T/+DhfThgz11dLEIAim86XIAKw9ef3hGxC/GdZgJjhpvO2OA4Zp7im52NWDjnUsrZ5SjzP YcwW2O5RbMuJGdzQ3F09fMz3frLQHrAVgszjssLxK8LEXowW5cz7G+p9anaXLJYFhSP+ls q0jLV1ObiPuiiLv++hTZQpxAT+pRzDmBvQYTrFzrq51sMdmDT1exFsV2gPecyy3P9lLbv3 CU6o8/843N6Z5VW2Y33HYak8hwgkvTrLcgNMGnsBSGd0rnMcyCl/Ko7fo/HYOlYWyMh7Ct tAL0REHCUO1Zuf0nmoWdQOLefHCHSKhs3Oh9YJuFqUnTB7ibxYPmX/JAsa/l9g== ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1643986703; a=rsa-sha256; cv=none; b=xKYGgHiw0BOD5kZj2e/gs7nMU4Am61mwflKLEEldJeVEYpQEfst2bFK2vptN5sB/CKYXce vA0vMXIQ1CZgrNGp8mvgSPX04GlxLz75tjKE0A09vyAm6V1rhSb7WK02Y5ViuHLlUaTC9b oIAF9cnfcHicIxSh7lKdJKVGsAxf8Iz/Qm4Twmtg1v122rtD8jkUI4mz4a1auUckjMhw0T zqN5UWfmv8HGDBF1v4lexBPffFa21ZnC6sA2+PfsjF0bZur8fC7peewdNItGQEy7wp6nzJ 7Fap0ftOUR5TRdBRWfCHnu+miuKmnxvAbymFZKGo4aHHl2ZNPBTeUm/52VpK/Q== ARC-Authentication-Results: i=1; mx1.freebsd.org; none X-ThisMailContainsUnwantedMimeParts: N The branch stable/13 has been updated by markj: URL: https://cgit.FreeBSD.org/src/commit/?id=1a233fd317690e5eaed1476756ac3e991ca03ff2 commit 1a233fd317690e5eaed1476756ac3e991ca03ff2 Author: Mark Johnston AuthorDate: 2022-01-28 17:49:28 +0000 Commit: Mark Johnston CommitDate: 2022-02-04 14:58:13 +0000 sort: Fix message catalogue usage - Check that catopen() succeeded before calling catclose(). musl will crash in the latter if the catalogue descriptor is -1. - Keep the message catalogue open for most of sort(1)'s actual operation. - Don't use catgets(3) to print error messages if catopen(3) had failed. Reviewed by: arichardson, emaste Sponsored by: The FreeBSD Foundation (cherry picked from commit 8d8b9b560afdf88f5537f259a4a14660d2a85b1f) --- usr.bin/sort/sort.c | 19 ++++++++++--------- usr.bin/sort/sort.h | 3 ++- 2 files changed, 12 insertions(+), 10 deletions(-) diff --git a/usr.bin/sort/sort.c b/usr.bin/sort/sort.c index 0fbbd9284aad..b7c8107cc22d 100644 --- a/usr.bin/sort/sort.c +++ b/usr.bin/sort/sort.c @@ -57,7 +57,7 @@ __FBSDID("$FreeBSD$"); #ifndef WITHOUT_NLS #include -nl_catd catalog; +nl_catd catalog = (nl_catd)-1; #endif #define OPTIONS "bcCdfghik:Mmno:RrsS:t:T:uVz" @@ -1011,6 +1011,10 @@ main(int argc, char **argv) set_tmpdir(); set_sort_opts(); +#ifndef WITHOUT_NLS + catalog = catopen("sort", NL_CAT_LOCALE); +#endif + fix_obsolete_keys(&argc, argv); while (((c = getopt_long(argc, argv, OPTIONS, long_options, NULL)) @@ -1194,17 +1198,9 @@ main(int argc, char **argv) argv = argv_from_file0; } -#ifndef WITHOUT_NLS - catalog = catopen("sort", NL_CAT_LOCALE); -#endif - if (sort_opts_vals.cflag && sort_opts_vals.mflag) errx(1, "%c:%c: %s", 'm', 'c', getstr(1)); -#ifndef WITHOUT_NLS - catclose(catalog); -#endif - if (keys_num == 0) { keys_num = 1; keys = sort_realloc(keys, sizeof(struct key_specs)); @@ -1343,5 +1339,10 @@ main(int argc, char **argv) sort_free(outfile); +#ifndef WITHOUT_NLS + if (catalog != (nl_catd)-1) + catclose(catalog); +#endif + return (result); } diff --git a/usr.bin/sort/sort.h b/usr.bin/sort/sort.h index 96b36a65f4e4..47ce6ea29ab3 100644 --- a/usr.bin/sort/sort.h +++ b/usr.bin/sort/sort.h @@ -49,7 +49,8 @@ #include extern nl_catd catalog; -#define getstr(n) catgets(catalog, 1, n, nlsstr[n]) +#define getstr(n) \ + (catalog == (nl_catd)-1 ? nlsstr[n] : catgets(catalog, 1, n, nlsstr[n])) #endif extern const char *nlsstr[];