Core/NetworkIO: Fix performance loss when handling bad packets

Revert f9cb4280bd8f1c29e4acbae5339bce40c3b52bcb which retrieved the callstack each time a ByteBufferPositionException or a ByteBufferSourceException occurred, for example with bad packet structure.
Core/QuestPOI: Mitigate possible DoS with CMSG_QUEST_POI_QUERY

Avoid sending POIs for same quest if the client somehow sent duplicates quest id in same CMSG_QUEST_POI_QUERY packet. This also reduce the effects of possible DoS and increases the difficulty to cause it.
Fix a typo which caused no quest POIs to be sent at all if the client queried data for 25 quests.
Core/Misc: Code cleanup

Remove old unused opcode handler WorldSession::HandlePageQuerySkippedOpcode() for CMSG_PAGE_TEXT_QUERY since it's already handled by WorldSession::HandlePageTextQueryOpcode()
Core/Misc: Added utility method for checking is two sorted containers have a common element
Scripts: Remove unused variables (kept as comment)
Script/Shadowmoon Valley: Remove hack in npc_shadowmoon_tuber_node.

This script will be converted to SAI (if possible) later.
Core/Misc: Kick player in case of multiple CMSG_PLAYER_LOGIN

Kick player in addition to logging an error in case of multiple CMSG_PLAYER_LOGIN.
Merge pull request #1 from TrinityCore/master

actualización del repo