exofs: Cap on the memcpy() size
This data comes from the device, so probably it's fairly trustworthy but it makes the static checkers happy if we check it. [Boaz] the system_id_len is zero, if not present, or always OSD_SYSTEMID_LEN. So always copy OSD_SYSTEMID_LEN bytes. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Boaz Harrosh <bharrosh@panasas.com>
This commit is contained in:
parent
3e57638bb1
commit
72749a270b
|
@ -529,7 +529,8 @@ static int exofs_devs_2_odi(struct exofs_dt_device_info *dt_dev,
|
||||||
struct osd_dev_info *odi)
|
struct osd_dev_info *odi)
|
||||||
{
|
{
|
||||||
odi->systemid_len = le32_to_cpu(dt_dev->systemid_len);
|
odi->systemid_len = le32_to_cpu(dt_dev->systemid_len);
|
||||||
memcpy(odi->systemid, dt_dev->systemid, odi->systemid_len);
|
if (likely(odi->systemid_len))
|
||||||
|
memcpy(odi->systemid, dt_dev->systemid, OSD_SYSTEMID_LEN);
|
||||||
|
|
||||||
odi->osdname_len = le32_to_cpu(dt_dev->osdname_len);
|
odi->osdname_len = le32_to_cpu(dt_dev->osdname_len);
|
||||||
odi->osdname = dt_dev->osdname;
|
odi->osdname = dt_dev->osdname;
|
||||||
|
|
Loading…
Reference in New Issue