diag: Update mask buffer after reallocation

Currently, reallocated mask update buffers are not
updated if the received mask range is more than the
mask update buffer length. Update the reallocated buffer
address before writing the mask to peripherals.

CRs-Fixed: 2266693
Change-Id: I6b506ce68e17b7da61926b0f9543157812a8c555
Signed-off-by: Manoj Prabhu B <bmanoj@codeaurora.org>
This commit is contained in:
Manoj Prabhu B 2018-06-27 16:05:57 +05:30 committed by Gerrit - the friendly Code Review server
parent 38a0bccad5
commit 0fe549f6b6

View file

@ -185,10 +185,11 @@ static void diag_send_log_mask_update(uint8_t peripheral, int equip_id)
} }
mask_info->update_buf = temp; mask_info->update_buf = temp;
mask_info->update_buf_len = header_len + mask_size; mask_info->update_buf_len = header_len + mask_size;
buf = temp;
} }
memcpy(buf, &ctrl_pkt, header_len); memcpy(buf, &ctrl_pkt, header_len);
if (mask_size > 0) if (mask_size > 0 && mask_size <= LOG_MASK_SIZE)
memcpy(buf + header_len, mask->ptr, mask_size); memcpy(buf + header_len, mask->ptr, mask_size);
mutex_unlock(&mask->lock); mutex_unlock(&mask->lock);
@ -286,9 +287,16 @@ static void diag_send_event_mask_update(uint8_t peripheral)
} else { } else {
mask_info->update_buf = temp; mask_info->update_buf = temp;
mask_info->update_buf_len = temp_len; mask_info->update_buf_len = temp_len;
buf = temp;
} }
} }
if (num_bytes > 0 && num_bytes < mask_info->mask_len)
memcpy(buf + sizeof(header), mask_info->ptr, num_bytes); memcpy(buf + sizeof(header), mask_info->ptr, num_bytes);
else {
pr_err("diag: num_bytes(%d) is not satisfying length condition\n",
num_bytes);
goto err;
}
write_len += num_bytes; write_len += num_bytes;
break; break;
default: default:
@ -404,6 +412,7 @@ static void diag_send_msg_mask_update(uint8_t peripheral, int first, int last)
} else { } else {
mask_info->update_buf = temp; mask_info->update_buf = temp;
mask_info->update_buf_len = temp_len; mask_info->update_buf_len = temp_len;
buf = temp;
pr_debug("diag: In %s, successfully reallocated msg_mask update buffer to len: %d\n", pr_debug("diag: In %s, successfully reallocated msg_mask update buffer to len: %d\n",
__func__, mask_info->update_buf_len); __func__, mask_info->update_buf_len);
} }