mirrors/PocketMine-MP
1
0
Fork 0
mirror of https://github.com/pmmp/PocketMine-MP.git synced 2025-09-04 17:06:16 +00:00
Code Issues Projects Releases Wiki Activity

Player: duct tape for yet another login sequence bug in the shitty net architecture

Browse Source
This commit is contained in:
Dylan K. Taylor
2020-05-13 12:59:43 +01:00
parent a70fa15690
commit b044550475
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/pocketmine/Player.php
View File

@ -261,6 +261,8 @@ class Player extends Human implements CommandSender, ChunkLoader, IPlayer{
/** @var bool */ /** @var bool */
public $loggedIn = false; public $loggedIn = false;
/** @var bool */
private $seenLoginPacket = false;
/** @var bool */ /** @var bool */
private $resourcePacksDone = false; private $resourcePacksDone = false;
@ -1816,9 +1818,10 @@ class Player extends Human implements CommandSender, ChunkLoader, IPlayer{
} }
public function handleLogin(LoginPacket $packet) : bool{ public function handleLogin(LoginPacket $packet) : bool{
if($this->loggedIn){ if($this->seenLoginPacket){
return false; return false;
} }
$this->seenLoginPacket = true;
if($packet->protocol !== ProtocolInfo::CURRENT_PROTOCOL){ if($packet->protocol !== ProtocolInfo::CURRENT_PROTOCOL){
if($packet->protocol < ProtocolInfo::CURRENT_PROTOCOL){ if($packet->protocol < ProtocolInfo::CURRENT_PROTOCOL){