Fix linking problem if only using link::outgoing (and not link::incoming)

which is perfectly legal but caused a confusing error message about
a 'server name mismatch'.

diff --git a/doc/RELEASE-NOTES b/doc/RELEASE-NOTES
index f56a69c67f868c0121d6ff2261b8d8af5ffac235..5f88ae3453f93ef0d9069ce2ad05a11fbe8b3245 100644 (file)
--- a/doc/RELEASE-NOTES
+++ b/doc/RELEASE-NOTES
@@ -82,6 +82,7 @@ Major issues fixed:
 
 Minor issues fixed:
 * set::restrict-extendedbans was not effective for stacked bans
+* linking problem if only using link::outgoing
 
 Other:
 * UnrealIRCd will no longer give user mode +z to users on WEBIRC

diff --git a/src/modules/m_server.c b/src/modules/m_server.c
index 7d40ae30c0e667b0bf023dba2f406d483c371106..128092bafcd123333a028a58dcbdfba8b61ca03b 100644 (file)
--- a/src/modules/m_server.c
+++ b/src/modules/m_server.c
@@ -263,9 +263,7 @@ int _verify_link(aClient *cptr, aClient *sptr, char *servername, ConfigItem_link
                 */
 
                /* Actually we still need to double check the servername to avoid confusion. */
-               link = Find_link(servername, cptr);
-
-               if (!link || strcasecmp(link->servername, cptr->serv->conf->servername))
+               if (strcasecmp(servername, cptr->serv->conf->servername))
                {
                        ircsnprintf(xerrmsg, sizeof(xerrmsg), "Outgoing connect from link block '%s' but server "
                                "introduced himself as '%s'. Server name mismatch.",