lkml.org 
[lkml]   [2014]   [Jan]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: 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


\
 
 \ /
  Last update: 2014-01-10 22:01    [W:0.045 / U:0.124 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site