Re: [PATCH 1/3] iwlwifi: mvm: Fix a memory leak in an error handling path in 'iwl_mvm_sar_get_wgds_table()'

2017-08-07 Thread Luca Coelho
On Mon, 2017-08-07 at 16:37 +0300, Kalle Valo wrote: > Luca Coelho writes: > > > On Mon, 2017-08-07 at 15:57 +0300, Kalle Valo wrote: > > > Luca Coelho writes: > > > > > > > From: Christophe Jaillet > > > > > > > > We should free 'wgds.pointer' here as done a few lines above in another > > >

Re: [PATCH 1/3] iwlwifi: mvm: Fix a memory leak in an error handling path in 'iwl_mvm_sar_get_wgds_table()'

2017-08-07 Thread Kalle Valo
Luca Coelho writes: > On Mon, 2017-08-07 at 15:57 +0300, Kalle Valo wrote: >> Luca Coelho writes: >> >> > From: Christophe Jaillet >> > >> > We should free 'wgds.pointer' here as done a few lines above in another >> > error handling path. >> > It was allocated within 'acpi_evaluate_object()'.

Re: [PATCH 1/3] iwlwifi: mvm: Fix a memory leak in an error handling path in 'iwl_mvm_sar_get_wgds_table()'

2017-08-07 Thread Luca Coelho
On Mon, 2017-08-07 at 15:57 +0300, Kalle Valo wrote: > Luca Coelho writes: > > > From: Christophe Jaillet > > > > We should free 'wgds.pointer' here as done a few lines above in another > > error handling path. > > It was allocated within 'acpi_evaluate_object()'. > > > > Fixes: c52030a01ccc (

Re: [PATCH 1/3] iwlwifi: mvm: Fix a memory leak in an error handling path in 'iwl_mvm_sar_get_wgds_table()'

2017-08-07 Thread Kalle Valo
Luca Coelho writes: > From: Christophe Jaillet > > We should free 'wgds.pointer' here as done a few lines above in another > error handling path. > It was allocated within 'acpi_evaluate_object()'. > > Fixes: c52030a01ccc ("iwlwifi: mvm: add GEO_TX_POWER_LIMIT cmd for geographic > tx power tabl

[PATCH 1/3] iwlwifi: mvm: Fix a memory leak in an error handling path in 'iwl_mvm_sar_get_wgds_table()'

2017-08-05 Thread Luca Coelho
From: Christophe Jaillet We should free 'wgds.pointer' here as done a few lines above in another error handling path. It was allocated within 'acpi_evaluate_object()'. Fixes: c52030a01ccc ("iwlwifi: mvm: add GEO_TX_POWER_LIMIT cmd for geographic tx power table") Signed-off-by: Christophe JAILLE