Re: [PATCH ovpn net] ovpn: fix use after free in unlock_ovpn()
Antonio Quartulli <[email protected]>
| Newsgroups | gmane.network.openvpn.devel |
|---|---|
| Message-ID | <178092833773.3219682.7592366542484695731.b4-ty@b4> |
From: Antonio Quartulli <[email protected]> On Mon, 08 Jun 2026 16:04:46 +0200, Marco Baffo wrote: > unlock_ovpn() iterates over the release_list using llist_for_each_entry() > and drops the peer reference inside the loop body via ovpn_peer_put(). > > If this drops the last reference, the peer is eventually freed. However, > llist_for_each_entry() reads peer->release_entry.next in the loop advance > expression, which runs after the body. By that time the peer may have > already been freed, resulting in a use after free when advancing to the > next list entry. > > [...] Applied, thanks! [1/1] ovpn: fix use after free in unlock_ovpn() commit: b53407df27741dc81a85e1aec63fefb1da19ee8d Best regards, -- Antonio Quartulli <[email protected]>