cpp/openttd-patchpack/source Changeset - r14514:69262db8f571

Changeset - r14514:69262db8f571

Parent rev.

Child rev.

[Not reviewed]

master

0 2 0

yexo - 14 years ago 2010-02-11 15:08:55
yexo@openttd.org

(svn r19095) -Fix: segfault when a baseset has an empty metadata field

2 files changed with 2 insertions and 13 deletions:

src/base_media_func.h

src/music.cpp

0 comments (0 inline, 0 general)

src/base_media_func.h

➞

Show inline comments

@@ @@ -24,7 +24,7 @@ template <class Tbase_set> /* static */ @@
  */
 #define fetch_metadata(name) \
 	item = metadata->GetItem(name, false); \
-	if (item == NULL || strlen(item->value) == 0) { \
+	if (item == NULL || StrEmpty(item->value)) { \
 		DEBUG(grf, 0, "Base " SET_TYPE "set detail loading: %s field missing", name); \
 		return false; \
+	}

src/music.cpp

➞

Show inline comments

@@ @@ -56,17 +56,6 @@ template <class Tbase_set> @@
 	return BaseMedia<Tbase_set>::used_set != NULL;
+}
 /**
  * Try to read a single piece of metadata and return false if it doesn't exist.
  * @param name the name of the item to fetch.
  */
 #define fetch_name(name) \
 	item = metadata->GetItem(name, false); \
 	if (item == NULL || strlen(item->value) == 0) { \
 		DEBUG(grf, 0, "Base " SET_TYPE "set detail loading: %s field missing", name); \
 		return false; \
+	}
 bool MusicSet::FillSetDetails(IniFile *ini, const char *path)
+{
 	bool ret = this->BaseSet<MusicSet, NUM_SONGS_AVAILABLE, GM_DIR>::FillSetDetails(ini, path);
@@ @@ -81,7 +70,7 @@ bool MusicSet::FillSetDetails(IniFile *i @@
+			}
 			IniItem *item = names->GetItem(filename, false);
-			if (item == NULL || strlen(item->value) == 0) {
+			if (item == NULL || StrEmpty(item->value)) {
 				DEBUG(grf, 0, "Base music set song name missing: %s", filename);
 				return false;
+			}

0 comments (0 inline, 0 general)