From owner-svn-src-head@freebsd.org Wed Nov 18 21:46:44 2020 Return-Path: Delivered-To: svn-src-head@mailman.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.nyi.freebsd.org (Postfix) with ESMTP id 83C4A473151 for ; Wed, 18 Nov 2020 21:46:44 +0000 (UTC) (envelope-from jrtc27@jrtc27.com) Received: from mail-wm1-f46.google.com (mail-wm1-f46.google.com [209.85.128.46]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (2048 bits) client-digest SHA256) (Client CN "smtp.gmail.com", Issuer "GTS CA 1O1" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4CbxGX343Wz4jHZ for ; Wed, 18 Nov 2020 21:46:44 +0000 (UTC) (envelope-from jrtc27@jrtc27.com) Received: by mail-wm1-f46.google.com with SMTP id a186so1766990wme.1 for ; Wed, 18 Nov 2020 13:46:44 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:subject:from:in-reply-to:date:cc :content-transfer-encoding:message-id:references:to; bh=sXfnZOYf5c21NAfoV0Jp0YCOfGUY++5NdASbcbP3Lzs=; b=duCebuM23g+Vnltgc5U9j+bpFyK71hfqEGesaceWzVeji7yJv+reQUDYRyxc26jgWL zCeTeEvhqxw36p+NP8OJf6d1sOrJL6C1Mw8bUT0RM2+xAHwJjbLmMqqSWez+/KuflYSW AiWYfRO9O2JS+9cYMBZrr5dro42o+HIF0xN5QCHmjGOAA4p/PvUQ5Biaz5FdOHNN1fQp Gpn/I1qFa7+cwRMnIoknYi08TxYBOImHiirZC1uKjMmgb4WYh7L9Nk0Zt8R5nmys+LTS ltu5uBO4ozE5ypURkHAJEZbJ2J10jd7SvPNtH+2z93Q6lK9pkjzwXWO/6Ygb9xfN+Sw8 G3ow== X-Gm-Message-State: AOAM530fPIdW7jpUvBIh2QAHa+b6oXvGQ8G3IcdEt+PF2j/lJhDzUeyy 1BKh0buNNCqCD/i9d1lpbJM6hg== X-Google-Smtp-Source: ABdhPJwO9/mfSiOEMvOhLb6OYPb/CyFQVmHclImjah/ZoYFpBLFiT3Z/cBqOVb5hGAyov/lJJ4ukqw== X-Received: by 2002:a1c:1b12:: with SMTP id b18mr1122274wmb.119.1605736002930; Wed, 18 Nov 2020 13:46:42 -0800 (PST) Received: from [192.168.149.251] (trinity-students-nat.trin.cam.ac.uk. [131.111.193.104]) by smtp.gmail.com with ESMTPSA id s202sm5463500wme.39.2020.11.18.13.46.42 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Wed, 18 Nov 2020 13:46:42 -0800 (PST) Content-Type: text/plain; charset=utf-8 Mime-Version: 1.0 (Mac OS X Mail 13.4 \(3608.120.23.2.4\)) Subject: Re: svn commit: r367813 - head/lib/libutil From: Jessica Clarke In-Reply-To: Date: Wed, 18 Nov 2020 21:46:41 +0000 Cc: =?utf-8?Q?Stefan_E=C3=9Fer?= , src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Content-Transfer-Encoding: quoted-printable Message-Id: References: <202011181944.0AIJiUU3003699@repo.freebsd.org> To: Mateusz Guzik X-Mailer: Apple Mail (2.3608.120.23.2.4) X-Rspamd-Queue-Id: 4CbxGX343Wz4jHZ X-Spamd-Bar: ---- Authentication-Results: mx1.freebsd.org; none X-Spamd-Result: default: False [-4.00 / 15.00]; REPLY(-4.00)[] X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 18 Nov 2020 21:46:44 -0000 On 18 Nov 2020, at 21:40, Mateusz Guzik wrote: >=20 > On 11/18/20, Stefan E=C3=9Fer wrote: >> Author: se >> Date: Wed Nov 18 19:44:30 2020 >> New Revision: 367813 >> URL: https://svnweb.freebsd.org/changeset/base/367813 >>=20 >> Log: >> Add function getlocalbase() to libutil. >>=20 >> This function returns the path to the local software base directory, = by >> default "/usr/local" (or the value of _PATH_LOCALBASE in = include/paths.h >> when building the world). >>=20 >> The value returned can be overridden by 2 methods: >>=20 >> - the LOCALBASE environment variable (ignored by SUID programs) >> - else a non-default user.localbase sysctl value >>=20 >> Reviewed by: hps (earlier version) >> Relnotes: yes >> Differential Revision: https://reviews.freebsd.org/D27236 >>=20 >> Added: >> head/lib/libutil/getlocalbase.3 (contents, props changed) >> head/lib/libutil/getlocalbase.c (contents, props changed) >> Modified: >> head/lib/libutil/Makefile >> head/lib/libutil/libutil.h >>=20 >> Modified: head/lib/libutil/Makefile >> = =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D >> --- head/lib/libutil/Makefile Wed Nov 18 19:35:30 2020 = (r367812) >> +++ head/lib/libutil/Makefile Wed Nov 18 19:44:30 2020 = (r367813) >> @@ -12,7 +12,8 @@ PACKAGE=3D runtime >> LIB=3D util >> SHLIB_MAJOR=3D 9 >>=20 >> -SRCS=3D _secure_path.c auth.c expand_number.c flopen.c = fparseln.c gr_util.c >> \ >> +SRCS=3D _secure_path.c auth.c expand_number.c flopen.c = fparseln.c \ >> + getlocalbase.c gr_util.c \ >> hexdump.c humanize_number.c kinfo_getfile.c \ >> kinfo_getallproc.c kinfo_getproc.c kinfo_getvmmap.c \ >> kinfo_getvmobject.c kld.c \ >> @@ -30,7 +31,7 @@ CFLAGS+=3D -DINET6 >>=20 >> CFLAGS+=3D -I${.CURDIR} -I${SRCTOP}/lib/libc/gen/ >>=20 >> -MAN+=3D expand_number.3 flopen.3 fparseln.3 hexdump.3 \ >> +MAN+=3D expand_number.3 flopen.3 fparseln.3 getlocalbase.3 = hexdump.3 \ >> humanize_number.3 kinfo_getallproc.3 kinfo_getfile.3 \ >> kinfo_getproc.3 kinfo_getvmmap.3 kinfo_getvmobject.3 kld.3 \ >> login_auth.3 login_cap.3 \ >>=20 >> Added: head/lib/libutil/getlocalbase.3 >> = =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D >> --- /dev/null 00:00:00 1970 (empty, because file is newly = added) >> +++ head/lib/libutil/getlocalbase.3 Wed Nov 18 19:44:30 2020 = (r367813) >> @@ -0,0 +1,99 @@ >> +.\" >> +.\" SPDX-License-Identifier: BSD-2-Clause-FreeBSD >> +.\" >> +.\" Copyright 2020 Scott Long >> +.\" Copyright 2020 Stefan E=C3=9Fer >> +.\" >> +.\" Redistribution and use in source and binary forms, with or = without >> +.\" modification, are permitted provided that the following = conditions >> +.\" are met: >> +.\" 1. Redistributions of source code must retain the above = copyright >> +.\" notice, this list of conditions and the following disclaimer. >> +.\" 2. Redistributions in binary form must reproduce the above = copyright >> +.\" notice, this list of conditions and the following disclaimer = in the >> +.\" documentation and/or other materials provided with the >> distribution. >> +.\" >> +.\" THIS SOFTWARE IS PROVIDED BY THE AUTHOR AND CONTRIBUTORS ``AS = IS'' AND >> +.\" ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED = TO, THE >> +.\" IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A = PARTICULAR >> PURPOSE >> +.\" ARE DISCLAIMED. IN NO EVENT SHALL THE AUTHOR OR CONTRIBUTORS BE >> LIABLE >> +.\" FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR >> CONSEQUENTIAL >> +.\" DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF = SUBSTITUTE >> GOODS >> +.\" OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS = INTERRUPTION) >> +.\" HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN = CONTRACT, >> STRICT >> +.\" LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING = IN ANY >> WAY >> +.\" OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE = POSSIBILITY OF >> +.\" SUCH DAMAGE. >> +.\" >> +.\" $FreeBSD$ >> +.\" >> +.Dd November 18, 2020 >> +.Dt GETLOCALBASE 3 >> +.Os >> +.Sh NAME >> +.Nm getlocalbase >> +.Nd "return the path to the local software directory" >> +.Sh LIBRARY >> +.Lb libutil >> +.Sh SYNOPSIS >> +.In libutil.h >> +.Ft const char* >> +.Fn getlocalbase "void" >> +.Sh DESCRIPTION >> +The >> +.Fn getlocalbase >> +function returns the path to the local software base directory. >> +Normally this is the >> +.Pa /usr/local >> +directory. >> +First the >> +.Ev LOCALBASE >> +environment variable is checked. >> +If that does not exist then the >> +.Va user.localbase >> +sysctl is checked. >> +If that also does not exist then the value of the >> +.Dv _PATH_LOCALBASE >> +compile-time variable is used. >> +If that is undefined then the default of >> +.Pa /usr/local >> +is used. >> +.Pp >> +The value returned by the >> +.Fn getlocalbase >> +function shall not be modified. >> +.Sh IMPLEMENTATION NOTES >> +Calls to >> +.Fn getlocalbase >> +will perform a setugid check on the running binary before checking = the >> +environment. >> +.Sh RETURN VALUES >> +The >> +.Fn getlocalbase >> +function always succeeds and returns a pointer to a string, whose = length >> +may exceed MAXPATHLEN if it has been derived from the environment = variable >> +LOCALBASE. >> +No length checks are performed on the result. >> +.Sh ENVIRONMENT >> +The >> +.Fn getlocalbase >> +library function retrieves the >> +.Ev LOCALBASE >> +environment variable. >> +.Sh ERRORS >> +The >> +.Fn getlocalbase >> +function always succeeds. >> +.Sh SEE ALSO >> +.Xr env 1 , >> +.Xr src.conf 5 , >> +.Xr sysctl 8 >> +.Sh HISTORY >> +The >> +.Nm >> +library function first appeared in >> +.Fx 13.0 . >> +.Sh AUTHORS >> +This >> +manual page was written by >> +.An Scott Long Aq Mt scottl@FreeBSD.org and Stefan E=C3=9Fer Aq Mt >> se@FreeBSD.org . >>=20 >> Added: head/lib/libutil/getlocalbase.c >> = =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D >> --- /dev/null 00:00:00 1970 (empty, because file is newly = added) >> +++ head/lib/libutil/getlocalbase.c Wed Nov 18 19:44:30 2020 = (r367813) >> @@ -0,0 +1,74 @@ >> +/*- >> + * SPDX-License-Identifier: BSD-2-Clause >> + * >> + * Copyright 2020 Stefan E=C3=9Fer >> + * >> + * Redistribution and use in source and binary forms, with or = without >> + * modification, are permitted provided that the following = conditions >> + * are met: >> + * 1. Redistributions of source code must retain the above copyright >> + * notice, this list of conditions and the following disclaimer. >> + * 2. Redistributions in binary form must reproduce the above = copyright >> + * notice, this list of conditions and the following disclaimer = in the >> + * documentation and/or other materials provided with the = distribution. >> + * >> + * THIS SOFTWARE IS PROVIDED BY THE REGENTS AND CONTRIBUTORS ``AS = IS'' AND >> + * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, = THE >> + * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A = PARTICULAR >> PURPOSE >> + * ARE DISCLAIMED. IN NO EVENT SHALL THE REGENTS OR CONTRIBUTORS BE >> LIABLE >> + * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR >> CONSEQUENTIAL >> + * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE = GOODS >> + * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS = INTERRUPTION) >> + * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN = CONTRACT, >> STRICT >> + * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN = ANY >> WAY >> + * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE = POSSIBILITY OF >> + * SUCH DAMAGE. >> + */ >> + >> +#include >> +__FBSDID("$FreeBSD$"); >> + >> +#include >> +#include >> +#include >> +#include >> +#include >> +#include >> +#include >> + >> +#ifndef _PATH_LOCALBASE >> +#define _PATH_LOCALBASE "/usr/local" >> +#endif >> + >> +const char * >> +getlocalbase(void) >> +{ >> + static const int localbase_oid[2] =3D {CTL_USER, = USER_LOCALBASE}; >=20 > There is no use for this to be static. >=20 >> + char *tmppath; >> + size_t tmplen; >> + static const char *localbase =3D NULL; >> + >> + if (issetugid() =3D=3D 0) { >> + tmppath =3D getenv("LOCALBASE"); >> + if (tmppath !=3D NULL && tmppath[0] !=3D '\0') >> + return (tmppath); >> + } >> + if (sysctl(localbase_oid, 2, NULL, &tmplen, NULL, 0) =3D=3D 0 && >> + (tmppath =3D malloc(tmplen)) !=3D NULL && >> + sysctl(localbase_oid, 2, tmppath, &tmplen, NULL, 0) =3D=3D = 0) { >=20 > Apart from the concurrency issue mentioned in the comment this is just > very wasteful. Instead you can have a small local buffer, say 128 > bytes and pass that to be populated. The sysctl handler than can > populate that and return an error if the size is too small. I don't > know if sysclt api allows it to return the set size as it is. Worst > case you can just retry with a bigger malloced buffer. >=20 > Once you get the result you can malloc a buffer and > atomic_cmpset_rel_ptr localbase to point to it. If this fails, another > thread got the result, you free your buffer and return (localbase). That would still need to be an acquire load (if using C11 atomics, technically consume is fine, but in practice is no more efficient). Jess