diff options
author | Freeyorp <Freeyorp101@hotmail.com> | 2009-12-19 20:12:06 +1300 |
---|---|---|
committer | Freeyorp <Freeyorp101@hotmail.com> | 2009-12-19 20:12:06 +1300 |
commit | 2968691478f6479e98377ca500a0da69bc0f392a (patch) | |
tree | db22141937b53751369ccf83eee1226c9960a96c | |
parent | 6ad0ab8123de1ee3f27ad38300dc7ef896bab569 (diff) | |
download | mana-client-2968691478f6479e98377ca500a0da69bc0f392a.tar.gz mana-client-2968691478f6479e98377ca500a0da69bc0f392a.tar.bz2 mana-client-2968691478f6479e98377ca500a0da69bc0f392a.tar.xz mana-client-2968691478f6479e98377ca500a0da69bc0f392a.zip |
Fix a race condition
If the server tells the client to update the mp bar when handling an matk
change message before the status window has initialised, a segfault
occurs
-rw-r--r-- | src/gui/statuswindow.cpp | 4 | ||||
-rw-r--r-- | src/net/ea/playerhandler.cpp | 8 |
2 files changed, 9 insertions, 3 deletions
diff --git a/src/gui/statuswindow.cpp b/src/gui/statuswindow.cpp index 94c95f68..4db8c751 100644 --- a/src/gui/statuswindow.cpp +++ b/src/gui/statuswindow.cpp @@ -112,7 +112,9 @@ StatusWindow::StatusWindow(): mMpLabel = new Label(_("MP:")); mMpBar = new ProgressBar((float) player_node->getMaxMP() / (float) player_node->getMaxMP(), - 80, 15, gcn::Color(26, 102, 230)); + 80, 15, Net::getPlayerHandler()->canUseMagic() ? + gcn::Color(26, 102, 230) : + gcn::Color(100, 100, 100)); place(0, 0, mLvlLabel, 3); // 5, 0 Job Level diff --git a/src/net/ea/playerhandler.cpp b/src/net/ea/playerhandler.cpp index 5db4acbe..0592bb55 100644 --- a/src/net/ea/playerhandler.cpp +++ b/src/net/ea/playerhandler.cpp @@ -260,10 +260,14 @@ void PlayerHandler::handleMessage(Net::MessageIn &msg) case 0x002b: player_node->setAttributeEffective(MATK, value + ATTR_BONUS(MATK)); player_node->setAttributeBase(MATK, value); - statusWindow->update(StatusWindow::MP); break; + if (statusWindow) + statusWindow->update(StatusWindow::MP); + break; case 0x002c: value += player_node->getAttributeBase(MATK); player_node->setAttributeEffective(MATK, value); - statusWindow->update(StatusWindow::MP); break; + if (statusWindow) + statusWindow->update(StatusWindow::MP); + break; case 0x002d: player_node->setAttributeEffective(DEF, value + ATTR_BONUS(DEF)); player_node->setAttributeBase(DEF, value); break; |