[PATCH rfc v3 bpf-next 4/9] bpf, obj: allow . char as part of the name

From: Daniel Borkmann <daniel@iogearbox.net>
To: ast@kernel.org
Cc: bpf@vger.kernel.org, netdev@vger.kernel.org, joe@wand.net.nz,
	john.fastabend@gmail.com, yhs@fb.com, andrii.nakryiko@gmail.com,
	jakub.kicinski@netronome.com, tgraf@suug.ch, lmb@cloudflare.com,
	Daniel Borkmann <daniel@iogearbox.net>
Subject: [PATCH rfc v3 bpf-next 4/9] bpf, obj: allow . char as part of the name
Date: Mon, 11 Mar 2019 22:51:20 +0100	[thread overview]
Message-ID: <20190311215125.17793-5-daniel@iogearbox.net> (raw)
In-Reply-To: <20190311215125.17793-1-daniel@iogearbox.net>

Trivial addition to allow '.' aside from '_' as "special" characters
in the object name. Used to allow for substrings in maps from loader
side such as ".bss", ".data", ".rodata", but could also be useful for
other purposes.

Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
---
 kernel/bpf/syscall.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/kernel/bpf/syscall.c b/kernel/bpf/syscall.c
index b5ba138351e1..04279747c092 100644
--- a/kernel/bpf/syscall.c
+++ b/kernel/bpf/syscall.c
@@ -456,10 +456,10 @@ static int bpf_obj_name_cpy(char *dst, const char *src)
 	const char *end = src + BPF_OBJ_NAME_LEN;
 
 	memset(dst, 0, BPF_OBJ_NAME_LEN);
-
-	/* Copy all isalnum() and '_' char */
+	/* Copy all isalnum(), '_' and '.' chars. */
 	while (src < end && *src) {
-		if (!isalnum(*src) && *src != '_')
+		if (!isalnum(*src) &&
+		    *src != '_' && *src != '.')
 			return -EINVAL;
 		*dst++ = *src++;
 	}
-- 
2.17.1

