Commit 4fc00b79 authored by Yonghong Song's avatar Yonghong Song Committed by Daniel Borkmann

bpf: Add missing newline characters in verifier error messages

Newline characters are added in two verifier error messages,
refactored in Commit afbf21dc ("bpf: Support readonly/readwrite
buffers in verifier"). This way, they do not mix with
messages afterwards.
Signed-off-by: default avatarYonghong Song <yhs@fb.com>
Signed-off-by: default avatarDaniel Borkmann <daniel@iogearbox.net>
Acked-by: default avatarSong Liu <songliubraving@fb.com>
Link: https://lore.kernel.org/bpf/20200728221801.1090349-1-yhs@fb.com
parent 80083428
...@@ -3069,7 +3069,7 @@ static int __check_buffer_access(struct bpf_verifier_env *env, ...@@ -3069,7 +3069,7 @@ static int __check_buffer_access(struct bpf_verifier_env *env,
{ {
if (off < 0) { if (off < 0) {
verbose(env, verbose(env,
"R%d invalid %s buffer access: off=%d, size=%d", "R%d invalid %s buffer access: off=%d, size=%d\n",
regno, buf_info, off, size); regno, buf_info, off, size);
return -EACCES; return -EACCES;
} }
...@@ -3078,7 +3078,7 @@ static int __check_buffer_access(struct bpf_verifier_env *env, ...@@ -3078,7 +3078,7 @@ static int __check_buffer_access(struct bpf_verifier_env *env,
tnum_strn(tn_buf, sizeof(tn_buf), reg->var_off); tnum_strn(tn_buf, sizeof(tn_buf), reg->var_off);
verbose(env, verbose(env,
"R%d invalid variable buffer offset: off=%d, var_off=%s", "R%d invalid variable buffer offset: off=%d, var_off=%s\n",
regno, off, tn_buf); regno, off, tn_buf);
return -EACCES; return -EACCES;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment