cpp/openttd-patchpack/source Changeset - r14625:3de0713d85fd

Changeset - r14625:3de0713d85fd

Parent rev.

Child rev.

[Not reviewed]

master

0 3 0

yexo - 14 years ago 2010-02-22 20:18:59
yexo@openttd.org

(svn r19212) -Fix: base graphics set with an empty filename in the obg file caused segfaults

3 files changed with 5 insertions and 4 deletions:

src/base_media_base.h

src/base_media_func.h

src/gfxinit.cpp

0 comments (0 inline, 0 general)

src/base_media_base.h

➞

Show inline comments

@@ @@ -108,9 +108,10 @@ struct BaseSet { @@
 	 * Read the set information from a loaded ini.
 	 * @param ini      the ini to read from
 	 * @param path     the path to this ini file (for filenames)
 	 * @param allow_empty_filename empty filenames are valid
 	 * @return true if loading was successful.
 	 */
 	bool FillSetDetails(IniFile *ini, const char *path);
+	bool FillSetDetails(IniFile *ini, const char *path, bool allow_empty_filename = true);
 	/**
 	 * Get the description for the given ISO code.

src/base_media_func.h

➞

Show inline comments

@@ @@ -30,7 +30,7 @@ template <class Tbase_set> /* static */ @@
+	}
 template <class T, size_t Tnum_files, Subdirectory Tsubdir>
 bool BaseSet<T, Tnum_files, Tsubdir>::FillSetDetails(IniFile *ini, const char *path)
+bool BaseSet<T, Tnum_files, Tsubdir>::FillSetDetails(IniFile *ini, const char *path, bool allow_empty_filename)
+{
 	memset(this, 0, sizeof(*this));
@@ @@ -69,7 +69,7 @@ bool BaseSet<T, Tnum_files, Tsubdir>::Fi @@
 		MD5File *file = &this->files[i];
 		/* Find the filename first. */
 		item = files->GetItem(BaseSet<T, Tnum_files, Tsubdir>::file_names[i], false);
 		if (item == NULL) {
+		if (item == NULL || (item->value == NULL && !allow_empty_filename)) {
 			DEBUG(grf, 0, "No " SET_TYPE " file for: %s", BaseSet<T, Tnum_files, Tsubdir>::file_names[i]);
 			return false;
+		}

src/gfxinit.cpp

➞

Show inline comments

@@ @@ -205,7 +205,7 @@ void GfxLoadSprites() @@
 bool GraphicsSet::FillSetDetails(IniFile *ini, const char *path)
+{
 	bool ret = this->BaseSet<GraphicsSet, MAX_GFT, DATA_DIR>::FillSetDetails(ini, path);
+	bool ret = this->BaseSet<GraphicsSet, MAX_GFT, DATA_DIR>::FillSetDetails(ini, path, false);
 	if (ret) {
 		IniGroup *metadata = ini->GetGroup("metadata");
 		IniItem *item;

0 comments (0 inline, 0 general)