session: Fix crash when the UCI option 'password' or 'username' is missing
[project/rpcd.git] / file.c
diff --git a/file.c b/file.c
index 23deb73e2470f6ccbe3f074db79a9930e200a324..1e5b2f4106f8fd3456e79eac8d9bfa83446d456d 100644 (file)
--- a/file.c
+++ b/file.c
@@ -17,6 +17,8 @@
  * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
  */
 
+#define _GNU_SOURCE
+
 #include <fcntl.h>
 #include <errno.h>
 #include <unistd.h>
 #include <rpcd/plugin.h>
 
 /* limit of sys & proc files */
-#define RPC_FILE_MIN_SIZE              (128)
+#define RPC_FILE_MIN_SIZE              (4096)
 
 /* limit of regular files and command output data */
 #define RPC_FILE_MAX_SIZE              (4096 * 64)
 
+/* limit of command line length for exec acl checks */
+#define RPC_CMDLINE_MAX_SIZE   (1024)
+
 #define ustream_for_each_read_buffer(stream, ptr, len) \
        for (ptr = ustream_get_read_buf(stream, &len);     \
             ptr != NULL && len > 0;                       \
@@ -68,25 +73,33 @@ struct rpc_file_exec_context {
 
 
 static struct blob_buf buf;
+static char *canonpath;
+static char cmdstr[RPC_CMDLINE_MAX_SIZE];
 
 enum {
        RPC_F_R_PATH,
+       RPC_F_R_SESSION,
        __RPC_F_R_MAX,
 };
 
-static const struct blobmsg_policy rpc_file_r_policy[__RPC_F_R_MAX] = {
-       [RPC_F_R_PATH] = { .name = "path", .type = BLOBMSG_TYPE_STRING },
+static const struct blobmsg_policy rpc_file_R_policy[__RPC_F_R_MAX] = {
+       [RPC_F_R_PATH]    = { .name = "path", .type = BLOBMSG_TYPE_STRING },
+       [RPC_F_R_SESSION] = { .name = "ubus_rpc_session",
+                             .type = BLOBMSG_TYPE_STRING },
 };
 
 enum {
        RPC_F_RB_PATH,
        RPC_F_RB_BASE64,
+       RPC_F_RB_SESSION,
        __RPC_F_RB_MAX,
 };
 
-static const struct blobmsg_policy rpc_file_rb_policy[__RPC_F_RB_MAX] = {
-       [RPC_F_RB_PATH]   = { .name = "path",   .type = BLOBMSG_TYPE_STRING },
-       [RPC_F_RB_BASE64] = { .name = "base64", .type = BLOBMSG_TYPE_BOOL   },
+static const struct blobmsg_policy rpc_file_RB_policy[__RPC_F_RB_MAX] = {
+       [RPC_F_RB_PATH]    = { .name = "path",   .type = BLOBMSG_TYPE_STRING },
+       [RPC_F_RB_BASE64]  = { .name = "base64", .type = BLOBMSG_TYPE_BOOL   },
+       [RPC_F_RB_SESSION] = { .name = "ubus_rpc_session",
+                              .type = BLOBMSG_TYPE_STRING },
 };
 
 enum {
@@ -95,28 +108,34 @@ enum {
        RPC_F_RW_APPEND,
        RPC_F_RW_MODE,
        RPC_F_RW_BASE64,
+       RPC_F_RW_SESSION,
        __RPC_F_RW_MAX,
 };
 
-static const struct blobmsg_policy rpc_file_rw_policy[__RPC_F_RW_MAX] = {
-       [RPC_F_RW_PATH]   = { .name = "path",   .type = BLOBMSG_TYPE_STRING },
-       [RPC_F_RW_DATA]   = { .name = "data",   .type = BLOBMSG_TYPE_STRING },
-       [RPC_F_RW_APPEND] = { .name = "append", .type = BLOBMSG_TYPE_BOOL  },
-       [RPC_F_RW_MODE]   = { .name = "mode",   .type = BLOBMSG_TYPE_INT32  },
-       [RPC_F_RW_BASE64] = { .name = "base64", .type = BLOBMSG_TYPE_BOOL   },
+static const struct blobmsg_policy rpc_file_RW_policy[__RPC_F_RW_MAX] = {
+       [RPC_F_RW_PATH]    = { .name = "path",   .type = BLOBMSG_TYPE_STRING },
+       [RPC_F_RW_DATA]    = { .name = "data",   .type = BLOBMSG_TYPE_STRING },
+       [RPC_F_RW_APPEND]  = { .name = "append", .type = BLOBMSG_TYPE_BOOL  },
+       [RPC_F_RW_MODE]    = { .name = "mode",   .type = BLOBMSG_TYPE_INT32  },
+       [RPC_F_RW_BASE64]  = { .name = "base64", .type = BLOBMSG_TYPE_BOOL   },
+       [RPC_F_RW_SESSION] = { .name = "ubus_rpc_session",
+                              .type = BLOBMSG_TYPE_STRING },
 };
 
 enum {
        RPC_E_CMD,
        RPC_E_PARM,
        RPC_E_ENV,
+       RPC_E_SESSION,
        __RPC_E_MAX,
 };
 
 static const struct blobmsg_policy rpc_exec_policy[__RPC_E_MAX] = {
-       [RPC_E_CMD]  = { .name = "command", .type = BLOBMSG_TYPE_STRING },
-       [RPC_E_PARM] = { .name = "params",  .type = BLOBMSG_TYPE_ARRAY  },
-       [RPC_E_ENV]  = { .name = "env",     .type = BLOBMSG_TYPE_TABLE  },
+       [RPC_E_CMD]     = { .name = "command", .type = BLOBMSG_TYPE_STRING },
+       [RPC_E_PARM]    = { .name = "params",  .type = BLOBMSG_TYPE_ARRAY  },
+       [RPC_E_ENV]     = { .name = "env",     .type = BLOBMSG_TYPE_TABLE  },
+       [RPC_E_SESSION] = { .name = "ubus_rpc_session",
+                           .type = BLOBMSG_TYPE_STRING },
 };
 
 static const char *d_types[] = {
@@ -153,33 +172,125 @@ rpc_errno_status(void)
        }
 }
 
+static bool
+rpc_file_access(const struct blob_attr *sid,
+                const char *path, const char *perm)
+{
+       if (!sid)
+               return true;
+
+       return ops->session_access(blobmsg_data(sid), "file", path, perm);
+}
+
+static char *
+rpc_canonicalize_path(const char *path)
+{
+       char *cp;
+       const char *p;
+
+       if (path == NULL || *path == '\0')
+               return NULL;
+
+       if (canonpath != NULL)
+               free(canonpath);
+
+       canonpath = strdup(path);
+
+       if (canonpath == NULL)
+               return NULL;
+
+       /* normalize */
+       for (cp = canonpath, p = path; *p != '\0'; ) {
+               if (*p != '/')
+                       goto next;
+
+               /* skip repeating / */
+               if (p[1] == '/') {
+                       p++;
+                       continue;
+               }
+
+               /* /./ or /../ */
+               if (p[1] == '.') {
+                       /* skip /./ */
+                       if ((p[2] == '\0') || (p[2] == '/')) {
+                               p += 2;
+                               continue;
+                       }
+
+                       /* collapse /x/../ */
+                       if ((p[2] == '.') && ((p[3] == '\0') || (p[3] == '/'))) {
+                               while ((cp > canonpath) && (*--cp != '/'))
+                                       ;
+
+                               p += 3;
+                               continue;
+                       }
+               }
+
+next:
+               *cp++ = *p++;
+       }
+
+       /* remove trailing slash if not root / */
+       if ((cp > canonpath + 1) && (cp[-1] == '/'))
+               cp--;
+       else if (cp == canonpath)
+               *cp++ = '/';
+
+       *cp = '\0';
+
+       return canonpath;
+}
+
 static struct blob_attr **
-rpc_check_path(struct blob_attr *msg, char **path, struct stat *s)
+__rpc_check_path(const struct blobmsg_policy *policy, size_t policy_len,
+                 int policy_path_idx, int policy_sid_idx, const char *perm,
+                 struct blob_attr *msg, char **path, struct stat *s)
 {
-       static struct blob_attr *tb[__RPC_F_R_MAX];
+       static struct blob_attr *tb[__RPC_F_RW_MAX]; /* largest _MAX constant */
 
-       blobmsg_parse(rpc_file_r_policy, __RPC_F_R_MAX, tb, blob_data(msg), blob_len(msg));
+       blobmsg_parse(policy, policy_len, tb, blob_data(msg), blob_len(msg));
 
-       if (!tb[RPC_F_R_PATH])
+       if (!tb[policy_path_idx])
        {
                errno = EINVAL;
                return NULL;
        }
 
-       *path = blobmsg_data(tb[RPC_F_R_PATH]);
+       *path = rpc_canonicalize_path(blobmsg_get_string(tb[policy_path_idx]));
 
-       if (stat(*path, s))
+       if (*path == NULL)
+       {
+               errno = ENOMEM;
+               return NULL;
+       }
+
+       if (!rpc_file_access(tb[policy_sid_idx], *path, perm))
+       {
+               errno = EACCES;
+               return NULL;
+       }
+
+       if (s != NULL && stat(*path, s) != 0)
                return NULL;
 
        return tb;
 }
 
+#define rpc_check_path(msg, policy_selector, perm, path, s) \
+       __rpc_check_path(rpc_file_ ## policy_selector ## _policy, \
+               ARRAY_SIZE(rpc_file_ ## policy_selector ## _policy), \
+               RPC_F_ ## policy_selector ## _PATH, \
+               RPC_F_ ## policy_selector ## _SESSION, \
+               perm, msg, path, s)
+
 static int
 rpc_file_read(struct ubus_context *ctx, struct ubus_object *obj,
               struct ubus_request_data *req, const char *method,
               struct blob_attr *msg)
 {
-       static struct blob_attr *tb[__RPC_F_RB_MAX];
+       struct blob_attr **tb;
        bool base64 = false;
        int fd, rv;
        ssize_t len;
@@ -187,14 +298,9 @@ rpc_file_read(struct ubus_context *ctx, struct ubus_object *obj,
        struct stat s;
        char *wbuf;
 
-       blobmsg_parse(rpc_file_rb_policy, __RPC_F_RB_MAX, tb, blob_data(msg), blob_len(msg));
+       tb = rpc_check_path(msg, RB, "read", &path, &s);
 
-       if (!tb[RPC_F_RB_PATH])
-               return rpc_errno_status();
-
-       path = blobmsg_data(tb[RPC_F_RB_PATH]);
-
-       if (stat(path, &s))
+       if (tb == NULL)
                return rpc_errno_status();
 
        if (s.st_size >= RPC_FILE_MAX_SIZE)
@@ -265,17 +371,20 @@ rpc_file_write(struct ubus_context *ctx, struct ubus_object *obj,
                struct ubus_request_data *req, const char *method,
                struct blob_attr *msg)
 {
-       struct blob_attr *tb[__RPC_F_RW_MAX];
+       struct blob_attr **tb;
        int append = O_TRUNC;
        mode_t prev_mode, mode = 0666;
        int fd, rv = 0;
+       char *path = NULL;
        void *data = NULL;
        ssize_t data_len = 0;
 
-       blobmsg_parse(rpc_file_rw_policy, __RPC_F_RW_MAX, tb,
-                     blob_data(msg), blob_len(msg));
+       tb = rpc_check_path(msg, RW, "write", &path, NULL);
 
-       if (!tb[RPC_F_RW_PATH] || !tb[RPC_F_RW_DATA])
+       if (tb == NULL)
+               return rpc_errno_status();
+
+       if (!tb[RPC_F_RW_DATA])
                return UBUS_STATUS_INVALID_ARGUMENT;
 
        data = blobmsg_data(tb[RPC_F_RW_DATA]);
@@ -288,7 +397,7 @@ rpc_file_write(struct ubus_context *ctx, struct ubus_object *obj,
                mode = blobmsg_get_u32(tb[RPC_F_RW_MODE]);
 
        prev_mode = umask(0);
-       fd = open(blobmsg_data(tb[RPC_F_RW_PATH]), O_CREAT | O_WRONLY | append, mode);
+       fd = open(path, O_CREAT | O_WRONLY | append, mode);
        umask(prev_mode);
        if (fd < 0)
                return rpc_errno_status();
@@ -330,7 +439,7 @@ rpc_file_md5(struct ubus_context *ctx, struct ubus_object *obj,
        uint8_t md5[16];
        char *wbuf;
 
-       if (!rpc_check_path(msg, &path, &s))
+       if (!rpc_check_path(msg, R, "read", &path, &s))
                return rpc_errno_status();
 
        if (!S_ISREG(s.st_mode))
@@ -352,6 +461,31 @@ rpc_file_md5(struct ubus_context *ctx, struct ubus_object *obj,
        return UBUS_STATUS_OK;
 }
 
+static void
+_rpc_file_add_stat(struct stat *s)
+{
+       int type;
+
+       type = S_ISREG(s->st_mode) ? DT_REG :
+               S_ISDIR(s->st_mode) ? DT_DIR :
+                S_ISCHR(s->st_mode) ? DT_CHR :
+                 S_ISBLK(s->st_mode) ? DT_BLK :
+                  S_ISFIFO(s->st_mode) ? DT_FIFO :
+                   S_ISLNK(s->st_mode) ? DT_LNK :
+                    S_ISSOCK(s->st_mode) ? DT_SOCK :
+                     DT_UNKNOWN;
+
+       blobmsg_add_string(&buf, "type", d_types[type]);
+       blobmsg_add_u32(&buf, "size",  s->st_size);
+       blobmsg_add_u32(&buf, "mode",  s->st_mode);
+       blobmsg_add_u32(&buf, "atime", s->st_atime);
+       blobmsg_add_u32(&buf, "mtime", s->st_mtime);
+       blobmsg_add_u32(&buf, "ctime", s->st_ctime);
+       blobmsg_add_u32(&buf, "inode", s->st_ino);
+       blobmsg_add_u32(&buf, "uid",   s->st_uid);
+       blobmsg_add_u32(&buf, "gid",   s->st_gid);
+}
+
 static int
 rpc_file_list(struct ubus_context *ctx, struct ubus_object *obj,
               struct ubus_request_data *req, const char *method,
@@ -359,11 +493,11 @@ rpc_file_list(struct ubus_context *ctx, struct ubus_object *obj,
 {
        DIR *fd;
        void *c, *d;
-       char *path;
        struct stat s;
        struct dirent *e;
+       char *path, *entrypath;
 
-       if (!rpc_check_path(msg, &path, &s))
+       if (!rpc_check_path(msg, R, "list", &path, NULL))
                return rpc_errno_status();
 
        if ((fd = opendir(path)) == NULL)
@@ -377,10 +511,18 @@ rpc_file_list(struct ubus_context *ctx, struct ubus_object *obj,
                if (!strcmp(e->d_name, ".") || !strcmp(e->d_name, ".."))
                        continue;
 
-               d = blobmsg_open_table(&buf, NULL);
-               blobmsg_add_string(&buf, "name", e->d_name);
-               blobmsg_add_string(&buf, "type", d_types[e->d_type]);
-               blobmsg_close_table(&buf, d);
+               if (asprintf(&entrypath, "%s/%s", path, e->d_name) < 0)
+                       continue;
+
+               if (!stat(entrypath, &s))
+               {
+                       d = blobmsg_open_table(&buf, NULL);
+                       blobmsg_add_string(&buf, "name", e->d_name);
+                       _rpc_file_add_stat(&s);
+                       blobmsg_close_table(&buf, d);
+               }
+
+               free(entrypath);
        }
 
        closedir(fd);
@@ -397,34 +539,16 @@ rpc_file_stat(struct ubus_context *ctx, struct ubus_object *obj,
               struct ubus_request_data *req, const char *method,
               struct blob_attr *msg)
 {
-       int type;
        char *path;
        struct stat s;
 
-       if (!rpc_check_path(msg, &path, &s))
+       if (!rpc_check_path(msg, R, "list", &path, &s))
                return rpc_errno_status();
 
        blob_buf_init(&buf, 0);
 
-       type = S_ISREG(s.st_mode) ? DT_REG :
-               S_ISDIR(s.st_mode) ? DT_DIR :
-                S_ISCHR(s.st_mode) ? DT_CHR :
-                 S_ISBLK(s.st_mode) ? DT_BLK :
-                  S_ISFIFO(s.st_mode) ? DT_FIFO :
-                   S_ISLNK(s.st_mode) ? DT_LNK :
-                    S_ISSOCK(s.st_mode) ? DT_SOCK :
-                     DT_UNKNOWN;
-
        blobmsg_add_string(&buf, "path", path);
-       blobmsg_add_string(&buf, "type", d_types[type]);
-       blobmsg_add_u32(&buf, "size",  s.st_size);
-       blobmsg_add_u32(&buf, "mode",  s.st_mode);
-       blobmsg_add_u32(&buf, "atime", s.st_atime);
-       blobmsg_add_u32(&buf, "mtime", s.st_mtime);
-       blobmsg_add_u32(&buf, "ctime", s.st_ctime);
-       blobmsg_add_u32(&buf, "inode", s.st_ino);
-       blobmsg_add_u32(&buf, "uid",   s.st_uid);
-       blobmsg_add_u32(&buf, "gid",   s.st_gid);
+       _rpc_file_add_stat(&s);
 
        ubus_send_reply(ctx, req, buf.head);
        blob_buf_free(&buf);
@@ -432,6 +556,75 @@ rpc_file_stat(struct ubus_context *ctx, struct ubus_object *obj,
        return 0;
 }
 
+static int
+rpc_file_remove_recursive(const char *path);
+
+static int
+rpc_file_remove_recursive(const char *path)
+{
+       DIR *fd;
+       int err = 0;
+       struct stat s;
+       struct dirent *e;
+       char *entrypath;
+
+       if ((fd = opendir(path)) == NULL)
+               return rpc_errno_status();
+
+       for (e = readdir(fd); e != NULL && err == 0; e = readdir(fd))
+       {
+               if (!strcmp(e->d_name, ".") || !strcmp(e->d_name, ".."))
+                       continue;
+
+               if (asprintf(&entrypath, "%s/%s", path, e->d_name) >= 0)
+               {
+                       if (!lstat(entrypath, &s))
+                       {
+                               if (S_ISDIR(s.st_mode))
+                                       err = rpc_file_remove_recursive(entrypath);
+                               else if (unlink(entrypath))
+                                       err = rpc_errno_status();
+                       }
+
+                       free(entrypath);
+               }
+               else
+               {
+                       err = UBUS_STATUS_UNKNOWN_ERROR;
+               }
+       }
+
+       closedir(fd);
+
+       if (!err && rmdir(path))
+               return rpc_errno_status();
+
+       return err;
+}
+
+static int
+rpc_file_remove(struct ubus_context *ctx, struct ubus_object *obj,
+                struct ubus_request_data *req, const char *method,
+                struct blob_attr *msg)
+{
+       struct stat s;
+       char *path = NULL;
+
+       if (!rpc_check_path(msg, R, "write", &path, NULL))
+               return rpc_errno_status();
+
+       if (lstat(path, &s))
+               return rpc_errno_status();
+
+       if (S_ISDIR(s.st_mode))
+               return rpc_file_remove_recursive(path);
+
+       if (unlink(path))
+               return rpc_errno_status();
+
+       return 0;
+}
+
 static const char *
 rpc_file_exec_lookup(const char *cmd)
 {
@@ -590,8 +783,15 @@ rpc_file_exec_epipe_state_cb(struct ustream *s)
                rpc_file_exec_reply(c, UBUS_STATUS_OK);
 }
 
+static void
+rpc_fdclose(int fd)
+{
+       if (fd > 2)
+               close(fd);
+}
+
 static int
-rpc_file_exec_run(const char *cmd,
+rpc_file_exec_run(const char *cmd, const struct blob_attr *sid,
                   const struct blob_attr *arg, const struct blob_attr *env,
                   struct ubus_context *ctx, struct ubus_request_data *req)
 {
@@ -605,27 +805,63 @@ rpc_file_exec_run(const char *cmd,
        struct blob_attr *cur;
 
        uint8_t arglen;
-       char **args, **tmp;
+       char *executable, **args, **tmp, *p;
 
        struct rpc_file_exec_context *c;
 
+       if (sid && env)
+               return UBUS_STATUS_PERMISSION_DENIED;
+
        cmd = rpc_file_exec_lookup(cmd);
 
        if (!cmd)
                return UBUS_STATUS_NOT_FOUND;
 
+       executable = rpc_canonicalize_path(cmd);
+
+       if (executable == NULL)
+               return UBUS_STATUS_UNKNOWN_ERROR;
+
+       if (!rpc_file_access(sid, executable, "exec"))
+       {
+               if (arg == NULL || strlen(executable) >= sizeof(cmdstr))
+                       return UBUS_STATUS_PERMISSION_DENIED;
+
+               arglen = 2;
+               p = cmdstr + sprintf(cmdstr, "%s", executable);
+
+               blobmsg_for_each_attr(cur, arg, rem)
+               {
+                       if (blobmsg_type(cur) != BLOBMSG_TYPE_STRING)
+                               continue;
+
+                       if (arglen == 255 ||
+                           p + blobmsg_data_len(cur) >= cmdstr + sizeof(cmdstr))
+                               return UBUS_STATUS_PERMISSION_DENIED;
+
+                       p += sprintf(p, " %s", blobmsg_get_string(cur));
+                       arglen++;
+               }
+
+               if (!rpc_file_access(sid, cmdstr, "exec"))
+                       return UBUS_STATUS_PERMISSION_DENIED;
+       }
+
        c = malloc(sizeof(*c));
 
        if (!c)
                return UBUS_STATUS_UNKNOWN_ERROR;
 
-       if (pipe(opipe) || pipe(epipe))
-               return rpc_errno_status();
+       if (pipe(opipe))
+               goto fail_opipe;
+
+       if (pipe(epipe))
+               goto fail_epipe;
 
        switch ((pid = fork()))
        {
        case -1:
-               return rpc_errno_status();
+               goto fail_fork;
 
        case 0:
                uloop_done();
@@ -639,11 +875,11 @@ rpc_file_exec_run(const char *cmd,
                dup2(opipe[1], 1);
                dup2(epipe[1], 2);
 
-               close(devnull);
-               close(opipe[0]);
-               close(opipe[1]);
-               close(epipe[0]);
-               close(epipe[1]);
+               rpc_fdclose(devnull);
+               rpc_fdclose(opipe[0]);
+               rpc_fdclose(opipe[1]);
+               rpc_fdclose(epipe[0]);
+               rpc_fdclose(epipe[1]);
 
                arglen = 2;
                args = malloc(sizeof(char *) * arglen);
@@ -651,7 +887,7 @@ rpc_file_exec_run(const char *cmd,
                if (!args)
                        return UBUS_STATUS_UNKNOWN_ERROR;
 
-               args[0] = (char *)cmd;
+               args[0] = (char *)executable;
                args[1] = NULL;
 
                if (arg)
@@ -693,7 +929,7 @@ rpc_file_exec_run(const char *cmd,
                        }
                }
 
-               if (execv(cmd, args))
+               if (execv(executable, args))
                        return rpc_errno_status();
 
        default:
@@ -717,6 +953,18 @@ rpc_file_exec_run(const char *cmd,
        }
 
        return UBUS_STATUS_OK;
+
+fail_fork:
+       close(epipe[0]);
+       close(epipe[1]);
+
+fail_epipe:
+       close(opipe[0]);
+       close(opipe[1]);
+
+fail_opipe:
+       free(c);
+       return rpc_errno_status();
 }
 
 static int
@@ -732,8 +980,8 @@ rpc_file_exec(struct ubus_context *ctx, struct ubus_object *obj,
        if (!tb[RPC_E_CMD])
                return UBUS_STATUS_INVALID_ARGUMENT;
 
-       return rpc_file_exec_run(blobmsg_data(tb[RPC_E_CMD]),
-                                             tb[RPC_E_PARM], tb[RPC_E_ENV], ctx, req);
+       return rpc_file_exec_run(blobmsg_data(tb[RPC_E_CMD]), tb[RPC_E_SESSION],
+                                tb[RPC_E_PARM], tb[RPC_E_ENV], ctx, req);
 }
 
 
@@ -741,16 +989,17 @@ static int
 rpc_file_api_init(const struct rpc_daemon_ops *o, struct ubus_context *ctx)
 {
        static const struct ubus_method file_methods[] = {
-               UBUS_METHOD("read",    rpc_file_read,  rpc_file_rb_policy),
-               UBUS_METHOD("write",   rpc_file_write, rpc_file_rw_policy),
-               UBUS_METHOD("list",    rpc_file_list,  rpc_file_r_policy),
-               UBUS_METHOD("stat",    rpc_file_stat,  rpc_file_r_policy),
-               UBUS_METHOD("md5",     rpc_file_md5,   rpc_file_r_policy),
-               UBUS_METHOD("exec",    rpc_file_exec,  rpc_exec_policy),
+               UBUS_METHOD("read",    rpc_file_read,   rpc_file_RB_policy),
+               UBUS_METHOD("write",   rpc_file_write,  rpc_file_RW_policy),
+               UBUS_METHOD("list",    rpc_file_list,   rpc_file_R_policy),
+               UBUS_METHOD("stat",    rpc_file_stat,   rpc_file_R_policy),
+               UBUS_METHOD("md5",     rpc_file_md5,    rpc_file_R_policy),
+               UBUS_METHOD("remove",  rpc_file_remove, rpc_file_R_policy),
+               UBUS_METHOD("exec",    rpc_file_exec,   rpc_exec_policy),
        };
 
        static struct ubus_object_type file_type =
-               UBUS_OBJECT_TYPE("luci-rpc-file", file_methods);
+               UBUS_OBJECT_TYPE("rpcd-plugin-file", file_methods);
 
        static struct ubus_object obj = {
                .name = "file",