cache_req: add support for group by id

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>

2 files changed, 58 insertions, 0 deletions

diff --git a/src/responder/common/responder_cache_req.c b/src/responder/common/responder_cache_req.c
index 4f94572a5..ed5c7a249 100644
--- a/src/responder/common/responder_cache_req.c
+++ b/src/responder/common/responder_cache_req.c
@@ -83,6 +83,7 @@ cache_req_input_create(TALLOC_CTX *mem_ctx,
         }
         break;
     case CACHE_REQ_USER_BY_ID:
+    case CACHE_REQ_GROUP_BY_ID:
         if (id == 0) {
             DEBUG(SSSDBG_CRIT_FAILURE, "Bug: id cannot be 0!\n");
             goto fail;
@@ -100,6 +101,7 @@ cache_req_input_create(TALLOC_CTX *mem_ctx,
         break;
 
     case CACHE_REQ_GROUP_BY_NAME:
+    case CACHE_REQ_GROUP_BY_ID:
         input->dp_type = SSS_DP_GROUP;
         break;
 
@@ -165,6 +167,14 @@ cache_req_input_set_domain(struct cache_req_input *input,
             goto done;
         }
         break;
+
+    case CACHE_REQ_GROUP_BY_ID:
+        fqn = talloc_asprintf(tmp_ctx, "GID:%d@%s", input->id, domain->name);
+        if (fqn == NULL) {
+            ret = ENOMEM;
+            goto done;
+        }
+        break;
     }
 
     input->domain = domain;
@@ -197,6 +207,9 @@ static errno_t cache_req_check_ncache(struct cache_req_input *input,
     case CACHE_REQ_USER_BY_ID:
         ret = sss_ncache_check_uid(ncache, neg_timeout, input->id);
         break;
+    case CACHE_REQ_GROUP_BY_ID:
+        ret = sss_ncache_check_gid(ncache, neg_timeout, input->id);
+        break;
     }
 
     if (ret == EEXIST) {
@@ -223,6 +236,7 @@ static void cache_req_add_to_ncache(struct cache_req_input *input,
                                    input->dom_objname);
         break;
     case CACHE_REQ_USER_BY_ID:
+    case CACHE_REQ_GROUP_BY_ID:
         /* Nothing to do. Those types must be unique among all domains so
          * the don't contain domain part. Therefore they must be set only
          * if all domains are search and the entry is not found. */
@@ -256,6 +270,9 @@ static void cache_req_add_to_ncache_global(struct cache_req_input *input,
     case CACHE_REQ_USER_BY_ID:
         ret = sss_ncache_set_uid(ncache, false, input->id);
         break;
+    case CACHE_REQ_GROUP_BY_ID:
+        ret = sss_ncache_set_gid(ncache, false, input->id);
+        break;
     }
 
     if (ret != EOK) {
@@ -294,6 +311,11 @@ static errno_t cache_req_get_object(TALLOC_CTX *mem_ctx,
         ret = sysdb_getgrnam_with_views(mem_ctx, input->domain,
                                         input->dom_objname, &result);
         break;
+    case CACHE_REQ_GROUP_BY_ID:
+        one_item_only = true;
+        ret = sysdb_getgrgid_with_views(mem_ctx, input->domain,
+                                        input->id, &result);
+        break;
     case CACHE_REQ_INITGROUPS:
         one_item_only = false;
         ret = sysdb_initgroups_with_views(mem_ctx, input->domain,
@@ -830,6 +852,28 @@ cache_req_group_by_name_send(TALLOC_CTX *mem_ctx,
 }
 
 struct tevent_req *
+cache_req_group_by_id_send(TALLOC_CTX *mem_ctx,
+                           struct tevent_context *ev,
+                           struct resp_ctx *rctx,
+                           struct sss_nc_ctx *ncache,
+                           int neg_timeout,
+                           int cache_refresh_percent,
+                           const char *domain,
+                           gid_t gid)
+{
+    struct cache_req_input *input;
+
+    input = cache_req_input_create(mem_ctx, CACHE_REQ_GROUP_BY_ID, NULL, gid);
+    if (input == NULL) {
+        return NULL;
+    }
+
+    return cache_req_steal_input_and_send(mem_ctx, ev, rctx, ncache,
+                                          neg_timeout, cache_refresh_percent,
+                                          domain, input);
+}
+
+struct tevent_req *
 cache_req_initgr_by_name_send(TALLOC_CTX *mem_ctx,
                               struct tevent_context *ev,
                               struct resp_ctx *rctx,
diff --git a/src/responder/common/responder_cache_req.h b/src/responder/common/responder_cache_req.h
index 65a0908f9..3b8f3f713 100644
--- a/src/responder/common/responder_cache_req.h
+++ b/src/responder/common/responder_cache_req.h
@@ -31,6 +31,7 @@ enum cache_req_type {
     CACHE_REQ_USER_BY_NAME,
     CACHE_REQ_USER_BY_ID,
     CACHE_REQ_GROUP_BY_NAME,
+    CACHE_REQ_GROUP_BY_ID,
     CACHE_REQ_INITGROUPS
 };
 
@@ -101,6 +102,19 @@ cache_req_group_by_name_send(TALLOC_CTX *mem_ctx,
     cache_req_recv(mem_ctx, req, _result, _domain)
 
 struct tevent_req *
+cache_req_group_by_id_send(TALLOC_CTX *mem_ctx,
+                           struct tevent_context *ev,
+                           struct resp_ctx *rctx,
+                           struct sss_nc_ctx *ncache,
+                           int neg_timeout,
+                           int cache_refresh_percent,
+                           const char *domain,
+                           gid_t gid);
+
+#define cache_req_group_by_id_recv(mem_ctx, req, _result, _domain) \
+    cache_req_recv(mem_ctx, req, _result, _domain)
+
+struct tevent_req *
 cache_req_initgr_by_name_send(TALLOC_CTX *mem_ctx,
                               struct tevent_context *ev,
                               struct resp_ctx *rctx,