Skip to content

Commit

Permalink
---
Browse files Browse the repository at this point in the history
yaml
---
r: 86943
b: refs/heads/master
c: f9c3a38
h: refs/heads/master
i:
  86941: 3a52b86
  86939: e414fed
  86935: 3336135
  86927: 1d1a625
  86911: 219931f
v: v3
  • Loading branch information
Eric Paris authored and James Morris committed Mar 5, 2008
1 parent 4ccb1a8 commit 9fa4ae7
Show file tree
Hide file tree
Showing 3 changed files with 66 additions and 3 deletions.
2 changes: 1 addition & 1 deletion [refs]
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
---
refs/heads/master: e0007529893c1c064be90bd21422ca0da4a0198e
refs/heads/master: f9c3a3802119a2d30f3e4a69aef30a81e09d0209
3 changes: 3 additions & 0 deletions trunk/fs/nfs/internal.h
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
*/

#include <linux/mount.h>
#include <linux/security.h>

struct nfs_string;

Expand Down Expand Up @@ -57,6 +58,8 @@ struct nfs_parsed_mount_data {
char *export_path;
int protocol;
} nfs_server;

struct security_mnt_opts lsm_opts;
};

/* client.c */
Expand Down
64 changes: 62 additions & 2 deletions trunk/fs/nfs/super.c
Original file line number Diff line number Diff line change
Expand Up @@ -684,15 +684,30 @@ static void nfs_parse_server_address(char *value,
static int nfs_parse_mount_options(char *raw,
struct nfs_parsed_mount_data *mnt)
{
char *p, *string;
char *p, *string, *secdata;
unsigned short port = 0;
int rc;

if (!raw) {
dfprintk(MOUNT, "NFS: mount options string was NULL.\n");
return 1;
}
dfprintk(MOUNT, "NFS: nfs mount opts='%s'\n", raw);

secdata = alloc_secdata();
if (!secdata)
goto out_nomem;

rc = security_sb_copy_data(raw, secdata);
if (rc)
goto out_security_failure;

rc = security_sb_parse_opts_str(secdata, &mnt->lsm_opts);
if (rc)
goto out_security_failure;

free_secdata(secdata);

while ((p = strsep(&raw, ",")) != NULL) {
substring_t args[MAX_OPT_ARGS];
int option, token;
Expand Down Expand Up @@ -1042,7 +1057,10 @@ static int nfs_parse_mount_options(char *raw,
out_nomem:
printk(KERN_INFO "NFS: not enough memory to parse option\n");
return 0;

out_security_failure:
free_secdata(secdata);
printk(KERN_INFO "NFS: security options invalid: %d\n", rc);
return 0;
out_unrec_vers:
printk(KERN_INFO "NFS: unrecognized NFS version number\n");
return 0;
Expand Down Expand Up @@ -1214,6 +1232,33 @@ static int nfs_validate_mount_data(void *options,
args->namlen = data->namlen;
args->bsize = data->bsize;
args->auth_flavors[0] = data->pseudoflavor;

/*
* The legacy version 6 binary mount data from userspace has a
* field used only to transport selinux information into the
* the kernel. To continue to support that functionality we
* have a touch of selinux knowledge here in the NFS code. The
* userspace code converted context=blah to just blah so we are
* converting back to the full string selinux understands.
*/
if (data->context[0]){
#ifdef CONFIG_SECURITY_SELINUX
int rc;
char *opts_str = kmalloc(sizeof(data->context) + 8, GFP_KERNEL);
if (!opts_str)
return -ENOMEM;
strcpy(opts_str, "context=");
data->context[NFS_MAX_CONTEXT_LEN] = '\0';
strcat(opts_str, &data->context[0]);
rc = security_sb_parse_opts_str(opts_str, &args->lsm_opts);
kfree(opts_str);
if (rc)
return rc;
#else
return -EINVAL;
#endif
}

break;
default: {
unsigned int len;
Expand Down Expand Up @@ -1476,6 +1521,8 @@ static int nfs_get_sb(struct file_system_type *fs_type,
};
int error;

security_init_mnt_opts(&data.lsm_opts);

/* Validate the mount data */
error = nfs_validate_mount_data(raw_data, &data, &mntfh, dev_name);
if (error < 0)
Expand Down Expand Up @@ -1515,6 +1562,10 @@ static int nfs_get_sb(struct file_system_type *fs_type,
goto error_splat_super;
}

error = security_sb_set_mnt_opts(s, &data.lsm_opts);
if (error)
goto error_splat_root;

s->s_flags |= MS_ACTIVE;
mnt->mnt_sb = s;
mnt->mnt_root = mntroot;
Expand All @@ -1523,12 +1574,15 @@ static int nfs_get_sb(struct file_system_type *fs_type,
out:
kfree(data.nfs_server.hostname);
kfree(data.mount_server.hostname);
security_free_mnt_opts(&data.lsm_opts);
return error;

out_err_nosb:
nfs_free_server(server);
goto out;

error_splat_root:
dput(mntroot);
error_splat_super:
up_write(&s->s_umount);
deactivate_super(s);
Expand Down Expand Up @@ -1608,6 +1662,9 @@ static int nfs_xdev_get_sb(struct file_system_type *fs_type, int flags,
mnt->mnt_sb = s;
mnt->mnt_root = mntroot;

/* clone any lsm security options from the parent to the new sb */
security_sb_clone_mnt_opts(data->sb, s);

dprintk("<-- nfs_xdev_get_sb() = 0\n");
return 0;

Expand Down Expand Up @@ -1850,6 +1907,8 @@ static int nfs4_get_sb(struct file_system_type *fs_type,
};
int error;

security_init_mnt_opts(&data.lsm_opts);

/* Validate the mount data */
error = nfs4_validate_mount_data(raw_data, &data, dev_name);
if (error < 0)
Expand Down Expand Up @@ -1898,6 +1957,7 @@ static int nfs4_get_sb(struct file_system_type *fs_type,
kfree(data.client_address);
kfree(data.nfs_server.export_path);
kfree(data.nfs_server.hostname);
security_free_mnt_opts(&data.lsm_opts);
return error;

out_free:
Expand Down

0 comments on commit 9fa4ae7

Please sign in to comment.