Quoting Dwight Engen (dwight.en...@oracle.com):
> On Fri, 3 May 2013 11:17:08 -0500
> Serge Hallyn <serge.hal...@ubuntu.com> wrote:
> 
> > can that be correct?  That changes the value of the
> > pointer.  NLMSG_DATA adds aligned sizeof(struct nlmsghdr).
> 
> The current code is relying on the fact that our genlmsg looks like
> this:
> 
> struct genlmsg {
>       struct nlmsghdr nlmsghdr;
>       struct genlmsghdr genlmsghdr;
> };
> 
> so NLMSG_DATA(&request->nlmsghdr) == genlmsghdr (I think sizeof struct
> nlmsghdr will make genlmsghdr be aligned anyway?). Right now lxc doesn't
> use this code so I cannot test it in lxc, but let me write a test
> program that dumps the pointers using the old way and the new way. We
> do have a 2*PAGE_SIZE buffer these point to so I'm sure its not
> overflowing, so maybe we do just want to ignore coverity.

Thanks, I'll go ahead and apply the rest and hold off on this one.

------------------------------------------------------------------------------
Get 100% visibility into Java/.NET code with AppDynamics Lite
It's a free troubleshooting tool designed for production
Get down to code-level detail for bottlenecks, with <2% overhead.
Download for free and get started troubleshooting in minutes.
http://p.sf.net/sfu/appdyn_d2d_ap2
_______________________________________________
Lxc-devel mailing list
Lxc-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/lxc-devel

Reply via email to