Commit 6ff4a0fa authored by Hengqi Chen's avatar Hengqi Chen Committed by Alexei Starovoitov
Browse files

bpf, arm64: Call bpf_jit_binary_pack_finalize() in bpf_jit_free() 



The current implementation seems incorrect and does NOT match the
comment above, use bpf_jit_binary_pack_finalize() instead.

Fixes: 1dad391d ("bpf, arm64: use bpf_prog_pack for memory management")
Acked-by: default avatarPuranjay Mohan <puranjay@kernel.org>
Signed-off-by: default avatarHengqi Chen <hengqi.chen@gmail.com>
Acked-by: default avatarSong Liu <song@kernel.org>
Acked-by: default avatarPuranjay Mohan <puranjay@kernel.org>
Link: https://lore.kernel.org/r/20250916232653.101004-1-hengqi.chen@gmail.com


Signed-off-by: default avatarAlexei Starovoitov <ast@kernel.org>
parent a24a2dda

arch/arm64/net/bpf_jit_comp.c

+1 −2
Original line number Diff line number Diff line
@@ -3115,8 +3115,7 @@ void bpf_jit_free(struct bpf_prog *prog) 
		 * before freeing it.

		 */

		if (jit_data) {

			bpf_arch_text_copy(&jit_data->ro_header->size, &jit_data->header->size,

					   sizeof(jit_data->header->size));

			bpf_jit_binary_pack_finalize(jit_data->ro_header, jit_data->header);

			kfree(jit_data);

		}

		prog->bpf_func -= cfi_get_offset();