We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

[patch] patch to support XCF files with 64bits offset - default branch

SDL_image-default.XCF.v11_64bits_offsets.patch (text/plain), 6.82 KB, created by miniupnp@free.fr on 2018-12-06 13:13:57 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: miniupnp@free.fr

Created: 2018-12-06 13:13:57 UTC

Size: 6.82 KB

patch

obsolete

># HG changeset patch
># User Thomas Bernard <miniupnp@free.fr>
># Date 1544101958 -3600
>#      Thu Dec 06 14:12:38 2018 +0100
># Node ID 521fc983e3437c144a6f58be44789c4d50945e7e
># Parent  c562f51f937641a5bd90854647948e407b79aa5f
>XCF v11+ with 64bits offsets support
>
>diff -r c562f51f9376 -r 521fc983e343 IMG_xcf.c
>--- a/IMG_xcf.c	Thu Dec 06 14:04:30 2018 +0100
>+++ b/IMG_xcf.c	Thu Dec 06 14:12:38 2018 +0100
>@@ -138,9 +138,11 @@
> 
> typedef struct {
>   char   sign [14];
>+  Uint32 file_version;
>   Uint32 width;
>   Uint32 height;
>   Sint32 image_type;
>+  Uint32 precision;
>   xcf_prop * properties;
> 
>   Uint32 * layer_file_offsets;
>@@ -240,6 +242,13 @@
>   return data;
> }
> 
>+static Uint64 read_offset (SDL_RWops * src, const xcf_header * h) {
>+  Uint64 offset;  // starting with version 11, offsets are 64 bits
>+  offset = (h->file_version >= 11) ? (Uint64)SDL_ReadBE32 (src) << 32 : 0;
>+  offset |= SDL_ReadBE32 (src);
>+  return offset;
>+}
>+
> 
> static Uint32 Swap32 (Uint32 v) {
>   return
>@@ -312,10 +321,15 @@
>   h->width       = SDL_ReadBE32 (src);
>   h->height      = SDL_ReadBE32 (src);
>   h->image_type  = SDL_ReadBE32 (src);
>+  h->file_version = (h->sign[10] - '0') * 100 + (h->sign[11] - '0') * 10 + (h->sign[12] - '0');
> #ifdef DEBUG
>-  printf ("XCF signature : %.14s\n", h->sign);
>+  printf ("XCF signature : %.14s (version %u)\n", h->sign, h->file_version);
>   printf (" (%u,%u) type=%u\n", h->width, h->height, h->image_type);
> #endif
>+  if (h->file_version >= 4)
>+    h->precision = SDL_ReadBE32 (src);
>+  else
>+    h->precision = 150;
> 
>   h->properties = NULL;
>   h->layer_file_offsets = NULL;
>@@ -360,7 +374,7 @@
>   SDL_free (l);
> }
> 
>-static xcf_layer * read_xcf_layer (SDL_RWops * src) {
>+static xcf_layer * read_xcf_layer (SDL_RWops * src, const xcf_header * h) {
>   xcf_layer * l;
>   xcf_prop    prop;
> 
>@@ -370,6 +384,9 @@
>   l->layer_type = SDL_ReadBE32 (src);
> 
>   l->name = read_string (src);
>+#ifdef DEBUG
>+  printf ("layer (%d,%d) type=%d '%s'\n", l->width, l->height, l->layer_type, l->name);
>+#endif
> 
>   do {
>     if (!xcf_read_property (src, &prop)) {
>@@ -384,8 +401,8 @@
>     }
>   } while (prop.id != PROP_END);
> 
>-  l->hierarchy_file_offset = SDL_ReadBE32 (src);
>-  l->layer_mask_offset     = SDL_ReadBE32 (src);
>+  l->hierarchy_file_offset = read_offset (src, h);
>+  l->layer_mask_offset     = read_offset (src, h);
> 
>   return l;
> }
>@@ -395,7 +412,7 @@
>   SDL_free (c);
> }
> 
>-static xcf_channel * read_xcf_channel (SDL_RWops * src) {
>+static xcf_channel * read_xcf_channel (SDL_RWops * src, const xcf_header * h) {
>   xcf_channel * l;
>   xcf_prop    prop;
> 
>@@ -404,6 +421,9 @@
>   l->height = SDL_ReadBE32 (src);
> 
>   l->name = read_string (src);
>+#ifdef DEBUG
>+  printf ("channel (%u,%u) '%s'\n", l->width, l->height, l->name);
>+#endif
> 
>   l->selection = 0;
>   do {
>@@ -431,7 +451,7 @@
>     }
>   } while (prop.id != PROP_END);
> 
>-  l->hierarchy_file_offset = SDL_ReadBE32 (src);
>+  l->hierarchy_file_offset = read_offset (src, h);
> 
>   return l;
> }
>@@ -441,7 +461,7 @@
>   SDL_free (h);
> }
> 
>-static xcf_hierarchy * read_xcf_hierarchy (SDL_RWops * src) {
>+static xcf_hierarchy * read_xcf_hierarchy (SDL_RWops * src, const xcf_header * head) {
>   xcf_hierarchy * h;
>   int i;
> 
>@@ -454,7 +474,7 @@
>   i = 0;
>   do {
>     h->level_file_offsets = (Uint32 *) SDL_realloc (h->level_file_offsets, sizeof (Uint32) * (i+1));
>-    h->level_file_offsets [i] = SDL_ReadBE32 (src);
>+    h->level_file_offsets [i] = read_offset (src, head);
>   } while (h->level_file_offsets [i++]);
> 
>   return h;
>@@ -465,7 +485,7 @@
>   SDL_free (l);
> }
> 
>-static xcf_level * read_xcf_level (SDL_RWops * src) {
>+static xcf_level * read_xcf_level (SDL_RWops * src, const xcf_header * h) {
>   xcf_level * l;
>   int i;
> 
>@@ -477,7 +497,7 @@
>   i = 0;
>   do {
>     l->tile_file_offsets = (Uint32 *) SDL_realloc (l->tile_file_offsets, sizeof (Uint32) * (i+1));
>-    l->tile_file_offsets [i] = SDL_ReadBE32 (src);
>+    l->tile_file_offsets [i] = read_offset (src, h);
>   } while (l->tile_file_offsets [i++]);
> 
>   return l;
>@@ -491,7 +511,8 @@
>   unsigned char * load;
> 
>   load = (unsigned char *) SDL_malloc (len); // expect this is okay
>-  SDL_RWread (src, load, len, 1);
>+  if (load != NULL)
>+    SDL_RWread (src, load, len, 1);
> 
>   return load;
> }
>@@ -507,6 +528,8 @@
>   }
> 
>   t = load = (unsigned char *) SDL_malloc (len);
>+  if (load == NULL)
>+    return NULL;
>   reallen = SDL_RWread (src, t, 1, len);
> 
>   data = (unsigned char *) SDL_calloc (1, x*y*bpp);
>@@ -612,7 +635,7 @@
>     Uint32         *row;
> 
>     SDL_RWseek(src, layer->hierarchy_file_offset, RW_SEEK_SET);
>-    hierarchy = read_xcf_hierarchy(src);
>+    hierarchy = read_xcf_hierarchy(src, head);
> 
>     if (hierarchy->bpp > 4) {  /* unsupported. */
>         SDL_Log("Unknown Gimp image bpp (%u)\n", (unsigned int) hierarchy->bpp);
>@@ -629,7 +652,7 @@
>     level = NULL;
>     for (i = 0; hierarchy->level_file_offsets[i]; i++) {
>         SDL_RWseek(src, hierarchy->level_file_offsets[i], RW_SEEK_SET);
>-        level = read_xcf_level(src);
>+        level = read_xcf_level(src, head);
> 
>         ty = tx = 0;
>         for (j = 0; level->tile_file_offsets[j]; j++) {
>@@ -810,7 +833,7 @@
> 
>   offsets = 0;
> 
>-  while ((offset = SDL_ReadBE32 (src))) {
>+  while ((offset = read_offset (src, head))) {
>     head->layer_file_offsets = (Uint32 *) SDL_realloc (head->layer_file_offsets, sizeof (Uint32) * (offsets+1));
>     head->layer_file_offsets [offsets] = (Uint32)offset;
>     offsets++;
>@@ -830,20 +853,22 @@
>     SDL_Rect rs, rd;
>     SDL_RWseek (src, head->layer_file_offsets [i-1], RW_SEEK_SET);
> 
>-    layer = read_xcf_layer (src);
>-    do_layer_surface (lays, src, head, layer, load_tile);
>-    rs.x = 0;
>-    rs.y = 0;
>-    rs.w = layer->width;
>-    rs.h = layer->height;
>-    rd.x = layer->offset_x;
>-    rd.y = layer->offset_y;
>-    rd.w = layer->width;
>-    rd.h = layer->height;
>+    layer = read_xcf_layer (src, head);
>+    if (layer != NULL) {
>+      do_layer_surface (lays, src, head, layer, load_tile);
>+      rs.x = 0;
>+      rs.y = 0;
>+      rs.w = layer->width;
>+      rs.h = layer->height;
>+      rd.x = layer->offset_x;
>+      rd.y = layer->offset_y;
>+      rd.w = layer->width;
>+      rd.h = layer->height;
> 
>-    if (layer->visible)
>-      SDL_BlitSurface (lays, &rs, surface, &rd);
>-    free_xcf_layer (layer);
>+      if (layer->visible)
>+        SDL_BlitSurface (lays, &rs, surface, &rd);
>+      free_xcf_layer (layer);
>+    }
>   }
> 
>   SDL_FreeSurface (lays);
>@@ -853,11 +878,13 @@
>   // read channels
>   channel = NULL;
>   chnls   = 0;
>-  while ((offset = SDL_ReadBE32 (src))) {
>+  while ((offset = read_offset (src, head))) {
>     channel = (xcf_channel **) SDL_realloc (channel, sizeof (xcf_channel *) * (chnls+1));
>     fp = SDL_RWtell (src);
>     SDL_RWseek (src, offset, RW_SEEK_SET);
>-    channel [chnls++] = (read_xcf_channel (src));
>+    channel [chnls] = (read_xcf_channel (src));
>+    if (channel [chnls] != NULL)
>+      chnls++;
>     SDL_RWseek (src, fp, RW_SEEK_SET);
>   }
>

Actions: View | Diff

Attachments on bug 4413: 3524 | 3525 | 3526 | 3527 | 3528 | 3529 | 3530 | 3531 | 3532

We are currently migrating Bugzilla to GitHub issues.Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.