emile_image: fix possible segmentation fault in AGRY88
authorSungtaek Hong <sth253.hong@samsung.com>
Wed, 7 Dec 2016 04:42:46 +0000 (13:42 +0900)
committerHermet Park <hermet@hermet.pe.kr>
Wed, 7 Dec 2016 04:42:46 +0000 (13:42 +0900)
Summary:
 - ptrag is set NULL and is allocated only when prop is rotated.
   but *ptrag = 0xFF00 | ptr[0]; without checking rotation.

Reviewers: jpeg, cedric, Hermet

Reviewed By: Hermet

Subscribers: conr2d

Differential Revision: https://phab.enlightenment.org/D4463

src/lib/emile/emile_image.c

index 59af9b4..e04df4c 100644 (file)
@@ -1860,10 +1860,11 @@ _emile_jpeg_data(Emile_Image *image,
    else
      {
         ptr2 = pixels;
+        ptrag = pixels;
         ptrg = pixels;
      }
 
-   if (!ptr2 && !ptrg)
+   if (!ptr2 && !ptrag && !ptrg)
      {
         *error = EMILE_IMAGE_LOAD_ERROR_RESOURCE_ALLOCATION_FAILED;
         goto on_error;