updated for version 7.3.428

Problem:    Win32: an xpm file without a mask crashes Vim.
Solution:   Fail when the mask is missing. (Dave Bodenstab)

diff --git a/src/version.c b/src/version.c
index 6732c887c5f08e4bc8f2072deb07bf846a8440de..6ba69ea03d8774be2dfd095ab13d7ff2e0a2b4ff 100644 (file)
--- a/src/version.c
+++ b/src/version.c
@@ -714,6 +714,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    428,
 /**/
     427,
 /**/

diff --git a/src/xpm_w32.c b/src/xpm_w32.c
index 45765e1f1535fedea448f9135a3c8838abd8d2e3..2ec32ed9f03aaf9d44168b9c356a703ffc06d6e4 100644 (file)
--- a/src/xpm_w32.c
+++ b/src/xpm_w32.c
@@ -1,4 +1,5 @@
-/*
+/* vi:set ts=8 sts=4 sw=4:
+ *
  * Load XPM image.
  *
  * This function is placed in separate file because Xpm headers conflict with
@@ -30,9 +31,10 @@
 #include "xpm.h"
 
 /*
- * Tries to load Xpm image from file 'filename'.
- * If fails return -1.
- * success - 0 and image and mask BITMAPS
+ * Tries to load an Xpm image from the file "filename".
+ * Returns -1 on failure.
+ * Returns 0 on success and stores image and mask BITMAPS in "hImage" and
+ * "hShape".
  */
     int
 LoadXpmImage(filename, hImage, hShape)
@@ -40,7 +42,7 @@ LoadXpmImage(filename, hImage, hShape)
     HBITMAP *hImage;
     HBITMAP *hShape;
 {
-    XImage         *img;   /* loaded image */
+    XImage         *img;  /* loaded image */
     XImage         *shp;  /* shapeimage */
     XpmAttributes   attr;
     int                    res;
@@ -51,10 +53,13 @@ LoadXpmImage(filename, hImage, hShape)
     DeleteDC(hdc);
     if (res < 0)
        return -1;
-    else
+    if (shp == NULL)
     {
-       *hImage = img->bitmap;
-       *hShape = shp->bitmap;
-       return 0;
+        if (img)
+           XDestroyImage(img);
+       return -1;
     }
+    *hImage = img->bitmap;
+    *hShape = shp->bitmap;
+    return 0;
 }