Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 9d2727a3 authored by Mohit Aggarwal's avatar Mohit Aggarwal Committed by Manoj Prabhu B
Browse files

diag: Add NULL pointer checks in mask updates 



Currently, NULL pointer checks are missing in mask
update functions which might lead to NULL pointer
dereference issues. This patch fixes the issue by
adding appropriate checks.

CRs-Fixed: 2015104
Change-Id: I9df99208e283c8d90921c6e20a6a80f88c18a327
Signed-off-by: default avatarMohit Aggarwal <maggarwa@codeaurora.org>
parent 8ba7f3de

drivers/char/diag/diag_masks.c

+7 −3
Original line number Diff line number Diff line
@@ -113,10 +113,12 @@ static void diag_send_log_mask_update(uint8_t peripheral, int equip_id) 
	else

		mask_info = &log_mask;



	if (!mask_info)

	if (!mask_info || !mask_info->ptr || !mask_info->update_buf)

		return;



	mask = (struct diag_log_mask_t *)mask_info->ptr;

	if (!mask->ptr)

		return;

	buf = mask_info->update_buf;



	switch (mask_info->status) {
@@ -225,7 +227,7 @@ static void diag_send_event_mask_update(uint8_t peripheral) 
	else

		mask_info = &event_mask;



	if (!mask_info)

	if (!mask_info || !mask_info->ptr || !mask_info->update_buf)

		return;



	buf = mask_info->update_buf;
@@ -306,10 +308,12 @@ static void diag_send_msg_mask_update(uint8_t peripheral, int first, int last) 
	else

		mask_info = &msg_mask;



	if (!mask_info)

	if (!mask_info || !mask_info->ptr || !mask_info->update_buf)

		return;



	mask = (struct diag_msg_mask_t *)mask_info->ptr;

	if (!mask->ptr)

		return;

	buf = mask_info->update_buf;

	mutex_lock(&mask_info->lock);

	switch (mask_info->status) {