On 9/29/22 3:44 AM, Maciej Fijalkowski wrote:

On Thu, Sep 29, 2022 at 11:01:33AM +0200, Magnus Karlsson wrote:

quoted

From: Magnus Karlsson <magnus.karlsson@intel.com>

Fix a double free at exit of the test suite.

Fixes: a693ff3ed561 ("selftests/xsk: Add support for executing tests on physical device")
Signed-off-by: Magnus Karlsson <magnus.karlsson@intel.com>
---
  tools/testing/selftests/bpf/xskxceiver.c | 3 ---
  1 file changed, 3 deletions(-)

diff --git a/tools/testing/selftests/bpf/xskxceiver.c b/tools/testing/selftests/bpf/xskxceiver.c
index ef33309bbe49..d1a5f3218c34 100644
--- a/tools/testing/selftests/bpf/xskxceiver.c
+++ b/tools/testing/selftests/bpf/xskxceiver.c

@@ -1953,9 +1953,6 @@ int main(int argc, char **argv)
  
  	pkt_stream_delete(tx_pkt_stream_default);
  	pkt_stream_delete(rx_pkt_stream_default);
-	free(ifobj_rx->umem);
-	if (!ifobj_tx->shared_umem)

shared_umem means ifobj_rx->umem and ifobj_tx->umem are the same?  No special 
handling is needed and ifobject_delete() will handle it?

quoted

-		free(ifobj_tx->umem);
  	ifobject_delete(ifobj_tx);
  	ifobject_delete(ifobj_rx);

So basically we free this inside ifobject_delete().