diff options
author | wrowe <wrowe@13f79535-47bb-0310-9956-ffa450edef68> | 2001-06-06 20:04:43 +0000 |
---|---|---|
committer | wrowe <wrowe@13f79535-47bb-0310-9956-ffa450edef68> | 2001-06-06 20:04:43 +0000 |
commit | be7851e7c8f54270ac5c683be4dd65533443c078 (patch) | |
tree | fc34fcdc4d14c28e449b71f324a1130627f27cb6 /user/win32/userinfo.c | |
parent | e12055ffc7fd52117a28d58b93675f034307be49 (diff) | |
download | libapr-be7851e7c8f54270ac5c683be4dd65533443c078.tar.gz |
Whoops, APR_BADARG is bogus [my doing.] Still return APR_EINVAL since
we don't know where the id came from, and if it's safe to pass to the
kernel.
git-svn-id: http://svn.apache.org/repos/asf/apr/apr/trunk@61722 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'user/win32/userinfo.c')
-rw-r--r-- | user/win32/userinfo.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/user/win32/userinfo.c b/user/win32/userinfo.c index f07154e06..3c160728e 100644 --- a/user/win32/userinfo.c +++ b/user/win32/userinfo.c @@ -109,11 +109,11 @@ APR_DECLARE(apr_status_t) apr_get_username(char **username, apr_uid_t userid, ap char name[MAX_PATH], domain[MAX_PATH]; DWORD cbname = sizeof(name), cbdomain = sizeof(domain); if (!userid) - return APR_BADARG; + return APR_EINVAL; if (!LookupAccountSid(NULL, userid, name, &cbname, domain, &cbdomain, &type)) return apr_get_os_error(); - if (type != SidTypeUser && type != SidTypeAlias) - return APR_BADARG; + if (type != SidTypeUser && type != SidTypeAlias && type != SidTypeWellKnownGroup) + return APR_EINVAL; *username = apr_pstrdup(p, name); return APR_SUCCESS; } @@ -121,9 +121,9 @@ APR_DECLARE(apr_status_t) apr_get_username(char **username, apr_uid_t userid, ap APR_DECLARE(apr_status_t) apr_compare_users(apr_uid_t left, apr_uid_t right) { if (!left || !right) - return APR_BADARG; + return APR_EINVAL; if (!IsValidSid(left) || !IsValidSid(right)) - return APR_BADARG; + return APR_EINVAL; if (!EqualSid(left, right)) return APR_EMISMATCH; return APR_SUCCESS; |