Messages in this thread | | | Date | Fri, 10 Jan 2014 12:43:54 -0800 | From | Greg Kroah-Hartman <> | Subject | Re: Intel MIC host driver: possible signed underflow (undefined behavior) in userspace API |
| |
On Fri, Jan 10, 2014 at 10:22:55AM -0800, Sudeep Dutt wrote: > On Fri, 2014-01-10 at 06:21 -0800, Greg Kroah-Hartman wrote: > > On Fri, Jan 10, 2014 at 05:56:25AM +0000, Mathieu Desnoyers wrote: > > > Hi, > > > > > > Looking at this commit: > > > > > > commit f69bcbf3b4c4b333dcd7a48eaf868bf0c88edab5 > > > Author: Ashutosh Dixit <ashutosh.dixit@intel.com> > > > Date: Thu Sep 5 16:42:18 2013 -0700 > > > > > > Intel MIC Host Driver Changes for Virtio Devices. > > > > > > Especially at: > > > > > > +struct mic_copy_desc { > > > +#ifdef __KERNEL__ > > > + struct iovec __user *iov; > > > +#else > > > + struct iovec *iov; > > > +#endif > > > + int iovcnt; > > > + __u8 vr_idx; > > > + __u8 update_used; > > > + __u32 out_len; > > > +};
Oh, there's also the obvious 32/64 bit userspace/kernel issue here as well, but I'm sure you all know how to handle that properly. It would be nice to fix that up, if you can change the ABI still.
thanks,
greg k-h
| |