Merged another fix from TMW for the new skill dialogue, which makes it so that if a skill is unknown, it doesn't

crash the client (it doesn't print garbage, like fate claims).

1 files changed, 20 insertions, 4 deletions

diff --git a/src/gui/skill.cpp b/src/gui/skill.cpp
index 32c961bd..4296c506 100644
--- a/src/gui/skill.cpp
+++ b/src/gui/skill.cpp
@@ -80,14 +80,21 @@ public:
     virtual void
     update(void)
     {
+	static const SkillInfo fakeSkillInfo = { "???", false };
+
         mEntriesNr = mDialog->getSkills().size();
         resize();
 
         for (int i = 0; i < mEntriesNr; i++) {
             SKILL *skill = mDialog->getSkills()[i];
-            SkillInfo *info = &skill_db[skill->id];
+            SkillInfo const *info;
             char tmp[128];
 
+            if (skill->id >= 0 && (unsigned int) skill->id < skill_db.size())
+                info = &skill_db[skill->id];
+            else
+                info = &fakeSkillInfo;
+
             sprintf(tmp, "%c%s", info->modifiable? ' ' : '*', info->name.c_str());
             gcn::Label *name_label = new gcn::Label(std::string(tmp));
 
@@ -188,9 +195,18 @@ void SkillDialog::update()
 
     int selectedSkill = mTable.getSelectedRow();
 
-    mIncButton->setEnabled(selectedSkill > -1
-                           && skill_db[mSkillList[selectedSkill]->id].modifiable
-                           && player_node->mSkillPoint > 0);
+    if (selectedSkill >= 0) {
+        int skillId = mSkillList[selectedSkill]->id;
+        bool modifiable;
+ 	 
+        if (skillId >= 0 && (unsigned int) skillId < skill_db.size())
+            modifiable = skill_db[skillId].modifiable;
+        else
+            modifiable = false;
+
+        mIncButton->setEnabled(modifiable && player_node->mSkillPoint > 0);
+    } else
+        mIncButton->setEnabled(false);
 
     mTableModel->update();
 }