diff options
Diffstat (limited to 'src')
-rw-r--r-- | src/modules/m_sasl.cpp | 80 | ||||
-rw-r--r-- | src/modules/m_spanningtree/encap.cpp | 11 | ||||
-rw-r--r-- | src/modules/m_spanningtree/postcommand.cpp | 38 |
3 files changed, 80 insertions, 49 deletions
diff --git a/src/modules/m_sasl.cpp b/src/modules/m_sasl.cpp index 5241a14e5..b205f91c1 100644 --- a/src/modules/m_sasl.cpp +++ b/src/modules/m_sasl.cpp @@ -51,7 +51,7 @@ class SaslAuthenticator : public classbase ServerInstance->PI->SendEncapsulatedData(params); } - SaslResult GetSaslResult(std::string &result_) + SaslResult GetSaslResult(const std::string &result_) { if (result_ == "F") return SASL_FAIL; @@ -63,25 +63,25 @@ class SaslAuthenticator : public classbase } /* checks for and deals with a state change. */ - SaslState ProcessInboundMessage(parameterlist &msg) + SaslState ProcessInboundMessage(const std::vector<std::string> &msg) { switch (this->state) { case SASL_INIT: - this->agent = msg[2]; - this->user->Write("AUTHENTICATE %s", msg[5].c_str()); + this->agent = msg[0]; + this->user->Write("AUTHENTICATE %s", msg[3].c_str()); this->state = SASL_COMM; break; case SASL_COMM: - if (msg[2] != this->agent) + if (msg[0] != this->agent) return this->state; - if (msg[4] != "D") - this->user->Write("AUTHENTICATE %s", msg[5].c_str()); + if (msg[2] != "D") + this->user->Write("AUTHENTICATE %s", msg[3].c_str()); else { this->state = SASL_DONE; - this->result = this->GetSaslResult(msg[5]); + this->result = this->GetSaslResult(msg[3]); } break; @@ -183,18 +183,52 @@ class CommandAuthenticate : public Command } }; +class CommandSASL : public Command +{ + Module* Creator; + public: + CommandSASL(InspIRCd* Instance, Module* creator) : Command(Instance, "SASL", 0, 2), Creator(creator) + { + this->source = "m_sasl.so"; + this->disabled = true; // should not be called by users + } + + CmdResult Handle(const std::vector<std::string>& parameters, User *user) + { + User* target = ServerInstance->FindNick(parameters[1]); + if (!target) + { + ServerInstance->Logs->Log("m_sasl", DEBUG,"User not found in sasl ENCAP event: %s", parameters[1].c_str()); + return CMD_FAILURE; + } + + SaslAuthenticator *sasl; + if (!target->GetExt("sasl_authenticator", sasl)) + return CMD_FAILURE; + + SaslState state = sasl->ProcessInboundMessage(parameters); + if (state == SASL_DONE) + { + delete sasl; + target->Shrink("sasl"); + } + return CMD_SUCCESS; + } +}; class ModuleSASL : public Module { - CommandAuthenticate sasl; + CommandAuthenticate auth; + CommandSASL sasl; public: ModuleSASL(InspIRCd* Me) - : Module(Me), sasl(Me, this) + : Module(Me), auth(Me, this), sasl(Me, this) { Implementation eventlist[] = { I_OnEvent, I_OnUserRegister, I_OnPostConnect, I_OnUserDisconnect, I_OnCleanup }; ServerInstance->Modules->Attach(eventlist, this, 5); + ServerInstance->AddCommand(&auth); ServerInstance->AddCommand(&sasl); if (!ServerInstance->Modules->Find("m_services_account.so") || !ServerInstance->Modules->Find("m_cap.so")) @@ -255,32 +289,6 @@ class ModuleSASL : public Module virtual void OnEvent(Event *ev) { GenericCapHandler(ev, "sasl", "sasl"); - - if (ev->GetEventID() == "encap_received") - { - parameterlist* parameters = (parameterlist*)ev->GetData(); - - if ((*parameters)[1] != "SASL") - return; - - User* target = ServerInstance->FindNick((*parameters)[3]); - if (!target) - { - ServerInstance->Logs->Log("m_sasl", DEBUG,"User not found in sasl ENCAP event: %s", (*parameters)[3].c_str()); - return; - } - - SaslAuthenticator *sasl_; - if (!target->GetExt("sasl_authenticator", sasl_)) - return; - - SaslState state = sasl_->ProcessInboundMessage(*parameters); - if (state == SASL_DONE) - { - delete sasl_; - target->Shrink("sasl"); - } - } } }; diff --git a/src/modules/m_spanningtree/encap.cpp b/src/modules/m_spanningtree/encap.cpp index c56f8dc74..7b85a49d0 100644 --- a/src/modules/m_spanningtree/encap.cpp +++ b/src/modules/m_spanningtree/encap.cpp @@ -22,15 +22,20 @@ -/** remote MOTD. leet, huh? */ +/** ENCAP */ bool TreeSocket::Encap(const std::string &prefix, parameterlist ¶ms) { if (params.size() > 1) { if (InspIRCd::Match(ServerInstance->Config->GetSID(), params[0])) { - Event event((char*) ¶ms, (Module*)this->Utils->Creator, "encap_received"); - event.Send(ServerInstance); + User* who = this->ServerInstance->FindUUID(prefix); + if (!who) + who = Utils->ServerUser; + + parameterlist plist(params.begin() + 2, params.end()); + ServerInstance->CallCommandHandler(params[1].c_str(), plist, who); + // discard return value, ENCAP shall succeed even if the command does not exist } params[params.size() - 1] = ":" + params[params.size() - 1]; diff --git a/src/modules/m_spanningtree/postcommand.cpp b/src/modules/m_spanningtree/postcommand.cpp index 428079c23..545ad3bfb 100644 --- a/src/modules/m_spanningtree/postcommand.cpp +++ b/src/modules/m_spanningtree/postcommand.cpp @@ -39,25 +39,43 @@ void ModuleSpanningTree::OnPostCommand(const std::string &command, const std::ve RouteDescriptor routing = thiscmd->GetRouting(user, parameters); + std::string sent_cmd = command; + parameterlist params; + if (routing.type == ROUTE_TYPE_LOCALONLY) + { return; + } + else if (routing.type == ROUTE_TYPE_OPT_BCAST) + { + params.push_back("*"); + params.push_back(command); + sent_cmd = "ENCAP"; + } + else if (routing.type == ROUTE_TYPE_OPT_UCAST) + { + params.push_back(routing.serverdest); + params.push_back(command); + sent_cmd = "ENCAP"; + } + else + { + Module* srcmodule = ServerInstance->Modules->Find(thiscmd->source); - Module* srcmodule = ServerInstance->Modules->Find(thiscmd->source); - - if (srcmodule && !(srcmodule->GetVersion().Flags & VF_COMMON)) { - ServerInstance->Logs->Log("m_spanningtree",ERROR,"Routed command %s from non-VF_COMMON module %s", - command.c_str(), thiscmd->source.c_str()); - return; + if (srcmodule && !(srcmodule->GetVersion().Flags & VF_COMMON)) { + ServerInstance->Logs->Log("m_spanningtree",ERROR,"Routed command %s from non-VF_COMMON module %s", + command.c_str(), thiscmd->source.c_str()); + return; + } } std::string output_text; ServerInstance->Parser->TranslateUIDs(thiscmd->translation, parameters, output_text, true, thiscmd); - parameterlist params; params.push_back(output_text); - if (routing.type == ROUTE_TYPE_BROADCAST) - Utils->DoOneToMany(user->uuid, command, params); + if (routing.type == ROUTE_TYPE_BROADCAST || routing.type == ROUTE_TYPE_OPT_BCAST) + Utils->DoOneToMany(user->uuid, sent_cmd, params); else - Utils->DoOneToOne(user->uuid, command, params, routing.serverdest); + Utils->DoOneToOne(user->uuid, sent_cmd, params, routing.serverdest); } |