$NetBSD: patch-aq,v 1.4 2007/11/29 18:36:40 wiz Exp $ --- xmms/bmp.c.orig 2006-07-16 13:40:04.000000000 +0000 +++ xmms/bmp.c @@ -19,6 +19,12 @@ */ #include "xmms.h" +#if HAVE_STDINT_H +#include +#elif !defined(UINT32_MAX) +#define UINT32_MAX 0xffffffffU +#endif + struct rgb_quad { guchar rgbBlue; @@ -183,7 +189,7 @@ GdkPixmap *read_bmp(gchar * filename) } else if (bitcount != 24 && bitcount != 16 && bitcount != 32) { - gint ncols, i; + guint32 ncols, i; ncols = offset - headSize - 14; if (headSize == 12) @@ -203,6 +209,13 @@ GdkPixmap *read_bmp(gchar * filename) fseek(file, offset, SEEK_SET); buffer = g_malloc(imgsize); fread(buffer, imgsize, 1, file); + /* verify buffer size */ + if (!h || !w || + w > (((UINT32_MAX - 3) / 3) / h) || + h > (((UINT32_MAX - 3) / 3) / w)) { + g_warning("read_bmp(): width(%u)*height(%u) too large", w, h); + goto failure; + } data = g_malloc0((w * 3 * h) + 3); /* +3 is just for safety */ if (bitcount == 1)