Kernel/linux-stable
1
0
Fork 0
mirror of https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git synced 2024-12-29 09:16:33 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

samples/landlock: Fix port parsing in sandboxer

Browse Source 
If you want to specify that no port can be bind()ed, you would think
(looking quickly at both help message and code) that setting
LL_TCP_BIND="" would do it.

However the code splits on ":" then applies atoi(), which does not allow
checking for errors. Passing an empty string returns 0, which is
interpreted as "allow bind(0)", which means bind to any ephemeral port.
This bug occurs whenever passing an empty string or when leaving a
trailing/leading colon, making it impossible to completely deny bind().

To reproduce:
export LL_FS_RO="/" LL_FS_RW="" LL_TCP_BIND=""
./sandboxer strace -e bind nc -n -vvv -l -p 0
Executing the sandboxed command...
bind(3, {sa_family=AF_INET, sin_port=htons(0),
     sin_addr=inet_addr("0.0.0.0")}, 16) = 0
Listening on 0.0.0.0 37629

Use strtoull(3) instead, which allows error checking. Check that the
entire string has been parsed correctly without overflows/underflows,
but not that the __u64 (the type of struct landlock_net_port_attr.port)
is a valid __u16 port: that is already done by the kernel.

Fixes: 5e990dcef1 ("samples/landlock: Support TCP restrictions")
Signed-off-by: Matthieu Buffet <matthieu@buffet.re>
Link: https://lore.kernel.org/r/20241019151534.1400605-2-matthieu@buffet.re
Signed-off-by: Mickaël Salaün <mic@digikod.net>
This commit is contained in:
Matthieu Buffet 2024-10-19 17:15:32 +02:00 committed by Mickaël Salaün
parent dad2f20715
commit 387285530d
No known key found for this signature in database
GPG Key ID: E5E3D0E88C82F6D2
1 changed files with 30 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
samples/landlock/sandboxer.c
View File

@ -60,6 +60,25 @@ static inline int landlock_restrict_self(const int ruleset_fd,
#define ENV_SCOPED_NAME "LL_SCOPED" #define ENV_SCOPED_NAME "LL_SCOPED"
#define ENV_DELIMITER ":" #define ENV_DELIMITER ":"
static int str2num(const char *numstr, __u64 *num_dst)
{
char *endptr = NULL;
int err = 0;
__u64 num;
errno = 0;
num = strtoull(numstr, &endptr, 10);
if (errno != 0)
err = errno;
/* Was the string empty, or not entirely parsed successfully? */
else if ((*numstr == '\0') || (*endptr != '\0'))
err = EINVAL;
else
*num_dst = num;
return err;
}
static int parse_path(char *env_path, const char ***const path_list) static int parse_path(char *env_path, const char ***const path_list)
{ {
int i, num_paths = 0; int i, num_paths = 0;
@ -160,7 +179,6 @@ static int populate_ruleset_net(const char *const env_var, const int ruleset_fd,
char *env_port_name, *env_port_name_next, *strport; char *env_port_name, *env_port_name_next, *strport;
struct landlock_net_port_attr net_port = { struct landlock_net_port_attr net_port = {
.allowed_access = allowed_access, .allowed_access = allowed_access,
.port = 0,
}; };
env_port_name = getenv(env_var); env_port_name = getenv(env_var);
@ -171,7 +189,17 @@ static int populate_ruleset_net(const char *const env_var, const int ruleset_fd,
env_port_name_next = env_port_name; env_port_name_next = env_port_name;
while ((strport = strsep(&env_port_name_next, ENV_DELIMITER))) { while ((strport = strsep(&env_port_name_next, ENV_DELIMITER))) {
net_port.port = atoi(strport); __u64 port;
if (strcmp(strport, "") == 0)
continue;
if (str2num(strport, &port)) {
fprintf(stderr, "Failed to parse port at \"%s\"\n",
strport);
goto out_free_name;
}
net_port.port = port;
if (landlock_add_rule(ruleset_fd, LANDLOCK_RULE_NET_PORT, if (landlock_add_rule(ruleset_fd, LANDLOCK_RULE_NET_PORT,
&net_port, 0)) { &net_port, 0)) {
fprintf(stderr, fprintf(stderr,