diff options
| author | Jakub Hrozek <jhrozek@redhat.com> | 2012-04-25 16:17:35 +0200 |
|---|---|---|
| committer | Stephen Gallagher <sgallagh@redhat.com> | 2012-05-02 08:10:16 -0400 |
| commit | 69e7d6649b58c66675ef38084868fc5356c5a240 (patch) | |
| tree | fba31f1c48e792809dd968d66815f8e50117d85f /src/db/sysdb_sudo.c | |
| parent | b6dfbf81c61d4431aaa81687ec53e892f8b71edb (diff) | |
| download | sssd_unused-69e7d6649b58c66675ef38084868fc5356c5a240.tar.gz sssd_unused-69e7d6649b58c66675ef38084868fc5356c5a240.tar.xz sssd_unused-69e7d6649b58c66675ef38084868fc5356c5a240.zip | |
SUDO: Return ret, not EOK
This patch fixes bad refactoring - the function used to return value
directly on error and EOK as the last statement. If was then converted
into using goto label, but the last statement was still returning EOK
instead of the value it should.
Diffstat (limited to 'src/db/sysdb_sudo.c')
| -rw-r--r-- | src/db/sysdb_sudo.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/db/sysdb_sudo.c b/src/db/sysdb_sudo.c index f579ba8b..8f396ef5 100644 --- a/src/db/sysdb_sudo.c +++ b/src/db/sysdb_sudo.c @@ -336,7 +336,7 @@ sysdb_get_sudo_user_info(TALLOC_CTX *mem_ctx, const char *username, *groupnames = talloc_steal(mem_ctx, sysdb_groupnames); done: talloc_free(tmp_ctx); - return EOK; + return ret; } errno_t |