Date: Thu, 10 Oct 2024 14:25:43 +0200 From: Kristof Provost <kp@FreeBSD.org> To: Konstantin Belousov <kib@FreeBSD.org> Cc: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org Subject: Re: git: 2851aafe96c1 - main - mlx5 ipsec_offload: ensure that driver does not dereference dead sahindex Message-ID: <9DD418B6-E346-4ED8-9212-6BD555FB2431@FreeBSD.org> In-Reply-To: <202410101000.49AA07wh039872@gitrepo.freebsd.org> References: <202410101000.49AA07wh039872@gitrepo.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--=_MailMate_25357E85-7349-467E-8FED-1670C4B2178C_= Content-Type: text/plain; format=flowed Content-Transfer-Encoding: quoted-printable On 10 Oct 2024, at 12:00, Konstantin Belousov wrote: > The branch main has been updated by kib: > > URL: = > https://cgit.FreeBSD.org/src/commit/?id=3D2851aafe96c1e357971f2b331fff8= 37ead20522b > > commit 2851aafe96c1e357971f2b331fff837ead20522b > Author: Konstantin Belousov <kib@FreeBSD.org> > AuthorDate: 2024-09-28 23:17:05 +0000 > Commit: Konstantin Belousov <kib@FreeBSD.org> > CommitDate: 2024-10-10 09:55:45 +0000 > > mlx5 ipsec_offload: ensure that driver does not dereference dead = > sahindex > > Take the sahtree rlock and check for the DEAD SA state before = > validating > and filling the SA xfrm attributes. > > Sponsored by: NVidia networking > --- I believe this commit breaks LINT-NOIP builds: --- kernel --- linking kernel ld: error: undefined symbol: ipsec_sahtree_rlock >>> referenced by mlx5_ipsec.c >>> mlx5_ipsec.o:(mlx5e_if_sa_newkey_onedir) >>> referenced by mlx5_ipsec.c >>> mlx5_ipsec.o:(mlx5e_if_sa_newkey_onedir) ld: error: undefined symbol: ipsec_sahtree_runlock >>> referenced by mlx5_ipsec.c >>> mlx5_ipsec.o:(mlx5e_if_sa_newkey_onedir) >>> referenced by mlx5_ipsec.c >>> mlx5_ipsec.o:(mlx5e_if_sa_newkey_onedir) >>> referenced by mlx5_ipsec.c >>> mlx5_ipsec.o:(mlx5e_if_sa_newkey_onedir) *** [kernel] Error code 1 Best regards, Kristof --=_MailMate_25357E85-7349-467E-8FED-1670C4B2178C_= Content-Type: text/html Content-Transfer-Encoding: quoted-printable <!DOCTYPE html> <html> <head> <meta http-equiv=3D"Content-Type" content=3D"text/xhtml; charset=3Dutf-8"= > </head> <body><div style=3D"font-family: sans-serif;"><div class=3D"markdown" sty= le=3D"white-space: normal;"> <p dir=3D"auto">On 10 Oct 2024, at 12:00, Konstantin Belousov wrote:</p> </div><div class=3D"plaintext" style=3D"white-space: normal;"><blockquote= style=3D"margin: 0 0 5px; padding-left: 5px; border-left: 2px solid #136= BCE; color: #136BCE;"><p dir=3D"auto">The branch main has been updated by= kib:</p> <p dir=3D"auto">URL: <a href=3D"https://cgit.FreeBSD.org/src/commit/?id=3D= 2851aafe96c1e357971f2b331fff837ead20522b">https://cgit.FreeBSD.org/src/co= mmit/?id=3D2851aafe96c1e357971f2b331fff837ead20522b</a></p> <p dir=3D"auto">commit 2851aafe96c1e357971f2b331fff837ead20522b <br> Author: Konstantin Belousov <kib@FreeBSD.org> <br> AuthorDate: 2024-09-28 23:17:05 +0000 <br> Commit: Konstantin Belousov <kib@FreeBSD.org> <br> CommitDate: 2024-10-10 09:55:45 +0000</p> <p dir=3D"auto"> mlx5 ipsec_offload: ensure that driver does not deref= erence dead sahindex</p> <p dir=3D"auto"> Take the sahtree rlock and check for the DEAD SA stat= e before validating <br> and filling the SA xfrm attributes.</p> <p dir=3D"auto"> Sponsored by: NVidia networking <br> ---</p> </blockquote></div> <div class=3D"markdown" style=3D"white-space: normal;"> <p dir=3D"auto">I believe this commit breaks LINT-NOIP builds:</p> <pre style=3D"margin-left: 15px; margin-right: 15px; padding: 5px; border= : thin solid gray; overflow-x: auto; max-width: 90vw; background-color: #= E4E4E4;"><code style=3D"padding: 0 0.25em; background-color: #E4E4E4;">--= - kernel --- linking kernel ld: error: undefined symbol: ipsec_sahtree_rlock >>> referenced by mlx5_ipsec.c >>> mlx5_ipsec.o:(mlx5e_if_sa_newkey_onedir) >>> referenced by mlx5_ipsec.c >>> mlx5_ipsec.o:(mlx5e_if_sa_newkey_onedir) ld: error: undefined symbol: ipsec_sahtree_runlock >>> referenced by mlx5_ipsec.c >>> mlx5_ipsec.o:(mlx5e_if_sa_newkey_onedir) >>> referenced by mlx5_ipsec.c >>> mlx5_ipsec.o:(mlx5e_if_sa_newkey_onedir) >>> referenced by mlx5_ipsec.c >>> mlx5_ipsec.o:(mlx5e_if_sa_newkey_onedir) *** [kernel] Error code 1 </code></pre> <p dir=3D"auto">Best regards,<br> Kristof</p> </div> </div> </body> </html> --=_MailMate_25357E85-7349-467E-8FED-1670C4B2178C_=--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?9DD418B6-E346-4ED8-9212-6BD555FB2431>