X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/6dab3ba7e5201f506a841cf1442ddf0ef00b1160..e2f9212c00094f3032511bc7d42efc7c1b91134b:/src/common/xpmdecod.cpp?ds=inline diff --git a/src/common/xpmdecod.cpp b/src/common/xpmdecod.cpp index 28d81ca4a3..434e8dedf9 100644 --- a/src/common/xpmdecod.cpp +++ b/src/common/xpmdecod.cpp @@ -105,6 +105,7 @@ license is as follows: # include "wx/defs.h" #endif +#if wxUSE_IMAGE && wxUSE_XPM #include "wx/stream.h" #include "wx/image.h" @@ -136,7 +137,7 @@ wxImage wxXPMDecoder::ReadFile(wxInputStream& stream) size_t i; if ( stream.Read(xpm_buffer, length).LastError() == wxSTREAM_READ_ERROR ) - return FALSE; + return wxNullImage; /* * Remove comments from the file: @@ -202,6 +203,12 @@ wxImage wxXPMDecoder::ReadFile(wxInputStream& stream) lines_cnt++; } + if ( !lines_cnt ) + { + // this doesn't really look an XPM image + return wxNullImage; + } + xpm_lines = new const char*[lines_cnt]; xpm_lines[0] = xpm_buffer; line = 1; @@ -522,12 +529,14 @@ static bool GetRGBFromName(const char *inname, bool *isNone, char *name; char *grey, *p; - // #rrggbb are not in database, we parse them directly - if ( *inname == '#' && strlen(inname) == 7 ) + // Neither #rrggbb nor #rrrrggggbbbb are in database, we parse them directly + size_t inname_len = strlen(inname); + if ( *inname == '#' && (inname_len == 7 || inname_len == 13)) { + size_t ofs = (inname_len == 7) ? 2 : 4; *r = ParseHexadecimal(inname[1], inname[2]); - *g = ParseHexadecimal(inname[3], inname[4]); - *b = ParseHexadecimal(inname[5], inname[6]); + *g = ParseHexadecimal(inname[1*ofs+1], inname[1*ofs+2]); + *b = ParseHexadecimal(inname[2*ofs+1], inname[2*ofs+2]); *isNone = FALSE; return TRUE; } @@ -751,4 +760,4 @@ wxImage wxXPMDecoder::ReadData(const char **xpm_data) return img; } - +#endif // wxUSE_IMAGE && wxUSE_XPM