From owner-freebsd-audit Fri Jul 6 1:56:33 2001 Delivered-To: freebsd-audit@freebsd.org Received: from ringworld.nanolink.com (sentinel.office1.bg [195.24.48.182]) by hub.freebsd.org (Postfix) with SMTP id 83E1137B408 for ; Fri, 6 Jul 2001 01:56:23 -0700 (PDT) (envelope-from roam@orbitel.bg) Received: (qmail 1052 invoked by uid 1000); 6 Jul 2001 09:00:49 -0000 Date: Fri, 6 Jul 2001 12:00:49 +0300 From: Peter Pentchev To: arch@FreeBSD.org Cc: audit@FreeBSD.org Subject: [Fwd: Re: kern/26317: /modules not created by make installkernel] Message-ID: <20010706120049.A590@ringworld.oblivion.bg> Mail-Followup-To: arch@FreeBSD.org, audit@FreeBSD.org Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.2.5i Sender: owner-freebsd-audit@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG Hi, Anybody see how much harm can come if this patch is committed? It does really solve a possible problem - a removed /modules dir - by interrupting the installworld process instead of happily installing the last module into a file named /modules. G'luck, Peter -- Nostalgia ain't what it used to be. ----- Forwarded message from Gregory Bond ----- Date: Sun, 8 Apr 2001 17:10:02 -0700 (PDT) To: freebsd-bugs@FreeBSD.org Cc: From: Gregory Bond Subject: Re: kern/26317: /modules not created by make installkernel Reply-To: Gregory Bond The following reply was made to PR kern/26317; it has been noted by GNATS. From: Gregory Bond To: David Taylor Cc: freebsd-gnats-submit@FreeBSD.ORG Subject: Re: kern/26317: /modules not created by make installkernel Date: Mon, 09 Apr 2001 10:03:24 +1000 This is a multipart MIME message. --==_Exmh_8133331360 Content-Type: text/plain; charset=us-ascii > It should at least bomb out with an error message, instead of pretending it > succesfully installed your kernel and modules, even though what it > _actually_ did was install one module as '/modules'... A simple patch to achieve this is attached. (I.e. install into /modules/ rather than /modules to force the error to be trapped.) --==_Exmh_8133331360 Content-Type: text/plain ; name="patch"; charset=us-ascii Content-Description: patch Content-Disposition: attachment; filename="patch" Index: kmod.mk =================================================================== RCS file: /usr/ncvs/src/sys/conf/kmod.mk,v retrieving revision 1.82.2.5 diff -u -r1.82.2.5 kmod.mk --- kmod.mk 2001/03/05 06:14:21 1.82.2.5 +++ kmod.mk 2001/04/08 23:59:26 @@ -217,7 +217,7 @@ realinstall: _SUBDIR ${INSTALL} ${COPY} -o ${KMODOWN} -g ${KMODGRP} -m ${KMODMODE} \ - ${_INSTALLFLAGS} ${PROG} ${DESTDIR}${KMODDIR} + ${_INSTALLFLAGS} ${PROG} ${DESTDIR}${KMODDIR}/ .if defined(LINKS) && !empty(LINKS) @set ${LINKS}; \ while test $$# -ge 2; do \ --==_Exmh_8133331360-- To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-bugs" in the body of the message ----- End forwarded message ----- To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-audit" in the body of the message