Re: [patch] bfa: use strncpy() instead of memcpy()

2016-02-03 Thread Martin K. Petersen
> "Dan" == Dan Carpenter writes: Dan> BFA_MFG_NAME is "QLogic" which is only 7 bytes, but we are copying Dan> 8 bytes. It's harmless because the badding byte is likely zero but Dan> it makes static checkers complain. Applied to 4.6/scsi-queue. -- Martin K.

RE: [patch] bfa: use strncpy() instead of memcpy()

2016-02-02 Thread Anil Gurumurthy
mes E.J. Bottomley <jbottom...@odin.com>; Martin K. Petersen <martin.peter...@oracle.com>; linux-scsi <linux-scsi@vger.kernel.org>; linux-kernel <linux-ker...@vger.kernel.org>; kernel-janit...@vger.kernel.org Subject: [patch] bfa: use strncpy() instead of memcpy() BFA_MFG_NAME

[patch] bfa: use strncpy() instead of memcpy()

2016-01-30 Thread Dan Carpenter
BFA_MFG_NAME is "QLogic" which is only 7 bytes, but we are copying 8 bytes. It's harmless because the badding byte is likely zero but it makes static checkers complain. Signed-off-by: Dan Carpenter --- Technically the memset() is not needed because strncpy() will pad