| Message ID | 157252457993.20908.6741530756091861014.stgit@devnote2 (mailing list archive) |
|---|---|
| State | Mainlined |
| Commit | 5b06eeae52c02dd0d9bc8488275a1207d410870b |
| Headers | show |
| Series | [BUGFIX] selftests: breakpoints: Fix a typo of function name | expand |
On Thu, Oct 31, 2019 at 09:23:00PM +0900, Masami Hiramatsu wrote: > Since commit 5821ba969511 ("selftests: Add test plan API to kselftest.h > and adjust callers") accidentally introduced 'a' typo in the front of > run_test() function, breakpoint_test_arm64.c became not able to be > compiled. > > Remove the 'a' from arun_test(). > > Fixes: 5821ba969511 ("selftests: Add test plan API to kselftest.h and adjust callers") > Reported-by: Jun Takahashi <takahashi.jun_s@aa.socionext.com> > Signed-off-by: Masami Hiramatsu <mhiramat@kernel.org> > Cc: Kees Cook <keescook@chromium.org> Argh, apologies! Yes, thank you for catching this. Reviewed-by: Kees Cook <keescook@chromium.org> -Kees > --- > .../selftests/breakpoints/breakpoint_test_arm64.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tools/testing/selftests/breakpoints/breakpoint_test_arm64.c b/tools/testing/selftests/breakpoints/breakpoint_test_arm64.c > index 58ed5eeab709..ad41ea69001b 100644 > --- a/tools/testing/selftests/breakpoints/breakpoint_test_arm64.c > +++ b/tools/testing/selftests/breakpoints/breakpoint_test_arm64.c > @@ -109,7 +109,7 @@ static bool set_watchpoint(pid_t pid, int size, int wp) > return false; > } > > -static bool arun_test(int wr_size, int wp_size, int wr, int wp) > +static bool run_test(int wr_size, int wp_size, int wr, int wp) > { > int status; > siginfo_t siginfo; >
diff --git a/tools/testing/selftests/breakpoints/breakpoint_test_arm64.c b/tools/testing/selftests/breakpoints/breakpoint_test_arm64.c index 58ed5eeab709..ad41ea69001b 100644 --- a/tools/testing/selftests/breakpoints/breakpoint_test_arm64.c +++ b/tools/testing/selftests/breakpoints/breakpoint_test_arm64.c @@ -109,7 +109,7 @@ static bool set_watchpoint(pid_t pid, int size, int wp) return false; } -static bool arun_test(int wr_size, int wp_size, int wr, int wp) +static bool run_test(int wr_size, int wp_size, int wr, int wp) { int status; siginfo_t siginfo;
Since commit 5821ba969511 ("selftests: Add test plan API to kselftest.h and adjust callers") accidentally introduced 'a' typo in the front of run_test() function, breakpoint_test_arm64.c became not able to be compiled. Remove the 'a' from arun_test(). Fixes: 5821ba969511 ("selftests: Add test plan API to kselftest.h and adjust callers") Reported-by: Jun Takahashi <takahashi.jun_s@aa.socionext.com> Signed-off-by: Masami Hiramatsu <mhiramat@kernel.org> Cc: Kees Cook <keescook@chromium.org> --- .../selftests/breakpoints/breakpoint_test_arm64.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)