From owner-freebsd-usb@FreeBSD.ORG Sun Nov 7 07:05:59 2004 Return-Path: Delivered-To: freebsd-usb@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 6C6FB16A4CE for ; Sun, 7 Nov 2004 07:05:59 +0000 (GMT) Received: from pimout1-ext.prodigy.net (pimout1-ext.prodigy.net [207.115.63.77]) by mx1.FreeBSD.org (Postfix) with ESMTP id 493E143D1F for ; Sun, 7 Nov 2004 07:05:56 +0000 (GMT) (envelope-from julian@elischer.org) Received: from [192.168.1.102] (adsl-68-123-122-146.dsl.snfc21.pacbell.net [68.123.122.146])iA775pMt083610; Sun, 7 Nov 2004 02:05:52 -0500 Message-ID: <418DC94E.3010003@elischer.org> Date: Sat, 06 Nov 2004 23:05:50 -0800 From: Julian Elischer User-Agent: Mozilla/5.0 (X11; U; FreeBSD i386; en-US; rv:1.8a3) Gecko/20041017 X-Accept-Language: en, hu MIME-Version: 1.0 To: Craig Rodrigues References: <20041107064227.GA79915@crodrigues.org> In-Reply-To: <20041107064227.GA79915@crodrigues.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit cc: freebsd-usb@freebsd.org Subject: Re: Changing permissions of /dev/usb[n] to 664? X-BeenThere: freebsd-usb@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: FreeBSD support for USB List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 07 Nov 2004 07:05:59 -0000 Craig Rodrigues wrote: > Hi, > > Is there any particular reason to leave the > permissions of /dev/usb[n] at 660 instead of 664? > > That would allow non-root users to run usbdevs > to list the USB devices on the system. > > > --- usb.c.orig Sun Nov 7 01:41:43 2004 > +++ usb.c Sun Nov 7 01:41:56 2004 > @@ -320,11 +320,11 @@ > /* The per controller devices (used for usb_discover) */ > /* XXX This is redundant now, but old usbd's will want it */ > sc->sc_usbdev = make_dev(&usb_cdevsw, device_get_unit(self), UID_ROOT, > - GID_OPERATOR, 0660, "usb%d", device_get_unit(self)); > + GID_OPERATOR, 0664, "usb%d", device_get_unit(self)); > if (usb_ndevs++ == 0) { > /* The device spitting out events */ > usb_dev = make_dev(&usb_cdevsw, USB_DEV_MINOR, UID_ROOT, > - GID_OPERATOR, 0660, "usb"); > + GID_OPERATOR, 0664, "usb"); > } > #endif > > > > Do you WANT non operators to be aware of the hardware configuration? Surely people who need to know can be in group operator. I'm not saying that you don't, just that it needs to be considerred. And is that ALL they can do if you change it?