On Thu, Aug 25, 2016 at 10:47:05AM -0600, Jason Gunthorpe wrote: > On Thu, Aug 25, 2016 at 01:53:06AM -0700, Yuval Shaia wrote: > > > - memcpy(dev->ibdev.node_desc, IWCH_NODE_DESC, sizeof(IWCH_NODE_DESC)); > > + memcpy(dev->ibdev.node_desc, IWCH_NODE_DESC, IB_DEVICE_NODE_DESC_MAX); > > That is not OK. Nor are any of the similar ones. > > Copies random kernel memory at the end of IWCH_NODE_DESC. Oops. Will post v1 in the next few minutes. > > > - memcpy(device->node_desc, device_modify->node_desc, 64); > > + memcpy(device->node_desc, device_modify->node_desc, > > + IB_DEVICE_NODE_DESC_MAX); > > And this and similar would be nicer as: > > > + memcpy(device->node_desc, device_modify->node_desc, > > sizeof(device_modify->node_desc)) > > Jason -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html