Messages in this thread | | | Date | Mon, 10 Mar 2003 12:28:10 +0100 | From | Andi Kleen <> | Subject | Re: sys32_ioctl -> compat_ioctl -- generic |
| |
> > > +{ > > > + struct file * filp; > > > > > + filp = fget(fd); > > > > > + /* find the name of the device. */ > > > + if (path) { > > > + struct file *f = fget(fd); > > > > Is it really necessary to do another fget(fd) ? > > This is andi's code, but it seems unneeded, fixed.
Yes, it's redundant.
-Andi
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |