diff mbox series

[BlueZ] tools: Fix static analysis warnings

Message ID 20210122194258.538970-1-brian.gix@intel.com (mailing list archive)
State New, archived
Headers show
Series [BlueZ] tools: Fix static analysis warnings | expand

Commit Message

Brian Gix Jan. 22, 2021, 7:42 p.m. UTC 
Usage of potentially NULL pointers
---
 tools/btmgmt.c | 12 ++++++++++++
 1 file changed, 12 insertions(+)

Comments

bluez.test.bot@gmail.com Jan. 22, 2021, 10:58 p.m. UTC | #1 
This is automated email and please do not reply to this email!

Dear submitter,

Thank you for submitting the patches to the linux bluetooth mailing list.
This is a CI test results with your patch series:
PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=420291

---Test result---

##############################
Test: CheckPatch - PASS

##############################
Test: CheckGitLint - PASS

##############################
Test: CheckBuild - PASS

##############################
Test: MakeCheck - PASS



---
Regards,
Linux Bluetooth
Brian Gix Jan. 26, 2021, 9:31 p.m. UTC | #2 
Applied

On Fri, 2021-01-22 at 11:42 -0800, Brian Gix wrote:
> Usage of potentially NULL pointers
> ---
>  tools/btmgmt.c | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
> 
> diff --git a/tools/btmgmt.c b/tools/btmgmt.c
> index 383e7199e..f4eb541fa 100644
> --- a/tools/btmgmt.c
> +++ b/tools/btmgmt.c
> @@ -4912,6 +4912,12 @@ static void cmd_advmon_add_pattern(int argc, char **argv)
>  
>  	cp_len = sizeof(*cp) + argc * sizeof(struct mgmt_adv_pattern);
>  	cp = malloc0(cp_len);
> +	if (!cp) {
> +		error("Failed to alloc patterns.");
> +		success = false;
> +		goto done;
> +	}
> +
>  	cp->pattern_count = argc;
>  
>  	for (i = 0; i < argc; i++) {
> @@ -4987,6 +4993,12 @@ static void cmd_advmon_add_pattern_rssi(int argc, char **argv)
>  
>  	cp_len = sizeof(*cp) + argc * sizeof(struct mgmt_adv_pattern);
>  	cp = malloc0(cp_len);
> +	if (!cp) {
> +		error("Failed to alloc patterns.");
> +		success = false;
> +		goto done;
> +	}
> +
>  	cp->pattern_count = argc;
>  	cp->rssi.high_threshold = rssi_high;
>  	cp->rssi.low_threshold = rssi_low;
diff mbox series

Patch

diff --git a/tools/btmgmt.c b/tools/btmgmt.c
index 383e7199e..f4eb541fa 100644
--- a/tools/btmgmt.c
+++ b/tools/btmgmt.c
@@ -4912,6 +4912,12 @@  static void cmd_advmon_add_pattern(int argc, char **argv)
 
 	cp_len = sizeof(*cp) + argc * sizeof(struct mgmt_adv_pattern);
 	cp = malloc0(cp_len);
+	if (!cp) {
+		error("Failed to alloc patterns.");
+		success = false;
+		goto done;
+	}
+
 	cp->pattern_count = argc;
 
 	for (i = 0; i < argc; i++) {
@@ -4987,6 +4993,12 @@  static void cmd_advmon_add_pattern_rssi(int argc, char **argv)
 
 	cp_len = sizeof(*cp) + argc * sizeof(struct mgmt_adv_pattern);
 	cp = malloc0(cp_len);
+	if (!cp) {
+		error("Failed to alloc patterns.");
+		success = false;
+		goto done;
+	}
+
 	cp->pattern_count = argc;
 	cp->rssi.high_threshold = rssi_high;
 	cp->rssi.low_threshold = rssi_low;