mirror of
https://github.com/torvalds/linux.git
synced 2024-11-10 14:11:52 +00:00
[PATCH] devpts: use lib/parser.c for parsing mount options
Item from "2.6 should fix" list. Signed-off-by: Domen Puncer <domen@coderock.org> Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com> Signed-off-by: Andrew Morton <akpm@osdl.org> Signed-off-by: Linus Torvalds <torvalds@osdl.org>
This commit is contained in:
parent
ee25e96fcd
commit
7a673c6b8f
@ -18,6 +18,7 @@
|
||||
#include <linux/mount.h>
|
||||
#include <linux/tty.h>
|
||||
#include <linux/devpts_fs.h>
|
||||
#include <linux/parser.h>
|
||||
|
||||
#define DEVPTS_SUPER_MAGIC 0x1cd1
|
||||
|
||||
@ -32,39 +33,60 @@ static struct {
|
||||
umode_t mode;
|
||||
} config = {.mode = 0600};
|
||||
|
||||
enum {
|
||||
Opt_uid, Opt_gid, Opt_mode,
|
||||
Opt_err
|
||||
};
|
||||
|
||||
static match_table_t tokens = {
|
||||
{Opt_uid, "uid=%u"},
|
||||
{Opt_gid, "gid=%u"},
|
||||
{Opt_mode, "mode=%o"},
|
||||
{Opt_err, NULL}
|
||||
};
|
||||
|
||||
static int devpts_remount(struct super_block *sb, int *flags, char *data)
|
||||
{
|
||||
int setuid = 0;
|
||||
int setgid = 0;
|
||||
uid_t uid = 0;
|
||||
gid_t gid = 0;
|
||||
umode_t mode = 0600;
|
||||
char *this_char;
|
||||
char *p;
|
||||
|
||||
this_char = NULL;
|
||||
while ((this_char = strsep(&data, ",")) != NULL) {
|
||||
int n;
|
||||
char dummy;
|
||||
if (!*this_char)
|
||||
config.setuid = 0;
|
||||
config.setgid = 0;
|
||||
config.uid = 0;
|
||||
config.gid = 0;
|
||||
config.mode = 0600;
|
||||
|
||||
while ((p = strsep(&data, ",")) != NULL) {
|
||||
substring_t args[MAX_OPT_ARGS];
|
||||
int token;
|
||||
int option;
|
||||
|
||||
if (!*p)
|
||||
continue;
|
||||
if (sscanf(this_char, "uid=%i%c", &n, &dummy) == 1) {
|
||||
setuid = 1;
|
||||
uid = n;
|
||||
} else if (sscanf(this_char, "gid=%i%c", &n, &dummy) == 1) {
|
||||
setgid = 1;
|
||||
gid = n;
|
||||
} else if (sscanf(this_char, "mode=%o%c", &n, &dummy) == 1)
|
||||
mode = n & ~S_IFMT;
|
||||
else {
|
||||
printk("devpts: called with bogus options\n");
|
||||
|
||||
token = match_token(p, tokens, args);
|
||||
switch (token) {
|
||||
case Opt_uid:
|
||||
if (match_int(&args[0], &option))
|
||||
return -EINVAL;
|
||||
config.uid = option;
|
||||
config.setuid = 1;
|
||||
break;
|
||||
case Opt_gid:
|
||||
if (match_int(&args[0], &option))
|
||||
return -EINVAL;
|
||||
config.gid = option;
|
||||
config.setgid = 1;
|
||||
break;
|
||||
case Opt_mode:
|
||||
if (match_octal(&args[0], &option))
|
||||
return -EINVAL;
|
||||
config.mode = option & ~S_IFMT;
|
||||
break;
|
||||
default:
|
||||
printk(KERN_ERR "devpts: called with bogus options\n");
|
||||
return -EINVAL;
|
||||
}
|
||||
}
|
||||
config.setuid = setuid;
|
||||
config.setgid = setgid;
|
||||
config.uid = uid;
|
||||
config.gid = gid;
|
||||
config.mode = mode;
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user