Bug 794949 - Plugin crash when opening png, jpeg or tiff with...

... non-latin unicode path.
g_win32_locale_filename_from_utf8() was sometimes returning NULL for
some paths on Windows. Then the call to gexiv2_metadata_open_path() with
a NULL value was crashing plug-ins.
This commit only prevents from crashing by simply failing to load
metadata when this occurs, which means losing metadata support on
Windows depending on filenames. A proper solution will have to be
implemented.
This commit is contained in:
Jehan
2018-04-05 00:12:00 +02:00
parent cb239e60f6
commit ba06a0fe86

View File

@ -848,6 +848,17 @@ gimp_metadata_load_from_file (GFile *file,
#ifdef G_OS_WIN32
filename = g_win32_locale_filename_from_utf8 (path);
/* FIXME!
* This call can return NULL, which later crashes the call to
* gexiv2_metadata_open_path().
* See bug 794949.
*/
if (! filename)
{
g_set_error (error, GIMP_METADATA_ERROR, 0,
_("Conversion of the filename to system codepage failed."));
return NULL;
}
#else
filename = g_strdup (path);
#endif