fix for crash scaling an image
authorCampbell Barton <ideasman42@gmail.com>
Fri, 29 Jun 2012 10:52:37 +0000 (10:52 +0000)
committerCampbell Barton <ideasman42@gmail.com>
Fri, 29 Jun 2012 10:52:37 +0000 (10:52 +0000)
source/blender/blenkernel/BKE_image.h
source/blender/blenkernel/intern/image.c
source/blender/makesrna/intern/rna_image_api.c

index e2263a5edb72e10b10f16def54f9dd49ad536e3b..699eb0a9f02fea5a8f5ff490cfcc551c45abdd55 100644 (file)
@@ -191,7 +191,7 @@ struct Image *BKE_image_copy(struct Image *ima);
 void BKE_image_merge(struct Image *dest, struct Image *source);
 
 /* scale the image */
-void BKE_image_scale(struct Image *image, int width, int height);
+int BKE_image_scale(struct Image *image, int width, int height);
 
 /* check if texture has alpha (depth=32) */
 int BKE_image_has_alpha(struct Image *image);
index d1fa7ecefeb02881dd897215be522bd0705b725c..beaf8f719e3587afe1fbdc16c7e4f0f4c84e7c82 100644 (file)
@@ -514,17 +514,21 @@ void BKE_image_merge(Image *dest, Image *source)
 }
 
 /* note, we could be clever and scale all imbuf's but since some are mipmaps its not so simple */
-void BKE_image_scale(Image *image, int width, int height)
+int BKE_image_scale(Image *image, int width, int height)
 {
        ImBuf *ibuf;
        void *lock;
 
        ibuf = BKE_image_acquire_ibuf(image, NULL, &lock);
 
-       IMB_scaleImBuf(ibuf, width, height);
-       ibuf->userflags |= IB_BITMAPDIRTY;
+       if (ibuf) {
+               IMB_scaleImBuf(ibuf, width, height);
+               ibuf->userflags |= IB_BITMAPDIRTY;
+       }
 
        BKE_image_release_ibuf(image, lock);
+
+       return (ibuf != NULL);
 }
 
 Image *BKE_image_load(const char *filepath)
index 14d01444681d5d65938bcd9e97467e26f2332402..14c9829b951384e1ccf3abec0e89643c23142487 100644 (file)
@@ -178,9 +178,11 @@ static void rna_Image_update(Image *image, ReportList *reports)
        IMB_rect_from_float(ibuf);
 }
 
-static void rna_Image_scale(Image *image, int width, int height)
+static void rna_Image_scale(Image *image, int width, int height, , ReportList *reports)
 {
-       BKE_image_scale(image, width, height);
+       if (!BKE_image_scale(image, width, height)) {
+               BKE_reportf(reports, RPT_ERROR, "Image \"%s\" does not have any image data", image->id.name + 2);
+       }
 }
 
 static int rna_Image_gl_load(Image *image, ReportList *reports, int filter, int mag)
@@ -285,6 +287,7 @@ void RNA_api_image(StructRNA *srna)
 
        func = RNA_def_function(srna, "scale", "rna_Image_scale");
        RNA_def_function_ui_description(func, "Scale the image in pixels");
+       RNA_def_function_flag(func, FUNC_USE_REPORTS);
        parm = RNA_def_int(func, "width", 0, 1, 10000, "", "Width", 1, 10000);
        RNA_def_property_flag(parm, PROP_REQUIRED);
        parm = RNA_def_int(func, "height", 0, 1, 10000, "", "Height", 1, 10000);