Comment 1:

Labels changed: added priority-later, packagebug, removed priority-triage.

Owner changed to @nigeltao.

Status changed to Accepted.

Comment 2 by dan.pupius:

Here's a workaround, should anyone else hit this issue.  You have to create a new zero
based image for each frame and instead of using SubImage copy the crop region
35c36,41
<        im.Image[index] = frame.SubImage(dstBounds).(*image.Paletted)
---
>        // Creates a new image bounded at (0,0) and instead of creating a SubImage,
>        // copy the cropped region into a new image.
>        b := image.Rect(0, 0, dstBounds.Dx(), dstBounds.Dy())
>        pm := image.NewPaletted(b, frame.Palette)
>        draw.Draw(pm, b, frame, dstBounds.Min, draw.Src)
>        im.Image[index] = pm

Comment 3:

I'll look at this after the 1.2 release. Feel free to ping this bug (after then) if I've
forgotten.

Comment 4:

Labels changed: added go1.3maybe.

Comment 5:

Labels changed: added release-none, removed go1.3maybe.

Comment 6:

Labels changed: added repo-main.

In go version go1.10 linux/amd64 the output looks like this: lucha.out.1.10.gif

This seems to be a legitimate output, since each Image Descriptor may have different dimensions and offsets from the Screen Descriptor (as long they stay within the bounds of Screen Descriptor).

While investigating any link between GIF issues, the documentation does not specify that the original size of the GIF (image config) is kept after crop. You cannot update the config of the image (logical screen descriptor) after cropping and it is not set to the minimal size. On 1.11 beta 2, the GIF is valid with appropriate offset but the Logical Descriptor has the original size which is incorrect.

0: lucha.gif (original file before editing)
Logical screen descriptor : [300x300] and fields == 1000100
Image descriptor: offset [0x0] for image size == [300x300] and image fields == 0
1: luchafromissue.gif (as issue was filed)
Logical screen descriptor : [300x150] and fields == 0
Image descriptor: offset [0x75] for image size == [300x150] and image fields == 10000101
 decoding fails with  gif: frame bounds larger than image bounds

The original issue with adapted logical descriptor (image config) while keeping the offset.

2: lucha111beta2.gif (using latest beta release)
Logical screen descriptor : [300x300] and fields == 1000100
Image descriptor: offset [0x75] for image size == [300x150] and image fields == 0

Latest behaviour produces a correct GIF by retaining the original Logical Descriptor.

3: luchacroppedbyPaint3D.gif (edited using Paint3D on Windows 10)
Logical screen descriptor : [300x150] and fields == 1110000
Image descriptor: offset [0x0] for image size == [300x150] and image fields == 10000101

A simplified GIF structure without offset.