tree 368f23716846afcc8f591f42626106e64ce731ae
parent fd4b4207f1f75c6e71a4e1a9d57f8abc08b300cd
author Slavin Liu <slavin452@gmail.com> 1757613479 +0800
committer Cloud Image Release <cloud-image-release@prod.google.com> 1759878487 -0700

ipvs: Defer ip_vs_ftp unregister during netns cleanup

On the netns cleanup path, __ip_vs_ftp_exit() may unregister ip_vs_ftp
before connections with valid cp->app pointers are flushed, leading to a
use-after-free.

Fix this by introducing a global `exiting_module` flag, set to true in
ip_vs_ftp_exit() before unregistering the pernet subsystem. In
__ip_vs_ftp_exit(), skip ip_vs_ftp unregister if called during netns
cleanup (when exiting_module is false) and defer it to
__ip_vs_cleanup_batch(), which unregisters all apps after all connections
are flushed. If called during module exit, unregister ip_vs_ftp
immediately.

BUG=b/449335583
TEST=presubmit,validation
RELEASE_NOTE=Fixed KCTF-134121b in the Linux kernel.

cos-patch: security-high
Fixes: 61b1ab4583e2 ("IPVS: netns, add basic init per netns.")
Suggested-by: Julian Anastasov <ja@ssi.bg>
Change-Id: If409c0fd0c23bc70ba59795f95301a0aff89825a
Signed-off-by: Slavin Liu <slavin452@gmail.com>
Signed-off-by: Julian Anastasov <ja@ssi.bg>
Signed-off-by: Florian Westphal <fw@strlen.de>
Reviewed-on: https://cos-review.googlesource.com/c/third_party/kernel/+/113623
Tested-by: Cusky Presubmit Bot <presubmit@cos-infra-prod.iam.gserviceaccount.com>
Reviewed-by: Chenglong Tang <chenglongtang@google.com>
Main-Branch-Verified: Cusky Presubmit Bot <presubmit@cos-infra-prod.iam.gserviceaccount.com>
Reviewed-on: https://cos-review.googlesource.com/c/third_party/kernel/+/113666