
3 Sep
2019
3 Sep
'19
11:25 p.m.
On Mon, Sep 2, 2019 at 4:55 PM Heinrich Schuchardt xypron.glpk@gmx.de wrote:
rpc_pkt.u.call.data is an array of uint32_t. There is no need to convert it to uint32_t *.
memcpy() expects void * as it 1st and 2nd argument. There is no point in converting pointers to char * before passing them to memcpy().
In ntohl(data[1]) != 0 calling ntohl() is superfluous. If the value is zero, does not depend on the byte order.
Signed-off-by: Heinrich Schuchardt xypron.glpk@gmx.de
Acked-by: Joe Hershberger joe.hershberger@ni.com