Re: IF only........

From: Oliver Xymoron (oxymoron@waste.org)
Date: Sat Jul 22 2000 - 11:31:50 EST


On Sat, 22 Jul 2000, Vojtech Pavlik wrote:

> On Fri, Jul 21, 2000 at 08:25:06PM -0500, Oliver Xymoron wrote:
> > On Fri, 21 Jul 2000, Andre Hedrick wrote:
> >
> > > With out the full touch it will not work.
> > > 20% are functionally that have to be there for 2.4 to stand a chance of
> > > being correct.
> >
> > Ok, separate out those parts and submit them on their own. I read your
> > patch and I figured that some percentage was clearly unrelated.
> >
> > > 80% is TASKFILE rewrite to give us the armor.
> >
> > That means still 50k and that's way too large a rewrite to be
> > considered a bugfix. I think something like the below can do it in less
> > than 10 lines for 2.4 and might even pass the "obviously correct"
> > criterion. Save the command parser business for 2.5.
>
> Ok, what about just this simple little patch?

Aside from having lines longer than 80 columns it looks good. Here's the
analogous SCSI patch:

--- scsi_ioctl.c~ Tue Mar 14 00:15:03 2000
+++ scsi_ioctl.c Sat Jul 22 11:21:36 2000
@@ -410,7 +410,7 @@
         case SCSI_IOCTL_PROBE_HOST:
                 return ioctl_probe(dev->host, arg);
         case SCSI_IOCTL_SEND_COMMAND:
- if (!capable(CAP_SYS_ADMIN))
+ if (!capable(CAP_SYS_ADMIN) || !capable(CAP_SYS_RAWIO))
                         return -EACCES;
                 return scsi_ioctl_send_command((Scsi_Device *) dev,
                                              (Scsi_Ioctl_Command *) arg);

--
 "Love the dolphins," she advised him. "Write by W.A.S.T.E.." 

- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.rutgers.edu Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Sun Jul 23 2000 - 21:00:19 EST