lkml.org 
[lkml]   [2023]   [Mar]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH] atm: he: fix potential ioremap leak of membase in he_dev
From


On 3/10/23 19:28, Francois Romieu wrote:
> Gencen Gan <u202011061@gmail.com> :
>> In the function he_start() in drivers/atm/he.c, there
>> is no unmapping of he_dev->membase in the branch that
>> exits due to an error like reset failure, which may
>> cause a memory leak.
>
> Why would he_dev->membase not be unmapped in he_stop() ?
>
> he_stop() is paired with he_start() as soon a he_start() returns
> anything different from 0 in he_init_one(). I see no other place
> where he_start() is used.

Yes, you're right. We will check more about reports from the static
checker Smatch.

Smatch should make a false positive here, I think it might be that,
Smatch has an assumption about do and its paired undo functions. The do
function should clean up its own allocation operations. And the paired
undo function can be only called if the do function succeeds.

+cc Dan Carpenter

Maybe @Dan could tell more about this point.

>
> The atm_dev/he_dev pointers also seem correctly set.
>

\
 
 \ /
  Last update: 2023-03-27 00:53    [W:0.060 / U:0.152 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site