From mboxrd@z Thu Jan 1 00:00:00 1970 Delivery-date: Fri, 06 Jan 2023 12:20:04 +0100 Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by lore.white.stw.pengutronix.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1pDklZ-003PG6-4w for lore@lore.pengutronix.de; Fri, 06 Jan 2023 12:20:04 +0100 Received: from localhost ([127.0.0.1] helo=metis.ext.pengutronix.de) by metis.ext.pengutronix.de with esmtp (Exim 4.92) (envelope-from ) id 1pDklX-0005Vq-PG; Fri, 06 Jan 2023 12:20:03 +0100 Received: from ptx.hi.pengutronix.de ([2001:67c:670:100:1d::c0]) by metis.ext.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1pDklQ-0005Vh-UK; Fri, 06 Jan 2023 12:19:56 +0100 Received: from mol by ptx.hi.pengutronix.de with local (Exim 4.92) (envelope-from ) id 1pDklQ-0002DO-N4; Fri, 06 Jan 2023 12:19:56 +0100 Date: Fri, 6 Jan 2023 12:19:56 +0100 From: Michael Olbrich To: Ladislav Michl Message-ID: <20230106111956.GC21075@pengutronix.de> Mail-Followup-To: Ladislav Michl , ptxdist@pengutronix.de References: <20230104091433.2419958-1-christian.melki@t2data.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-Sent-From: Pengutronix Hildesheim X-URL: http://www.pengutronix.de/ X-Accept-Language: de,en X-Accept-Content-Type: text/plain User-Agent: Mutt/1.10.1 (2018-07-13) Subject: Re: [ptxdist] [PATCH v2] busybox: Version bump. 1.35.0 -> 1.36.0 X-BeenThere: ptxdist@pengutronix.de X-Mailman-Version: 2.1.29 Precedence: list List-Id: PTXdist Development Mailing List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: ptxdist@pengutronix.de Cc: ptxdist@pengutronix.de Sender: "ptxdist" X-SA-Exim-Connect-IP: 127.0.0.1 X-SA-Exim-Mail-From: ptxdist-bounces@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false On Fri, Jan 06, 2023 at 12:17:16PM +0100, Ladislav Michl wrote: > one more thing, commit f51d04cbcf2c ("treewide: Consistently use ':' > instead to separate user and group for chown") says: > > | Since coreutils 9.1 chown warns when a '.' is used as separator. Fix this > | consistently also for documentation and scripts that are expected to use > | busybox's chown which (up to now) has no problem with '.'. > > On Wed, Jan 04, 2023 at 10:14:33AM +0100, Christian Melki wrote: > > After a year of low activity and questionmarks, > > we finally have a new release of busybox. > > Release changelog @ https://busybox.net/ > [snip] > > diff --git a/config/busybox/Config.in b/config/busybox/Config.in > > index 329e2f8e8..199cc6ab2 100644 > > --- a/config/busybox/Config.in > > +++ b/config/busybox/Config.in > > @@ -267,10 +267,10 @@ config BUSYBOX_FEATURE_SUID_CONFIG > > > > The file has to be owned by user root, group root and has to be > > writeable only by root: > > - (chown 0:0 /etc/busybox.conf; chmod 600 /etc/busybox.conf) > > + (chown 0.0 /etc/busybox.conf; chmod 600 /etc/busybox.conf) > > The busybox executable has to be owned by user root, group > > root and has to be setuid root for this to work: > > - (chown 0:0 /bin/busybox; chmod 4755 /bin/busybox) > > + (chown 0.0 /bin/busybox; chmod 4755 /bin/busybox) > > > > Robert 'sandman' Griebl has more information here: > > . > [snip] > > diff --git a/config/busybox/networking/Config.in b/config/busybox/networking/Config.in > > index 27e0f44ba..3a3ad56d9 100644 > > --- a/config/busybox/networking/Config.in > > +++ b/config/busybox/networking/Config.in > > @@ -959,7 +959,7 @@ config BUSYBOX_TELNETD > > FEATURE_SUID enabled. And finally, you should make > > certain that busybox has been installed setuid root: > > > > - chown root:root /bin/busybox > > + chown root.root /bin/busybox > > chmod 4755 /bin/busybox > > > > with all that done, telnetd _should_ work.... > > Above two changes are reverting that, so it should be probably dropped. Hi, this is fine. This is an imported file and the commit mentioned above should not have touched this. It's just a comment anyways so it doesn't really matter. Michael -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |