diff options
| author | Bart Van Assche <bvanassche@acm.org> | 2018-10-09 00:28:51 +0300 |
|---|---|---|
| committer | Christoph Hellwig <hch@lst.de> | 2018-10-17 09:58:26 +0300 |
| commit | 5eadc9cce17100caef88e972abeeeca7ef6d8a92 (patch) | |
| tree | b1a5db932cce38dae81a9ba074d640b3c2525657 /drivers/nvme/target/admin-cmd.c | |
| parent | 40581d1a91a1527e1e15350e479156810a389a96 (diff) | |
| download | linux-5eadc9cce17100caef88e972abeeeca7ef6d8a92.tar.xz | |
nvmet: use strlcpy() instead of strcpy()
Although the code modified by this patch looks fine to me, this patch avoids
that Coverity reports the following complaint (ID 1364971 and ID 1364973):
"You might overrun the 256-character fixed-size string id->subnqn".
Signed-off-by: Bart Van Assche <bvanassche@acm.org>
Reviewed-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Diffstat (limited to 'drivers/nvme/target/admin-cmd.c')
| -rw-r--r-- | drivers/nvme/target/admin-cmd.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/nvme/target/admin-cmd.c b/drivers/nvme/target/admin-cmd.c index 7a45f4477679..1179f6314323 100644 --- a/drivers/nvme/target/admin-cmd.c +++ b/drivers/nvme/target/admin-cmd.c @@ -353,7 +353,7 @@ static void nvmet_execute_identify_ctrl(struct nvmet_req *req) if (req->port->inline_data_size) id->sgls |= cpu_to_le32(1 << 20); - strcpy(id->subnqn, ctrl->subsys->subsysnqn); + strlcpy(id->subnqn, ctrl->subsys->subsysnqn, sizeof(id->subnqn)); /* Max command capsule size is sqe + single page of in-capsule data */ id->ioccsz = cpu_to_le32((sizeof(struct nvme_command) + |