netdev - [PATCH net-next v2 0/6] vsock/test: Tests for memory leaks

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-next>] [day] [month] [year] [list]

Message-Id: <20241216-test-vsock-leaks-v2-0-55e1405742fc@rbox.co>
Date: Mon, 16 Dec 2024 13:00:56 +0100
From: Michal Luczaj <mhal@...x.co>
To: Stefano Garzarella <sgarzare@...hat.com>
Cc: netdev@...r.kernel.org, Michal Luczaj <mhal@...x.co>, 
 Luigi Leonardi <leonardi@...hat.com>
Subject: [PATCH net-next v2 0/6] vsock/test: Tests for memory leaks

Series adds tests for recently fixed memory leaks[1]:

commit d7b0ff5a8667 ("virtio/vsock: Fix accept_queue memory leak")
commit fbf7085b3ad1 ("vsock: Fix sk_error_queue memory leak")
commit 60cf6206a1f5 ("virtio/vsock: Improve MSG_ZEROCOPY error handling")

Patch 1/6 is a non-functional preparatory cleanup.
Patch 2/6 is a test suite extension that I try to smuggle in, but is
unrelated to the tests and can be safely dropped.
Patch 3/6 explains the need of kmemleak scans.
Patches 4-5-6 add the tests.

[1]: https://lore.kernel.org/netdev/20241107-vsock-mem-leaks-v2-0-4e21bfcfc818@rbox.co/

Signed-off-by: Michal Luczaj <mhal@...x.co>
---
Changes in v2:
- Introduce a vsock_test option to run a single test
- ZC completion test: rewrite, comment, describe failslab approach (Stefano)
- accept_queue test: rewrite, comment (Stefano)
- Annotate functions and commits about the need of kmemleak (Stefano)
- Add README section about kmemleak (Stefano)
- Collect R-b (Luigi, Stefano)
- Link to v1: https://lore.kernel.org/r/20241206-test-vsock-leaks-v1-0-c31e8c875797@rbox.co

---
Michal Luczaj (6):
      vsock/test: Use NSEC_PER_SEC
      vsock/test: Introduce option to run a single test
      vsock/test: Add README blurb about kmemleak usage
      vsock/test: Add test for accept_queue memory leak
      vsock/test: Add test for sk_error_queue memory leak
      vsock/test: Add test for MSG_ZEROCOPY completion memory leak

 tools/testing/vsock/README       |  15 +++
 tools/testing/vsock/util.c       |  20 ++-
 tools/testing/vsock/util.h       |   2 +
 tools/testing/vsock/vsock_test.c | 263 ++++++++++++++++++++++++++++++++++++++-
 4 files changed, 296 insertions(+), 4 deletions(-)
---
base-commit: 2c2b61d2138f472e50b5531ec0cb4a1485837e21
change-id: 20241203-test-vsock-leaks-38f9559f5636

Best regards,
-- 
Michal Luczaj <mhal@...x.co>