diff options
author | Fabiano FidĂȘncio <fidencio@redhat.com> | 2017-08-14 13:35:20 +0200 |
---|---|---|
committer | Jakub Hrozek <jhrozek@redhat.com> | 2017-08-28 20:55:07 +0200 |
commit | 8888d7a46371ddd2c2514c3e81b58bb1090902a2 (patch) | |
tree | 5cfe92cc1ea12b82888b66cee4907d7980e56885 | |
parent | 9908bdc9755e744c3e2c7c746a4edf95f9083ef5 (diff) | |
download | sssd-8888d7a46371ddd2c2514c3e81b58bb1090902a2.tar.gz sssd-8888d7a46371ddd2c2514c3e81b58bb1090902a2.tar.xz sssd-8888d7a46371ddd2c2514c3e81b58bb1090902a2.zip |
CACHE_REQ: Don't error out when searching by id = 0
This code path can be easily triggered by calling `id 0` and SSSD should
not error out in this case.
Previous patches in this series already add uid and gid 0 to the
negative cache and we can properly handle this situation.
Related: https://pagure.io/SSSD/sssd/issue/3460
Signed-off-by: Fabiano FidĂȘncio <fidencio@redhat.com>
Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
-rw-r--r-- | src/responder/common/cache_req/cache_req_data.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/src/responder/common/cache_req/cache_req_data.c b/src/responder/common/cache_req/cache_req_data.c index 5ab1493b8..8726e139f 100644 --- a/src/responder/common/cache_req/cache_req_data.c +++ b/src/responder/common/cache_req/cache_req_data.c @@ -119,12 +119,6 @@ cache_req_data_create(TALLOC_CTX *mem_ctx, case CACHE_REQ_USER_BY_ID: case CACHE_REQ_GROUP_BY_ID: case CACHE_REQ_OBJECT_BY_ID: - if (input->id == 0) { - DEBUG(SSSDBG_CRIT_FAILURE, "Bug: id cannot be 0!\n"); - ret = ERR_INTERNAL; - goto done; - } - data->id = input->id; break; case CACHE_REQ_OBJECT_BY_SID: |