]> git.netwichtig.de Git - user/henk/code/inspircd.git/blob - src/modules/m_spanningtree/treesocket2.cpp
Revert automated conversion by Special, as it (unfortunately) neglects some details...
[user/henk/code/inspircd.git] / src / modules / m_spanningtree / treesocket2.cpp
1 /*       +------------------------------------+
2  *       | Inspire Internet Relay Chat Daemon |
3  *       +------------------------------------+
4  *
5  *  InspIRCd: (C) 2002-2008 InspIRCd Development Team
6  * See: http://www.inspircd.org/wiki/index.php/Credits
7  *
8  * This program is free but copyrighted software; see
9  *            the file COPYING for details.
10  *
11  * ---------------------------------------------------
12  */
13
14 #include "inspircd.h"
15 #include "commands/cmd_whois.h"
16 #include "commands/cmd_stats.h"
17 #include "socket.h"
18 #include "wildcard.h"
19 #include "xline.h"
20 #include "transport.h"
21 #include "socketengine.h"
22
23 #include "m_spanningtree/main.h"
24 #include "m_spanningtree/utils.h"
25 #include "m_spanningtree/treeserver.h"
26 #include "m_spanningtree/link.h"
27 #include "m_spanningtree/treesocket.h"
28 #include "m_spanningtree/resolvers.h"
29 #include "m_spanningtree/handshaketimer.h"
30
31 /* $ModDep: m_spanningtree/resolvers.h m_spanningtree/main.h m_spanningtree/utils.h m_spanningtree/treeserver.h m_spanningtree/link.h m_spanningtree/treesocket.h m_spanningtree/handshaketimer.h */
32
33 void TreeSocket::WriteLine(std::string line)
34 {
35         Instance->Logs->Log("m_spanningtree",DEBUG, "S[%d] O %s", this->GetFd(), line.c_str());
36         line.append("\r\n");
37         this->Write(line);
38 }
39
40
41 /* Handle ERROR command */
42 bool TreeSocket::Error(std::deque<std::string> &params)
43 {
44         if (params.size() < 1)
45                 return false;
46         this->Instance->SNO->WriteToSnoMask('l',"ERROR from %s: %s",(!InboundServerName.empty() ? InboundServerName.c_str() : myhost.c_str()),params[0].c_str());
47         /* we will return false to cause the socket to close. */
48         return false;
49 }
50
51 void TreeSocket::Split(const std::string &line, std::deque<std::string> &n)
52 {
53         n.clear();
54         irc::tokenstream tokens(line);
55         std::string param;
56         while (tokens.GetToken(param))
57         {
58                 n.push_back(param);
59         }
60         return;
61 }
62
63 bool TreeSocket::ProcessLine(std::string &line)
64 {
65         std::deque<std::string> params;
66         irc::string command;
67         std::string prefix;
68
69         line = line.substr(0, line.find_first_of("\r\n"));
70
71         if (line.empty())
72                 return true;
73
74         Instance->Logs->Log("m_spanningtree",DEBUG, "S[%d] I %s", this->GetFd(), line.c_str());
75
76         this->Split(line.c_str(),params);
77
78         if (params.empty())
79                 return true;
80
81         if ((params[0][0] == ':') && (params.size() > 1))
82         {
83                 prefix = params[0].substr(1);
84                 params.pop_front();
85
86                 if (prefix.empty())
87                 {
88                         this->SendError("BUG (?) Empty prefix recieved.");
89                         return false;
90                 }
91         }
92
93         command = params[0].c_str();
94         params.pop_front();
95
96         switch (this->LinkState)
97         {
98                 TreeServer* Node;
99
100                 case WAIT_AUTH_1:
101                         /*
102                          * State WAIT_AUTH_1:
103                          *  Waiting for SERVER command from remote server. Server initiating
104                          *  the connection sends the first SERVER command, listening server
105                          *  replies with theirs if its happy, then if the initiator is happy,
106                          *  it starts to send its net sync, which starts the merge, otherwise
107                          *  it sends an ERROR.
108                          */
109                         if (command == "PASS")
110                         {
111                                 /*
112                                  * Ignore this silently. Some services packages insist on sending PASS, even
113                                  * when it is not required (i.e. by us). We have to ignore this here, otherwise
114                                  * as it's an unknown command (effectively), it will cause the connection to be
115                                  * closed, which probably isn't what people want. -- w00t
116                                  */
117                         }
118                         else if (command == "SERVER")
119                         {
120                                 return this->Inbound_Server(params);
121                         }
122                         else if (command == "ERROR")
123                         {
124                                 return this->Error(params);
125                         }
126                         else if (command == "USER")
127                         {
128                                 this->SendError("Client connections to this port are prohibited.");
129                                 return false;
130                         }
131                         else if (command == "CAPAB")
132                         {
133                                 return this->Capab(params);
134                         }
135                         else
136                         {
137                                 // XXX ...wtf.
138                                 irc::string error = "Invalid command in negotiation phase: " + command;
139                                 this->SendError(assign(error));
140                                 return false;
141                         }
142                 break;
143                 case WAIT_AUTH_2:
144                         /*
145                          * State WAIT_AUTH_2:
146                          *  We have sent SERVER to the other side of the connection. Now we're waiting for them to start BURST.
147                          *  The other option at this stage of things, of course, is for them to close our connection thanks
148                          *  to invalid credentials.. -- w
149                          */
150                         if (command == "SERVER")
151                         {
152                                 /*
153                                  * Connection is either attempting to re-auth itself (stupid) or sending netburst without sending BURST.
154                                  * Both of these aren't allowable, so block them here. -- w
155                                  */
156                                 this->SendError("You may not re-authenticate or commence netburst without sending BURST.");
157                                 return true;
158                         }
159                         else if (command == "BURST")
160                         {
161                                 if (params.size())
162                                 {
163                                         time_t them = atoi(params[0].c_str());
164                                         time_t delta = them - Instance->Time();
165                                         if ((delta < -600) || (delta > 600))
166                                         {
167                                                 Instance->SNO->WriteToSnoMask('l',"\2ERROR\2: Your clocks are out by %d seconds (this is more than five minutes). Link aborted, \2PLEASE SYNC YOUR CLOCKS!\2",abs((long)delta));
168                                                 SendError("Your clocks are out by "+ConvToStr(abs((long)delta))+" seconds (this is more than five minutes). Link aborted, PLEASE SYNC YOUR CLOCKS!");
169                                                 return false;
170                                         }
171                                         else if ((delta < -30) || (delta > 30))
172                                         {
173                                                 Instance->SNO->WriteToSnoMask('l',"\2WARNING\2: Your clocks are out by %d seconds. Please consider synching your clocks.", abs((long)delta));
174                                         }
175                                 }
176                                 this->LinkState = CONNECTED;
177                                 Link* lnk = Utils->FindLink(InboundServerName);
178
179                                 Node = new TreeServer(this->Utils, this->Instance, InboundServerName, InboundDescription, InboundSID, Utils->TreeRoot, this, lnk ? lnk->Hidden : false);
180
181                                 if (Node->DuplicateID())
182                                 {
183                                         this->SendError("Server ID "+InboundSID+" already exists on the network!");
184                                         this->Instance->SNO->WriteToSnoMask('l',"Server \2"+InboundServerName+"\2 being introduced from \2" + prefix + "\2 denied, server ID already exists on the network. Closing link.");
185                                         return false;
186                                 }
187
188                                 Utils->TreeRoot->AddChild(Node);
189                                 params.clear();
190                                 params.push_back(InboundServerName);
191                                 params.push_back("*");
192                                 params.push_back("1");
193                                 params.push_back(InboundSID);
194                                 params.push_back(":"+InboundDescription);
195                                 Utils->DoOneToAllButSender(Instance->Config->GetSID(),"SERVER",params,InboundServerName);
196                                 Node->bursting = true;
197                                 this->DoBurst(Node);
198                         }
199                         else if (command == "ERROR")
200                         {
201                                 return this->Error(params);
202                         }
203                         else if (command == "CAPAB")
204                         {
205                                 return this->Capab(params);
206                         }
207
208                 break;
209                 case LISTENER:
210                         /*
211                          * This really shouldn't happen.
212                          */
213                         this->SendError("Internal error -- listening socket accepted its own descriptor!!!");
214                         return false;
215                 break;
216                 case CONNECTING:
217                         /*
218                          * State CONNECTING:
219                          *  We're connecting (OUTGOING) to another server. They are in state WAIT_AUTH_1 until they verify
220                          *  our credentials, when they proceed into WAIT_AUTH_2 and send SERVER to us. We then send BURST
221                          *  + our netburst, which will put them into CONNECTED state. -- w
222                          */
223                         if (command == "SERVER")
224                         {
225                                 // Our credentials have been accepted, send netburst. (this puts US into the CONNECTED state)
226                                 return this->Outbound_Reply_Server(params);
227                         }
228                         else if (command == "ERROR")
229                         {
230                                 return this->Error(params);
231                         }
232                         else if (command == "CAPAB")
233                         {
234                                 return this->Capab(params);
235                         }
236                 break;
237                 case CONNECTED:
238                         /*
239                         * State CONNECTED:
240                          *  Credentials have been exchanged, we've gotten their 'BURST' (or sent ours).
241                          *  Anything from here on should be accepted a little more reasonably.
242                          */
243                         if (!prefix.empty())
244                         {
245                                 /*
246                                  * Check for fake direction here, and drop any instances that are found.
247                                  * What is fake direction? Imagine the following server setup:
248                                  *    0AA <-> 0AB <-> 0AC
249                                  * Fake direction would be 0AC sending a message to 0AB claiming to be from
250                                  * 0AA, or something similar. Basically, a message taking a path that *cannot*
251                                  * be correct.
252                                  *
253                                  * When would this be seen?
254                                  * Well, hopefully never. It could be caused by race conditions, bugs, or
255                                  * "miscreant" servers, though, so let's check anyway. -- w
256                                  */
257                                 std::string direction = prefix;
258
259                                 User *t = this->Instance->FindUUID(prefix);
260                                 if (t)
261                                 {
262                                         direction = t->server;
263                                 }
264
265                                 TreeServer* route_back_again = Utils->BestRouteTo(direction);
266                                 if ((!route_back_again) || (route_back_again->GetSocket() != this))
267                                 {
268                                         if (route_back_again)
269                                                 Instance->Logs->Log("m_spanningtree",DEBUG,"Protocol violation: Fake direction in command '%s' from connection '%s'",line.c_str(),this->GetName().c_str());
270                                         return true;
271                                 }
272                                 /* Fix by brain:
273                                  * When there is activity on the socket, reset the ping counter so
274                                  * that we're not wasting bandwidth pinging an active server.
275                                  */
276                                 route_back_again->SetNextPingTime(Instance->Time() + Utils->PingFreq);
277                                 route_back_again->SetPingFlag();
278                         }
279                         else
280                         {
281                                 /*
282                                  * Empty prefix from a server to server link:
283                                  *  This is somewhat bad/naughty, so let's set the prefix
284                                  *  to be the link that we got it from, so we don't break anything. -- w
285                                  */
286                                 TreeServer* n = Utils->FindServer(GetName());
287                                 if (n)
288                                         prefix = n->GetID();
289                                 else
290                                         prefix = GetName();
291                         }
292
293                         /*
294                          * First up, check for any malformed commands (e.g. MODE without a timestamp)
295                          * and rewrite commands where necessary (SVSMODE -> MODE for services). -- w
296                          */
297                         if (command == "SVSMODE") // This isn't in an "else if" so we still force FMODE for changes on channels.
298                                 command = "MODE";
299
300                         if (command == "MODE")
301                         {
302                                 if (params.size() >= 2)
303                                 {
304                                         Channel* channel = Instance->FindChan(params[0]);
305                                         if (channel)
306                                         {
307                                                 this->SendError("MODE may no longer be used on channels. Please use FMODE, with correct timestamp rules.");
308                                                 return false;
309                                         }
310                                 }
311                         }
312
313
314                         /*
315                          * Now, check for (and parse) commands as appropriate. -- w
316                          */
317
318                         /* Find the server that this command originated from, used in the handlers below */
319                         TreeServer *ServerSource = Utils->FindServer(prefix);
320
321                         /* Find the link we just got this from so we don't bounce it back incorrectly */
322                         std::string sourceserv = this->myhost;
323                         if (!this->InboundServerName.empty())
324                         {
325                                 sourceserv = this->InboundServerName;
326                         }
327
328                         /*
329                          * XXX one of these days, this needs to be moved into class Commands.
330                          */
331                         if (command == "UID")
332                         {
333                                 return this->ParseUID(prefix, params);
334                         }
335                         else if (command == "FJOIN")
336                         {
337                                 return this->ForceJoin(prefix,params);
338                         }
339                         else if ((command == "NOTICE" || command == "PRIVMSG") && (Utils->IsServer(prefix)))
340                         {
341                                 return this->ServerMessage(assign(command), prefix, params, sourceserv);
342                         }
343                         else if (command == "STATS")
344                         {
345                                 return this->Stats(prefix, params);
346                         }
347                         else if (command == "MOTD")
348                         {
349                                 return this->Motd(prefix, params);
350                         }
351                         else if (command == "KILL" && ServerSource)
352                         {
353                                 // Kill from a server
354                                 return this->RemoteKill(prefix,params);
355                         }
356                         else if (command == "MODULES")
357                         {
358                                 return this->Modules(prefix, params);
359                         }
360                         else if (command == "ADMIN")
361                         {
362                                 return this->Admin(prefix, params);
363                         }
364                         else if (command == "MAP")
365                         {
366                                 User* user = Instance->FindNick(prefix);
367                                 if (user)
368                                 {
369                                         std::vector<std::string> p(params.begin(), params.end());
370                                         return Utils->Creator->HandleMap(p, user);
371                                 }
372                         }
373                         else if (command == "SERVER")
374                         {
375                                 return this->RemoteServer(prefix,params);
376                         }
377                         else if (command == "ERROR")
378                         {
379                                 return this->Error(params);
380                         }
381                         else if (command == "OPERTYPE")
382                         {
383                                 return this->OperType(prefix,params);
384                         }
385                         else if (command == "FMODE")
386                         {
387                                 return this->ForceMode(prefix,params);
388                         }
389                         else if (command == "FTOPIC")
390                         {
391                                 return this->ForceTopic(prefix,params);
392                         }
393                         else if (command == "METADATA")
394                         {
395                                 return this->MetaData(prefix,params);
396                         }
397                         else if (command == "PING")
398                         {
399                                 return this->LocalPing(prefix,params);
400                         }
401                         else if (command == "PONG")
402                         {
403                                 return this->LocalPong(prefix,params);
404                         }
405                         else if (command == "VERSION")
406                         {
407                                 return this->ServerVersion(prefix,params);
408                         }
409                         else if (command == "FHOST")
410                         {
411                                 return this->ChangeHost(prefix,params);
412                         }
413                         else if (command == "FNAME")
414                         {
415                                 return this->ChangeName(prefix,params);
416                         }
417                         else if (command == "ADDLINE")
418                         {
419                                 return this->AddLine(prefix,params);
420                         }
421                         else if (command == "DELLINE")
422                         {
423                                 return this->DelLine(prefix,params);
424                         }
425                         else if (command == "SVSNICK")
426                         {
427                                 return this->ForceNick(prefix,params);
428                         }
429                         else if (command == "OPERQUIT")
430                         {
431                                 return this->OperQuit(prefix,params);
432                         }
433                         else if (command == "IDLE")
434                         {
435                                 return this->Whois(prefix,params);
436                         }
437                         else if (command == "PUSH")
438                         {
439                                 return this->Push(prefix,params);
440                         }
441                         else if (command == "TIME")
442                         {
443                                 return this->Time(prefix,params);
444                         }
445                         else if ((command == "KICK") && (Utils->IsServer(prefix)))
446                         {
447                                 if (params.size() == 3)
448                                 {
449                                         TreeServer* pf = Utils->FindServer(prefix);
450                                         User* user = this->Instance->FindNick(params[1]);
451                                         Channel* chan = this->Instance->FindChan(params[0]);
452                                         if (pf && user && chan)
453                                         {
454                                                 if (!chan->ServerKickUser(user, params[2].c_str(), false, pf->GetName().c_str()))
455                                                         /* Yikes, the channels gone! */
456                                                         delete chan;
457                                         }
458                                 }
459
460                                 return Utils->DoOneToAllButSenderRaw(line,sourceserv,prefix,command,params);
461                         }
462                         else if (command == "SVSJOIN")
463                         {
464                                 return this->ServiceJoin(prefix,params);
465                         }
466                         else if (command == "SVSPART")
467                         {
468                                 return this->ServicePart(prefix,params);
469                         }
470                         else if (command == "SQUIT")
471                         {
472                                 if (params.size() == 2)
473                                 {
474                                         this->Squit(Utils->FindServer(params[0]),params[1]);
475                                 }
476                                 return true;
477                         }
478                         else if (command == "MODENOTICE")
479                         {
480                                 if (params.size() >= 2)
481                                 {
482                                         if (ServerSource)
483                                                 Instance->Users->WriteMode(params[0].c_str(), WM_AND, "*** From %s: %s", (ServerSource ? ServerSource->GetName().c_str() : prefix.c_str()), params[1].c_str());
484                                 }
485                                 return Utils->DoOneToAllButSenderRaw(line, sourceserv, prefix, command, params);
486                         }
487                         else if (command == "SNONOTICE")
488                         {
489                                 if (params.size() >= 2)
490                                 {
491                                         Instance->SNO->WriteToSnoMask(*(params[0].c_str()), "From " + (ServerSource ? ServerSource->GetName().c_str() : prefix) + ": "+ params[1]);
492                                         return Utils->DoOneToAllButSenderRaw(line, sourceserv, prefix, command, params);
493                                 }
494
495                         }
496                         else if (command == "BURST")
497                         {
498                                 // Set prefix server as bursting
499                                 if (!ServerSource)
500                                 {
501                                         this->Instance->SNO->WriteToSnoMask('l', "WTF: Got BURST from a nonexistant server(?): %s", (ServerSource ? ServerSource->GetName().c_str() : prefix.c_str()));
502                                         return false;
503                                 }
504
505                                 ServerSource->bursting = true;
506                                 return Utils->DoOneToAllButSenderRaw(line, sourceserv, prefix, command, params);
507                         }
508                         else if (command == "ENDBURST")
509                         {
510                                 if (!ServerSource)
511                                 {
512                                         this->Instance->SNO->WriteToSnoMask('l', "WTF: Got ENDBURST from a nonexistant server(?): %s", (ServerSource ? ServerSource->GetName().c_str() : prefix.c_str()));
513                                         return false;
514                                 }
515
516                                 ServerSource->FinishBurst();
517                                 return Utils->DoOneToAllButSenderRaw(line, sourceserv, prefix, command, params);
518                         }
519                         else if (command == "ENCAP")
520                         {
521                                 return this->Encap(prefix, params);
522                         }
523                         else if (command == "MODE")
524                         {
525                                 // Server-prefix MODE.
526                                 std::vector<std::string> modelist(params.begin(), params.end());
527
528                                 /* We don't support this for channel mode changes any more! */
529                                 if (params.size() >= 1)
530                                 {
531                                         if (Instance->FindChan(params[0]))
532                                         {
533                                                 this->SendError("Protocol violation by '"+(ServerSource ? ServerSource->GetName().c_str() : prefix)+"'! MODE for channel mode changes is not supported by the InspIRCd 1.2 protocol. You must use FMODE to preserve channel timestamps.");
534                                                 return false;
535                                         }
536                                 }
537
538                                 // Insert into the parser
539                                 this->Instance->SendMode(modelist, this->Instance->FakeClient);
540
541                                 // Pass out to the network
542                                 return Utils->DoOneToAllButSenderRaw(line,sourceserv,prefix,command,params);
543                         }
544                         else
545                         {
546                                 /*
547                                  * Not a special s2s command. Emulate the user doing it.
548                                  * This saves us having a huge ugly command parser again.
549                                  */
550                                 User *who = this->Instance->FindUUID(prefix);
551
552                                 if (!who)
553                                 {
554                                         // this looks ugly because command is an irc::string
555                                         this->SendError("Command (" + std::string(command.c_str()) + ") from unknown prefix (" + prefix + ")! Dropping link.");
556                                         return false;
557                                 }
558
559                                 if (command == "NICK")
560                                 {
561                                         if (params.size() != 2)
562                                         {
563                                                 SendError("Protocol violation: NICK message without TS - :"+std::string(who->uuid)+" NICK "+params[0]);
564                                                 return false;
565                                         }
566                                         /* Update timestamp on user when they change nicks */
567                                         who->age = atoi(params[1].c_str());
568
569                                         /*
570                                          * On nick messages, check that the nick doesnt already exist here.
571                                          * If it does, perform collision logic.
572                                          */
573                                         User* x = this->Instance->FindNickOnly(params[0]);
574                                         if ((x) && (x != who))
575                                         {
576                                                 int collideret = 0;
577                                                 /* x is local, who is remote */
578                                                 collideret = this->DoCollision(x, who->age, who->ident, who->GetIPString(), who->uuid);
579                                                 if (collideret != 1)
580                                                 {
581                                                         /*
582                                                          * Remote client lost, or both lost, parsing this nickchange would be
583                                                          * pointless, as the incoming client's server will soon recieve SVSNICK to
584                                                          * change its nick to its UID. :) -- w00t
585                                                          */
586                                                         return true;
587                                                 }
588                                         }
589                                 }
590
591                                 // its a user
592                                 std::vector<std::string> strparams(params.begin(), params.end());
593
594                                 switch (this->Instance->CallCommandHandler(command.c_str(), strparams, who))
595                                 {
596                                         case CMD_INVALID:
597                                                 /*
598                                                  * XXX: command is irc::string, hence ugliness
599                                                  */
600                                                 this->SendError("Unrecognised or malformed command '" + std::string(command.c_str()) + "' -- possibly loaded mismatched modules");
601                                                 return false;
602                                                 break;
603                                         case CMD_FAILURE:
604                                                 /*
605                                                  * CMD_LOCALONLY is aliased to CMD_FAILURE, so this won't go out onto the network.
606                                                  */
607                                                 return true;
608                                                 break;
609                                         default:
610                                                 /* CMD_SUCCESS falls through here */
611                                                 break;
612                                 }
613
614                                 return Utils->DoOneToAllButSenderRaw(line,sourceserv,prefix,command,params);
615
616                         }
617                         return true;
618                         break; // end of state CONNECTED (phew).
619         }
620         return true;
621 }
622
623 std::string TreeSocket::GetName()
624 {
625         std::string sourceserv = this->myhost;
626         if (!this->InboundServerName.empty())
627         {
628                 sourceserv = this->InboundServerName;
629         }
630         return sourceserv;
631 }
632
633 void TreeSocket::OnTimeout()
634 {
635         if (this->LinkState == CONNECTING)
636         {
637                 Utils->Creator->RemoteMessage(NULL, "CONNECT: Connection to \002%s\002 timed out.", myhost.c_str());
638                 Link* MyLink = Utils->FindLink(myhost);
639                 if (MyLink)
640                         Utils->DoFailOver(MyLink);
641         }
642 }
643
644 void TreeSocket::OnClose()
645 {
646         // Test fix for big fuckup
647         if (this->LinkState != CONNECTED)
648                 return;
649
650         // Connection closed.
651         // If the connection is fully up (state CONNECTED)
652         // then propogate a netsplit to all peers.
653         std::string quitserver = this->myhost;
654         if (!this->InboundServerName.empty())
655         {
656                 quitserver = this->InboundServerName;
657         }
658         TreeServer* s = Utils->FindServer(quitserver);
659         if (s)
660         {
661                 Squit(s,"Remote host closed the connection");
662         }
663
664         if (!quitserver.empty())
665         {
666                 Utils->Creator->RemoteMessage(NULL,"Connection to '\2%s\2' failed.",quitserver.c_str());
667                 time_t server_uptime = Instance->Time() - this->age;
668                 if (server_uptime)
669                         Utils->Creator->RemoteMessage(NULL,"Connection to '\2%s\2' was established for %s", quitserver.c_str(), Utils->Creator->TimeToStr(server_uptime).c_str());
670         }
671 }
672
673 int TreeSocket::OnIncomingConnection(int newsock, char* ip)
674 {
675         bool found = false;
676
677         found = (std::find(Utils->ValidIPs.begin(), Utils->ValidIPs.end(), ip) != Utils->ValidIPs.end());
678         if (!found)
679         {
680                 for (std::vector<std::string>::iterator i = Utils->ValidIPs.begin(); i != Utils->ValidIPs.end(); i++)
681                 {
682                         if (*i == "*" || irc::sockets::MatchCIDR(ip, *i))
683                         {
684                                 found = true;
685                                 break;
686                         }
687                 }
688
689                 if (!found)
690                 {
691                         Utils->Creator->RemoteMessage(NULL,"Server connection from %s denied (no link blocks with that IP address)", ip);
692                         Instance->SE->Close(newsock);
693                         return false;
694                 }
695         }
696
697         TreeSocket* s = new TreeSocket(this->Utils, this->Instance, newsock, ip, this->Hook);
698         s = s; /* Whinge whinge whinge, thats all GCC ever does. */
699         return true;
700 }