linux-kernel - Re: [PATCH] cfg80211: fix wext-compat memory leak

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <e7bb48f5-3933-6d0b-160f-8d0801e3c1a7@message-id.googlemail.com>
Date:   Mon, 1 Oct 2018 10:55:34 +0200
From:   Stefan Seyfried <stefan.seyfried@...glemail.com>
To:     Johannes Berg <johannes@...solutions.net>,
        linux-kernel@...r.kernel.org, linux-wireless@...r.kernel.org
Cc:     Stefan Seyfried <seife+kernel@...systems.com>
Subject: Re: [PATCH] cfg80211: fix wext-compat memory leak

Hi Johannes,

Am 01.10.18 um 09:12 schrieb Johannes Berg:
> Hi Stefan,
> 
> Thanks for the investigation & fix. I've changed the fix to use
> cfg80211_sinfo_release_content(),

Being totally unfamiliar with the code, I was not sure about that :-)

> take care of the error path in
> cfg80211_wext_giwrate() and added a Fixes tag :-)

That's certainly much better.

Thanks!

(kudos to Vlastimil Babka who pointed out trace-cmd to me on the
 opensuse-kernel malinglist, which made the investigation quite easy)
-- 
Stefan Seyfried

"For a successful technology, reality must take precedence over
 public relations, for nature cannot be fooled." -- Richard Feynman