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); > + }