be2iscsi: check ip buffer before copying

[platform/adaptation/renesas_rcar/renesas_kernel.git] / drivers / scsi / be2iscsi / be_mgmt.c

diff --git a/drivers/scsi/be2iscsi/be_mgmt.c b/drivers/scsi/be2iscsi/be_mgmt.c
index b2fcac7..5bb9406 100644 (file)
--- a/drivers/scsi/be2iscsi/be_mgmt.c
+++ b/drivers/scsi/be2iscsi/be_mgmt.c
@@ -897,17 +897,20 @@ mgmt_static_ip_modify(struct beiscsi_hba *phba,
 
        if (ip_action == IP_ACTION_ADD) {
                memcpy(req->ip_params.ip_record.ip_addr.addr, ip_param->value,
-                      ip_param->len);
+                      sizeof(req->ip_params.ip_record.ip_addr.addr));
 
                if (subnet_param)
                        memcpy(req->ip_params.ip_record.ip_addr.subnet_mask,
-                              subnet_param->value, subnet_param->len);
+                              subnet_param->value,
+                              sizeof(req->ip_params.ip_record.ip_addr.subnet_mask));
        } else {
                memcpy(req->ip_params.ip_record.ip_addr.addr,
-                      if_info->ip_addr.addr, ip_param->len);
+                      if_info->ip_addr.addr,
+                      sizeof(req->ip_params.ip_record.ip_addr.addr));
 
                memcpy(req->ip_params.ip_record.ip_addr.subnet_mask,
-                      if_info->ip_addr.subnet_mask, ip_param->len);
+                      if_info->ip_addr.subnet_mask,
+                      sizeof(req->ip_params.ip_record.ip_addr.subnet_mask));
        }
 
        rc = mgmt_exec_nonemb_cmd(phba, &nonemb_cmd, NULL, 0);
@@ -935,7 +938,7 @@ static int mgmt_modify_gateway(struct beiscsi_hba *phba, uint8_t *gt_addr,
        req->action = gtway_action;
        req->ip_addr.ip_type = BE2_IPV4;
 
-       memcpy(req->ip_addr.addr, gt_addr, param_len);
+       memcpy(req->ip_addr.addr, gt_addr, sizeof(req->ip_addr.addr));
 
        return mgmt_exec_nonemb_cmd(phba, &nonemb_cmd, NULL, 0);
 }