Unverified Commit 5f264976 authored by openeuler-ci-bot's avatar openeuler-ci-bot Committed by Gitee
Browse files

!12163 bpf: Take return from set_memory_rox() into account with bpf_jit_binary_lock_ro() 

Merge Pull Request from: @ci-robot 
 
PR sync from: Yuan Can <yuancan@huawei.com>
https://mailweb.openeuler.org/hyperkitty/list/kernel@openeuler.org/message/35BQ6CTI2BXA2757TZTRDO44HSWOFBRG/ 
 
https://gitee.com/src-openeuler/kernel/issues/IAGEOD 
 
Link:https://gitee.com/openeuler/kernel/pulls/12163

 

Reviewed-by: default avatarYang Yingliang <yangyingliang@huawei.com>
Signed-off-by: default avatarYang Yingliang <yangyingliang@huawei.com>
parents 7bb9435b 3462233d

arch/arm/net/bpf_jit_32.c

+12 −13
Original line number Diff line number Diff line
@@ -2003,28 +2003,21 @@ struct bpf_prog *bpf_int_jit_compile(struct bpf_prog *prog) 
	/* If building the body of the JITed code fails somehow,

	 * we fall back to the interpretation.

	 */

	if (build_body(&ctx) < 0) {

		image_ptr = NULL;

		bpf_jit_binary_free(header);

		prog = orig_prog;

		goto out_imms;

	}

	if (build_body(&ctx) < 0)

		goto out_free;

	build_epilogue(&ctx);



	/* 3.) Extra pass to validate JITed Code */

	if (validate_code(&ctx)) {

		image_ptr = NULL;

		bpf_jit_binary_free(header);

		prog = orig_prog;

		goto out_imms;

	}

	if (validate_code(&ctx))

		goto out_free;

	flush_icache_range((u32)header, (u32)(ctx.target + ctx.idx));



	if (bpf_jit_enable > 1)

		/* there are 2 passes here */

		bpf_jit_dump(prog->len, image_size, 2, ctx.target);



	bpf_jit_binary_lock_ro(header);

	if (bpf_jit_binary_lock_ro(header))

		goto out_free;

	prog->bpf_func = (void *)ctx.target;

	prog->jited = 1;

	prog->jited_len = image_size;
@@ -2041,5 +2034,11 @@ struct bpf_prog *bpf_int_jit_compile(struct bpf_prog *prog) 
		bpf_jit_prog_release_other(prog, prog == orig_prog ?

					   tmp : orig_prog);

	return prog;



out_free:

	image_ptr = NULL;

	bpf_jit_binary_free(header);

	prog = orig_prog;

	goto out_imms;

}

arch/arm64/net/bpf_jit_comp.c

+8 −1
Original line number Diff line number Diff line
@@ -1425,7 +1425,14 @@ struct bpf_prog *bpf_int_jit_compile(struct bpf_prog *prog) 
			prog->jited_len = 0;

			goto out_off;

		}

		bpf_jit_binary_lock_ro(header);

		if (bpf_jit_binary_lock_ro(header)) {

			bpf_jit_binary_free(header);

			prog = orig_prog;

			prog->bpf_func = NULL;

			prog->jited = 0;

			prog->jited_len = 0;

			goto out_off;

		}

	} else {

		jit_data->ctx = ctx;

		jit_data->image = image_ptr;

arch/mips/net/ebpf_jit.c

+2 −1
Original line number Diff line number Diff line
@@ -1912,7 +1912,8 @@ struct bpf_prog *bpf_int_jit_compile(struct bpf_prog *prog) 
		/* Dump JIT code */

		bpf_jit_dump(prog->len, image_size, 2, ctx.target);



	bpf_jit_binary_lock_ro(header);

	if (bpf_jit_binary_lock_ro(header))

		goto out_err;

	prog->bpf_func = (void *)ctx.target;

	prog->jited = 1;

	prog->jited_len = image_size;

arch/riscv/net/bpf_jit_core.c

+8 −1
Original line number Diff line number Diff line
@@ -163,7 +163,14 @@ struct bpf_prog *bpf_int_jit_compile(struct bpf_prog *prog) 
	bpf_flush_icache(jit_data->header, ctx->insns + ctx->ninsns);



	if (!prog->is_func || extra_pass) {

		bpf_jit_binary_lock_ro(jit_data->header);

		if (bpf_jit_binary_lock_ro(jit_data->header)) {

			bpf_jit_binary_free(jit_data->header);

			prog = orig_prog;

			prog->bpf_func = NULL;

			prog->jited = 0;

			prog->jited_len = 0;

			goto out_offset;

		}

		for (i = 0; i < prog->len; i++)

			ctx->offset[i] = ninsns_rvoff(ctx->offset[i]);

		bpf_prog_fill_jited_linfo(prog, ctx->offset);

arch/s390/net/bpf_jit_comp.c

+5 −1
Original line number Diff line number Diff line
@@ -1807,7 +1807,11 @@ struct bpf_prog *bpf_int_jit_compile(struct bpf_prog *fp) 
		print_fn_code(jit.prg_buf, jit.size_prg);

	}

	if (!fp->is_func || extra_pass) {

		bpf_jit_binary_lock_ro(header);

		if (bpf_jit_binary_lock_ro(header)) {

			bpf_jit_binary_free(header);

			fp = orig_fp;

			goto free_addrs;

		}

	} else {

		jit_data->header = header;

		jit_data->ctx = jit;