Messages in this thread | | | Subject | Re: select() for delay. | From | "Madhu K.S." <> | Date | Tue, 25 Oct 2005 11:56:41 +0530 |
| |
Hi All,
Someone please comment on the entire patch functionality. I tested this patch, it seems to work fine.
Kindly suggest.
On Mon, 2005-10-24 at 18:57, Arjan van de Ven wrote: > On Mon, 2005-10-24 at 09:18 -0400, Steven Rostedt wrote: > > Hi Maduhu, > > > > On Mon, 2005-10-24 at 16:25 +0530, madhu.subbaiah@wipro.com wrote: > > > > > + put_user(sec, &tvp->tv_sec); > > > + put_user(usec, &tvp->tv_usec); > > > > I won't comment on the rest of the patch, but this part is definitely > > wrong. The pointer tvp is a user space address and once you dereference > > that pointer to get to tv_sec, you can have a fault, which might > > segfault the > > &pointer->member doesn't dereference the pointer, it just adds the > offset of "member" to the content of the pointer. > >
- 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/
| |