Messages in this thread | | | Subject | Re: [PATCH net] be2net: Fix memory leak in be_cmd_get_profile_config() | From | Ivan Vecera <> | Date | Wed, 5 Sep 2018 14:45:07 +0200 |
| |
On 5.9.2018 14:37, Petr Oros wrote: > DMA allocated memory is lost in be_cmd_get_profile_config() when we > call it with non-NULL port_res parameter. > > Signed-off-by: Petr Oros <poros@redhat.com> > --- > drivers/net/ethernet/emulex/benet/be_cmds.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/ethernet/emulex/benet/be_cmds.c b/drivers/net/ethernet/emulex/benet/be_cmds.c > index ff92ab1daeb8..1e9d882c04ef 100644 > --- a/drivers/net/ethernet/emulex/benet/be_cmds.c > +++ b/drivers/net/ethernet/emulex/benet/be_cmds.c > @@ -4500,7 +4500,7 @@ int be_cmd_get_profile_config(struct be_adapter *adapter, > port_res->max_vfs += le16_to_cpu(pcie->num_vfs); > } > } > - return status; > + goto err; > } > > pcie = be_get_pcie_desc(resp->func_param, desc_count, >
Good catch, Petr.
Reviewed-by: Ivan Vecera <ivecera@redhat.com>
| |