On Tue, Nov 10, 2015 at 06:58:05PM +0000, Patrick McHardy wrote:
> On 10.11, Pablo Neira Ayuso wrote:
> > On Tue, Nov 10, 2015 at 06:30:34PM +0000, Patrick McHardy wrote:
> > > > __module_get(src->ops->type->owner);
> > > > - memcpy(dst, src, src->ops->size);
> > > > + if (src->ops->clone) {
> > > > + memcpy(dst, src, sizeof(*src));
> > >
> > > Why copy if we clone? The function should do a full initialization if it
> > > is
> > > present I would say.
> >
> > This is not copying the variable length data area of the expression,
> > just the expression head.
>
> Ah right. But that is only ->ops. We can set this directly, should generate
> better code and be easier to understand.
I left the memcpy just to avoid that we forget in case we ever get
more data there (unlikely). So I'll set the pointer instead.
If no further objections, will make those two changes locally and will
push this upstream.
Thanks!
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
