[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <MW5PR11MB5810DF9944097F37229CCA6EA3AD2@MW5PR11MB5810.namprd11.prod.outlook.com>
Date: Mon, 31 Mar 2025 10:25:03 +0000
From: "Korenblit, Miriam Rachel" <miriam.rachel.korenblit@...el.com>
To: Dan Carpenter <dan.carpenter@...aro.org>, "Anjaneyulu, Pagadala Yesu"
<pagadala.yesu.anjaneyulu@...el.com>
CC: "Berg, Johannes" <johannes.berg@...el.com>, "Berg, Benjamin"
<benjamin.berg@...el.com>, "linux-wireless@...r.kernel.org"
<linux-wireless@...r.kernel.org>, "linux-kernel@...r.kernel.org"
<linux-kernel@...r.kernel.org>, "kernel-janitors@...r.kernel.org"
<kernel-janitors@...r.kernel.org>
Subject: RE: [PATCH next] wifi: iwlwifi: mld: silence uninitialized variable
warning
> -----Original Message-----
> From: Dan Carpenter <dan.carpenter@...aro.org>
> Sent: Friday, 21 March 2025 16:36
> To: Anjaneyulu, Pagadala Yesu <pagadala.yesu.anjaneyulu@...el.com>
> Cc: Korenblit, Miriam Rachel <miriam.rachel.korenblit@...el.com>; Berg,
> Johannes <johannes.berg@...el.com>; Berg, Benjamin
> <benjamin.berg@...el.com>; linux-wireless@...r.kernel.org; linux-
> kernel@...r.kernel.org; kernel-janitors@...r.kernel.org
> Subject: [PATCH next] wifi: iwlwifi: mld: silence uninitialized variable warning
>
> The "resp_len" isn't initliazed if iwl_dhc_resp_data() fails.
>
> Fixes: b611cf6b57a8 ("wifi: iwlwifi: mld: add support for
> DHC_TOOLS_UMAC_GET_TAS_STATUS command")
> Signed-off-by: Dan Carpenter <dan.carpenter@...aro.org>
> ---
> drivers/net/wireless/intel/iwlwifi/mld/debugfs.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/wireless/intel/iwlwifi/mld/debugfs.c
> b/drivers/net/wireless/intel/iwlwifi/mld/debugfs.c
> index 453ce2ba39d1..89d95e9b4f30 100644
> --- a/drivers/net/wireless/intel/iwlwifi/mld/debugfs.c
> +++ b/drivers/net/wireless/intel/iwlwifi/mld/debugfs.c
> @@ -396,8 +396,8 @@ static ssize_t iwl_dbgfs_tas_get_status_read(struct
> iwl_mld *mld, char *buf,
> .data[0] = &cmd,
> };
> struct iwl_dhc_tas_status_resp *resp = NULL;
> + u32 resp_len = 0;
> ssize_t pos = 0;
> - u32 resp_len;
> u32 status;
> int ret;
>
> --
> 2.47.2
Acked-by: Miri Korenblit <miriam.rachel.korenblit@...el.com>
Powered by blists - more mailing lists