summaryrefslogtreecommitdiff
path: root/src/providers/proxy
diff options
context:
space:
mode:
authorJakub Hrozek <jhrozek@redhat.com>2012-04-25 17:15:21 +0200
committerStephen Gallagher <sgallagh@redhat.com>2012-05-03 11:46:17 -0400
commitc9041cb7addc1a49e0771246d17de101662fbcbc (patch)
treef5b94be10422e350275c49ee1798cd7cd625c64a /src/providers/proxy
parent3353308213d623b2ad0f0044859039c0f8cb7ef5 (diff)
downloadsssd-c9041cb7addc1a49e0771246d17de101662fbcbc.tar.gz
sssd-c9041cb7addc1a49e0771246d17de101662fbcbc.tar.bz2
sssd-c9041cb7addc1a49e0771246d17de101662fbcbc.zip
PROXY: return correct return codes
We were reporting on the value of "status" instead of "ret'. We also didn't set ret to EOK in cases group contained no members.
Diffstat (limited to 'src/providers/proxy')
-rw-r--r--src/providers/proxy/proxy_id.c16
1 files changed, 9 insertions, 7 deletions
diff --git a/src/providers/proxy/proxy_id.c b/src/providers/proxy/proxy_id.c
index e1325b74..8a8c7ca8 100644
--- a/src/providers/proxy/proxy_id.c
+++ b/src/providers/proxy/proxy_id.c
@@ -143,8 +143,8 @@ done:
talloc_zfree(tmpctx);
if (ret) {
DEBUG(SSSDBG_OP_FAILURE,
- ("proxy -> getpwnam_r failed for '%s' <%d>\n",
- name, status));
+ ("proxy -> getpwnam_r failed for '%s' <%d>: %s\n",
+ name, ret, strerror(ret)));
}
return ret;
}
@@ -334,7 +334,8 @@ done:
talloc_zfree(tmpctx);
if (ret) {
DEBUG(SSSDBG_CRIT_FAILURE,
- ("proxy -> getpwuid_r failed for '%d' <%d>\n", uid, status));
+ ("proxy -> getpwuid_r failed for '%d' <%d>: %s\n",
+ uid, ret, strerror(ret)));
}
return ret;
}
@@ -656,6 +657,7 @@ static errno_t proxy_process_missing_users(struct sysdb_ctx *sysdb,
}
}
+ ret = EOK;
done:
talloc_free(tmp_ctx);
return ret;
@@ -848,8 +850,8 @@ done:
talloc_zfree(tmpctx);
if (ret) {
DEBUG(SSSDBG_OP_FAILURE,
- ("proxy -> getgrnam_r failed for '%s' <%d>\n",
- name, status));
+ ("proxy -> getgrnam_r failed for '%s' <%d>: %s\n",
+ name, ret, strerror(ret)));
}
return ret;
}
@@ -917,8 +919,8 @@ done:
talloc_zfree(tmpctx);
if (ret) {
DEBUG(SSSDBG_OP_FAILURE,
- ("proxy -> getgrgid_r failed for '%d' <%d>\n",
- gid, status));
+ ("proxy -> getgrgid_r failed for '%d' <%d>: %s\n",
+ gid, ret, strerror(ret)));
}
return ret;
}