Commit 418a3c13 authored by Zheng Yejian's avatar Zheng Yejian
Browse files

livepatch: Fix warning C_RULE_ID_SINGLE_BRANCH_IF_AND_LOOP_BRACKET 

hulk inclusion
category: cleanup
bugzilla: https://gitee.com/openeuler/kernel/issues/I9R2TB



--------------------------------

Remove bracket in single branch of if statement.

Signed-off-by: default avatarZheng Yejian <zhengyejian1@huawei.com>
parent 2c0312d4

arch/arm/kernel/livepatch.c

+1 −2
Original line number Diff line number Diff line
@@ -75,9 +75,8 @@ bool arch_check_jump_insn(unsigned long func_addr) 
	u32 *insn = (u32*)func_addr;



	for (i = 0; i < CHECK_JUMP_RANGE; i++) {

		if (is_jump_insn(*insn)) {

		if (is_jump_insn(*insn))

			return true;

		}

		insn++;

	}

	return false;

arch/arm64/kernel/livepatch.c

+1 −2
Original line number Diff line number Diff line
@@ -68,9 +68,8 @@ bool arch_check_jump_insn(unsigned long func_addr) 
	u32 *insn = (u32*)func_addr;



	for (i = 0; i < CHECK_JUMP_RANGE; i++) {

		if (is_jump_insn(*insn)) {

		if (is_jump_insn(*insn))

			return true;

		}

		insn++;

	}

	return false;

arch/powerpc/kernel/livepatch_32.c

+1 −2
Original line number Diff line number Diff line
@@ -65,9 +65,8 @@ bool arch_check_jump_insn(unsigned long func_addr) 
	u32 *insn = (u32*)func_addr;



	for (i = 0; i < CHECK_JUMP_RANGE; i++) {

		if (is_jump_insn(*insn)) {

		if (is_jump_insn(*insn))

			return true;

		}

		insn++;

	}

	return false;

arch/powerpc/kernel/livepatch_64.c

+1 −2
Original line number Diff line number Diff line
@@ -70,9 +70,8 @@ static bool check_jump_insn(unsigned long func_addr) 
	u32 *insn = (u32*)func_addr;



	for (i = 0; i < CHECK_JUMP_RANGE; i++) {

		if (is_jump_insn(*insn)) {

		if (is_jump_insn(*insn))

			return true;

		}

		insn++;

	}

	return false;