X-Git-Url: https://git.netwichtig.de/gitweb/?a=blobdiff_plain;f=src%2Fmodules%2Fm_swhois.cpp;h=41248832df0244bc19227bf4492a6b8b5fbcf0bd;hb=c4458ecc70025aeac7ca87115ed0a698e7bbcdad;hp=98ed007d0ae2dc497e5fe3448461f20aab57ef66;hpb=5d407fb44c759524881712a80febb86b4506ddbf;p=user%2Fhenk%2Fcode%2Finspircd.git

diff --git a/src/modules/m_swhois.cpp b/src/modules/m_swhois.cpp
index 98ed007d0..41248832d 100644
--- a/src/modules/m_swhois.cpp
+++ b/src/modules/m_swhois.cpp
@@ -31,7 +31,7 @@ class cmd_swhois : public command_t
 		this->source = "m_swhois.so";
 	}
 
-	void Handle(char** parameters, int pcnt, userrec* user)
+	void Handle(const char** parameters, int pcnt, userrec* user)
 	{
 		userrec* dest = Srv->FindNick(std::string(parameters[0]));
 		if(dest)
@@ -45,7 +45,8 @@ class cmd_swhois : public command_t
 				line.append(parameters[i]);
 			}
 			
-			std::string* text = (std::string*)dest->GetExt("swhois");
+			std::string* text;
+			dest->GetExt("swhois", text);
 	
 			if(text)
 			{
@@ -65,7 +66,7 @@ class cmd_swhois : public command_t
 			}
 			
 			text = new std::string(line);
-			dest->Extend("swhois", (char*)text);
+			dest->Extend("swhois", text);
 		}
 	}
 };
@@ -99,10 +100,10 @@ class ModuleSWhois : public Module
 	// :kenny.chatspike.net 320 Brain Azhrarn :is getting paid to play games.
 	virtual void OnWhois(userrec* source, userrec* dest)
 	{
-		char* desc = dest->GetExt("swhois");
-		if (desc)
+		std::string* swhois;
+		dest->GetExt("swhois", swhois);
+		if (swhois)
 		{
-			std::string* swhois = (std::string*)desc;
 			WriteServ(source->fd,"320 %s %s :%s",source->nick,dest->nick,swhois->c_str());
 		}
 	}
@@ -118,11 +119,10 @@ class ModuleSWhois : public Module
 		if (extname == "swhois")
 		{
 			// check if this user has an swhois field to send
-			char* field = user->GetExt("swhois");
-			if (field)
+			std::string* swhois;
+			user->GetExt("swhois", swhois);
+			if (swhois)
 			{
-				// get our extdata out with a cast
-				std::string* swhois = (std::string*)field;
 				// call this function in the linking module, let it format the data how it
 				// sees fit, and send it on its way. We dont need or want to know how.
 				proto->ProtoSendMetaData(opaque,TYPE_USER,user,extname,*swhois);
@@ -133,10 +133,10 @@ class ModuleSWhois : public Module
 	// when a user quits, tidy up their metadata
 	virtual void OnUserQuit(userrec* user, const std::string &message)
 	{
-		char* field = user->GetExt("swhois");
-		if (field)
+		std::string* swhois;
+		user->GetExt("swhois", swhois);
+		if (swhois)
 		{
-			std::string* swhois = (std::string*)field;
 			user->Shrink("swhois");
 			DELETE(swhois);
 		}
@@ -148,10 +148,10 @@ class ModuleSWhois : public Module
 		if (target_type == TYPE_USER)
 		{
 			userrec* user = (userrec*)item;
-			char* field = user->GetExt("swhois");
-			if (field)
+			std::string* swhois;
+			user->GetExt("swhois", swhois);
+			if (swhois)
 			{
-				std::string* swhois = (std::string*)field;
 				user->Shrink("swhois");
 				DELETE(swhois);
 			}
@@ -172,10 +172,11 @@ class ModuleSWhois : public Module
 		{
 			userrec* dest = (userrec*)target;
 			// if they dont already have an swhois field, accept the remote server's
-			if (!dest->GetExt("swhois"))
+			std::string* text;
+			if (!dest->GetExt("swhois", text))
 			{
 				std::string* text = new std::string(extdata);
-				dest->Extend("swhois",(char*)text);
+				dest->Extend("swhois",text);
 			}
 		}
 	}
@@ -192,14 +193,15 @@ class ModuleSWhois : public Module
 				
 				if(swhois.length())
 				{
-					if(std::string* old = (std::string*)user->GetExt("swhois"))
+					std::string* old;
+					if(user->GetExt("swhois", old))
 					{
 						user->Shrink("swhois");
 						DELETE(old);
 					}
 			
 					std::string* text = new std::string(swhois);
-					user->Extend("swhois", (char*)text);
+					user->Extend("swhois", text);
 					
 					break;
 				}