lkml.org 
[lkml]   [2022]   [Apr]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v2 1/6] staging: r8188eu: remove unused member free_bss_buf
On Sun, Apr 17, 2022 at 11:16:38PM +0300, Pavel Skripkin wrote:
> Hi Jaehee,
>
> On 4/17/22 23:14, Jaehee Park wrote:
> > My understanding of Pavel's response is the free_bss_buf member of the
> > pmlmepriv structure wasn't being used anywhere and that the
> > rtw_free_mlme_riv_ie_data function frees the memory of the pmlmepriv
> > structure so the second check is redundant.
> >
> > However, as Fabio said, the free_bss_buf member is being used and pbuf
> > memory is not being freed.
> > So I'll revert the patch as it was originally (which was just removing
> > the {} around the single if statement).
> >
>
> Why just `pbuf` allocation can't be removed? This memory is just unused,
> isn't it?
>
>
>
>
> With regards,
> Pavel Skripkin


The free_bss_buf member is unused. So it can just be removed right?
I guess I'm confused by what Pablo is saying about causing a memory
leak by getting rid of the pointer to the memory allocated by pbuf.
Sorry if I misunderstood.

Thanks,
Jaehee

\
 
 \ /
  Last update: 2022-04-17 22:43    [W:0.055 / U:2.076 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site