api: fixed the permission check issue for DVR (and maybe other)

diff --git a/src/api/api_idnode.c b/src/api/api_idnode.c
index bc2a254992b20b8290a040b63a5628aabadd1e8e..58abcc1caacdbf04d752cb23b4b01191b8039d3a 100644 (file)
--- a/src/api/api_idnode.c
+++ b/src/api/api_idnode.c
@@ -144,7 +144,8 @@ api_idnode_grid
     e = htsmsg_create_map();
     htsmsg_add_str(e, "uuid", idnode_uuid_as_str(ins.is_array[i]));
     in = ins.is_array[i];
-    idnode_perm_set(in, perm);
+    if (idnode_perm(in, perm, NULL))
+      continue;
     idnode_read0(in, e, flist, 0, perm->aa_lang);
     idnode_perm_unset(in);
     htsmsg_add_msg(list, NULL, e);
@@ -411,7 +412,10 @@ api_idnode_tree
   /* Root node */
   if (isroot && node) {
     htsmsg_t *m;
-    idnode_perm_set(node, perm);
+    if (idnode_perm(node, perm, NULL)) {
+      pthread_mutex_unlock(&global_lock);
+      return EINVAL;
+    }
     m = idnode_serialize(node, perm->aa_lang);
     idnode_perm_unset(node);
     htsmsg_add_u32(m, "leaf", idnode_is_leaf(node));
@@ -426,7 +430,8 @@ api_idnode_tree
       for(i = 0; i < v->is_count; i++) {
         idnode_t *in = v->is_array[i];
         htsmsg_t *m;
-        idnode_perm_set(in, perm);
+        if (idnode_perm(in, perm, NULL))
+          continue;
         m = idnode_serialize(v->is_array[i], perm->aa_lang);
         idnode_perm_unset(in);
         htsmsg_add_u32(m, "leaf", idnode_is_leaf(v->is_array[i]));

diff --git a/src/idnode.h b/src/idnode.h
index 908169840ef027ad98bb1d7289b35f7d7bf95bfe..59c6d4d0242ecc2612066f91993a6632c825f8fb 100644 (file)
--- a/src/idnode.h
+++ b/src/idnode.h
@@ -203,7 +203,6 @@ int       idnode_write0 (idnode_t *self, htsmsg_t *m, int optmask, int dosave);
 #define idnode_update(in, m)   idnode_write0(in, m, PO_RDONLY | PO_WRONCE, 1)
 
 int idnode_perm(idnode_t *self, struct access *a, htsmsg_t *msg_to_write);
-static inline void idnode_perm_set(idnode_t *self, struct access *a) { self->in_access = a; }
 static inline void idnode_perm_unset(idnode_t *self) { self->in_access = NULL; }
 
 idnode_list_mapping_t * idnode_list_link