On 17-04-20 12:09 PM, Eric Dumazet wrote:
On Thu, 2017-04-20 at 09:06 -0400, Jamal Hadi Salim wrote:nest = nla_nest_start(skb, TCA_ACT_TAB); if (nest == NULL) @@ -1113,6 +1136,8 @@ static int tc_dump_action(struct sk_buff *skb, struct netlink_callback *cb) if (ret > 0) { nla_nest_end(skb, nest); ret = skb->len; + memcpy(nla_data(count_attr), &cb->args[1], sizeof(u32));This will not work on BigEndian 64bit hosts, since cb->args[1] is bigger than 32bit.
Ok, thanks. I will assign to a 32 bit var first then memcpy in the next iteration (tomorrow). cheers, jamal
