On Mon, 2016-04-18 at 14:10 -0700, Roopa Prabhu wrote:

> +     if (filter_mask & IFLA_STATS_FILTER_BIT(IFLA_STATS_LINK_64)) {
> +             struct rtnl_link_stats64 *sp;
> +
> +             attr = nla_reserve(skb, IFLA_STATS_LINK_64,
> +                                sizeof(struct rtnl_link_stats64));
> +             if (!attr)
> +                     goto nla_put_failure;
> +
> +             sp = nla_data(attr);

Are you sure we have a guarantee that sp is aligned to u64 fields ?

x86 does not care, but some arches would have a potential misalign
access here.


> +             dev_get_stats(dev, sp);
> +     }


Reply via email to