mirror of
https://github.com/samba-team/samba.git
synced 2025-11-09 20:23:51 +03:00
r12819: Fix swat authentication again. We need to pass the socket_address
structure around, so the auth code knows where the request came from. Andrew Bartlett
This commit is contained in:
committed by
Gerald (Jerry) Carter
parent
36c1f67f12
commit
7a7b2668c0
@@ -29,7 +29,7 @@
|
||||
static int ejs_doauth(MprVarHandle eid,
|
||||
TALLOC_CTX *tmp_ctx, struct MprVar *auth, const char *username,
|
||||
const char *password, const char *domain, const char *workstation,
|
||||
const char *authtype)
|
||||
struct socket_address *remote_host, const char *authtype)
|
||||
{
|
||||
struct auth_usersupplied_info *user_info = NULL;
|
||||
struct auth_serversupplied_info *server_info = NULL;
|
||||
@@ -63,7 +63,7 @@ static int ejs_doauth(MprVarHandle eid,
|
||||
|
||||
user_info->workstation_name = workstation;
|
||||
|
||||
user_info->remote_host = NULL;
|
||||
user_info->remote_host = remote_host;
|
||||
|
||||
user_info->password_state = AUTH_PASSWORD_PLAIN;
|
||||
user_info->password.plaintext = talloc_strdup(user_info, password);
|
||||
@@ -75,7 +75,9 @@ static int ejs_doauth(MprVarHandle eid,
|
||||
|
||||
nt_status = auth_check_password(auth_context, tmp_ctx, user_info, &server_info);
|
||||
if (!NT_STATUS_IS_OK(nt_status)) {
|
||||
mprSetPropertyValue(auth, "report", mprString("Login Failed"));
|
||||
mprSetPropertyValue(auth, "report",
|
||||
mprString(talloc_asprintf(mprMemCtx(), "Login Failed: %s",
|
||||
get_friendly_nt_error_msg(nt_status))));
|
||||
mprSetPropertyValue(auth, "result", mprCreateBoolVar(False));
|
||||
goto done;
|
||||
}
|
||||
@@ -111,8 +113,9 @@ static int ejs_userAuth(MprVarHandle eid, int argc, struct MprVar **argv)
|
||||
const char *workstation;
|
||||
struct MprVar auth;
|
||||
struct cli_credentials *creds;
|
||||
struct socket_address *remote_host;
|
||||
|
||||
if (argc != 1 || argv[0]->type != MPR_TYPE_OBJECT) {
|
||||
if (argc != 2 || argv[0]->type != MPR_TYPE_OBJECT || argv[1]->type != MPR_TYPE_OBJECT) {
|
||||
ejsSetErrorMsg(eid, "userAuth invalid arguments, this function requires an object.");
|
||||
return -1;
|
||||
}
|
||||
@@ -120,7 +123,13 @@ static int ejs_userAuth(MprVarHandle eid, int argc, struct MprVar **argv)
|
||||
/* get credential values from credentials object */
|
||||
creds = mprGetPtr(argv[0], "creds");
|
||||
if (creds == NULL) {
|
||||
ejsSetErrorMsg(eid, "userAuth requires a 'creds' element");
|
||||
ejsSetErrorMsg(eid, "userAuth requires a 'creds' first parameter");
|
||||
return -1;
|
||||
}
|
||||
|
||||
remote_host = mprGetPtr(argv[1], "socket_address");
|
||||
if (remote_host == NULL) {
|
||||
ejsSetErrorMsg(eid, "userAuth requires a socket address second parameter");
|
||||
return -1;
|
||||
}
|
||||
|
||||
@@ -139,10 +148,10 @@ static int ejs_userAuth(MprVarHandle eid, int argc, struct MprVar **argv)
|
||||
|
||||
auth = mprObject("auth");
|
||||
|
||||
if (domain && (strcmp("System User", domain) == 0)) {
|
||||
ejs_doauth(eid, tmp_ctx, &auth, username, password, domain, workstation, "unix");
|
||||
if (domain && (strcmp("SYSTEM USER", domain) == 0)) {
|
||||
ejs_doauth(eid, tmp_ctx, &auth, username, password, domain, workstation, remote_host, "unix");
|
||||
} else {
|
||||
ejs_doauth(eid, tmp_ctx, &auth, username, password, domain, workstation, "sam");
|
||||
ejs_doauth(eid, tmp_ctx, &auth, username, password, domain, workstation, remote_host, "sam");
|
||||
}
|
||||
|
||||
mpr_Return(eid, auth);
|
||||
|
||||
@@ -28,7 +28,7 @@ f.display();
|
||||
creds.set_domain(form.Domain);
|
||||
creds.set_workstation(request['REMOTE_HOST']);
|
||||
|
||||
auth = userAuth(creds);
|
||||
auth = userAuth(creds, request['REMOTE_SOCKET_ADDRESS']);
|
||||
if (auth == undefined) {
|
||||
write("<b>Invalid login - please try again<br /></b>\n");
|
||||
} else if (auth.result) {
|
||||
@@ -38,6 +38,7 @@ f.display();
|
||||
session.authinfo.username = auth.username;
|
||||
session.authinfo.domain = auth.domain;
|
||||
session.authinfo.credentials = creds;
|
||||
session.authinfo.session_info = auth.session_info;
|
||||
|
||||
/* if the user was asking for the login page, then now
|
||||
redirect them to the main page. Otherwise just
|
||||
@@ -48,8 +49,10 @@ f.display();
|
||||
} else {
|
||||
redirect(session_uri(request.REQUEST_URI));
|
||||
}
|
||||
} else {
|
||||
} else if (auth.report == undefined) {
|
||||
write("<b>Login failed - please try again<br /></b>\n");
|
||||
} else {
|
||||
write("<b>Login failed: " + auth.report + " - please try again<br /></b>\n");
|
||||
}
|
||||
}
|
||||
%>
|
||||
|
||||
Reference in New Issue
Block a user