On Thu, Sep 29, 2022 at 11:01:33AM +0200, Magnus Karlsson wrote:
> From: Magnus Karlsson <magnus.karlsson@xxxxxxxxx>
>
> 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@xxxxxxxxx>
> ---
> 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)
> - free(ifobj_tx->umem);
> ifobject_delete(ifobj_tx);
> ifobject_delete(ifobj_rx);
So basically we free this inside ifobject_delete().
Acked-by: Maciej Fijalkowski <maciej.fijalkowski@xxxxxxxxx>
>
>
> base-commit: 8526f0d6135f77451566463ace6f0fb8b72cedaa
> --
> 2.34.1
>
