X-Git-Url: https://git.netwichtig.de/gitweb/?a=blobdiff_plain;f=src%2Fcommands%2Fcmd_who.cpp;h=a9f27de83c7cb725644ec74cf0d241859652de10;hb=2d4a319d961e3f9e6aa9f7926f9ed320d72de8da;hp=091ba728b45c6baca97ce2780db1f3b74e00c959;hpb=dd98d56cabcd08a3d5207454e002b187af98d28a;p=user%2Fhenk%2Fcode%2Finspircd.git

diff --git a/src/commands/cmd_who.cpp b/src/commands/cmd_who.cpp
index 091ba728b..a9f27de83 100644
--- a/src/commands/cmd_who.cpp
+++ b/src/commands/cmd_who.cpp
@@ -15,7 +15,9 @@
 #include "wildcard.h"
 #include "commands/cmd_who.h"
 
-static char *get_first_visible_channel(User *u)
+static const std::string star = "*";
+
+static const std::string& get_first_visible_channel(User *u)
 {
 	UCListIter i = u->chans.begin();
 	if (i != u->chans.end())
@@ -24,7 +26,7 @@ static char *get_first_visible_channel(User *u)
 			return i->first->name;
 	}
 
-	return "*";
+	return star;
 }
 
 bool CommandWho::whomatch(User* user, const char* matchtext)
@@ -134,13 +136,13 @@ bool CommandWho::CanView(Channel* chan, User* user)
 
 void CommandWho::SendWhoLine(User* user, const std::string &initial, Channel* ch, User* u, std::vector<std::string> &whoresults)
 {
-	std::string lcn = get_first_visible_channel(u);
-	Channel* chlast = ServerInstance->FindChan(lcn);
-
 	/* Not visible to this user */
 	if (u->Visibility && !u->Visibility->VisibleTo(user))
 		return;
 
+	const std::string& lcn = get_first_visible_channel(u);
+	Channel* chlast = ServerInstance->FindChan(lcn);
+
 	std::string wholine =	initial + (ch ? ch->name : lcn) + " " + u->ident + " " + (opt_showrealhost ? u->host : u->dhost) + " " +
 				((*ServerInstance->Config->HideWhoisServer && !IS_OPER(user)) ? ServerInstance->Config->HideWhoisServer : u->server) +
 				" " + u->nick + " ";
@@ -165,7 +167,7 @@ void CommandWho::SendWhoLine(User* user, const std::string &initial, Channel* ch
 	whoresults.push_back(wholine);
 }
 
-CmdResult CommandWho::Handle (const char** parameters, int pcnt, User *user)
+CmdResult CommandWho::Handle (const std::vector<std::string>& parameters, User *user)
 {
 	/*
 	 * XXX - RFC says:
@@ -191,13 +193,14 @@ CmdResult CommandWho::Handle (const char** parameters, int pcnt, User *user)
 	std::vector<std::string> whoresults;
 	std::string initial = "352 " + std::string(user->nick) + " ";
 
-	const char* matchtext = NULL;
+	char matchtext[MAXBUF];
 	bool usingwildcards = false;
 
 	/* Change '0' into '*' so the wildcard matcher can grok it */
-	matchtext = parameters[0];
-	if (!strcmp(matchtext,"0"))
-		matchtext = "*";
+	if (parameters[0] == "0")
+		strlcpy(matchtext, "*", MAXBUF);
+	else
+		strlcpy(matchtext, parameters[0].c_str(), MAXBUF);
 
 	for (const char* check = matchtext; *check; check++)
 	{
@@ -208,15 +211,15 @@ CmdResult CommandWho::Handle (const char** parameters, int pcnt, User *user)
 		}
 	}
 
-	if (pcnt > 1)
-	{
-		/* parse flags */
-		const char *iter = parameters[1];
+	if (ServerInstance->FindServerName(matchtext))
+		usingwildcards = true;
 
+	if (parameters.size() > 1)
+	{
 		/* Fix for bug #444, WHO flags count as a wildcard */
 		usingwildcards = true;
 
-		while (*iter)
+		for (std::string::const_iterator iter = parameters[1].begin(); iter != parameters[1].end(); ++iter)
 		{
 			switch (*iter)
 			{
@@ -256,8 +259,6 @@ CmdResult CommandWho::Handle (const char** parameters, int pcnt, User *user)
 					opt_far = true;
 				break;
 			}
-
-			*iter++;
 		}
 	}
 
@@ -336,13 +337,13 @@ CmdResult CommandWho::Handle (const char** parameters, int pcnt, User *user)
 	{
 		for (std::vector<std::string>::const_iterator n = whoresults.begin(); n != whoresults.end(); n++)
 			user->WriteServ(*n);
-		user->WriteServ("315 %s %s :End of /WHO list.",user->nick, *parameters[0] ? parameters[0] : "*");
+		user->WriteNumeric(315, "%s %s :End of /WHO list.",user->nick.c_str(), *parameters[0].c_str() ? parameters[0].c_str() : "*");
 		return CMD_SUCCESS;
 	}
 	else
 	{
 		/* BZZT! Too many results. */
-		user->WriteServ("315 %s %s :Too many results",user->nick, parameters[0]);
+		user->WriteNumeric(315, "%s %s :Too many results",user->nick.c_str(), parameters[0].c_str());
 		return CMD_FAILURE;
 	}
 }