cur file saved as a png has black pixels where it needs to be transparent

[mmemoo]

its a really easy-to-reproduce issue, just run this code with the file i attached in Pillow 12.1.0 :

from PIL import Image

im = Image.open("aero_unavail.cur")
im.save("aero_unavail.png")

(the file is provided as a txt file since github doesn't allow cur files to be uploaded, just change the extension.)

aero_unavail.txt

[radarhere]

Hi. This has been raised before in #8594. In that issue, the conclusion was that the specification says it should actually be black. However, we added a setting for this. If you set USE_RAW_ALPHA to True, then you will get the result you expect.

from PIL import Image, BmpImagePlugin
BmpImagePlugin.USE_RAW_ALPHA = True

im = Image.open("./aero_unavail.cur")
im.save("aero_unavail.png")

[mmemoo]

hi, thank you for the help.
why does this happen for some cur files?

[radarhere]

https://learn.microsoft.com/en-us/windows/win32/wmdm/-bitmapinfoheader describes 32 bitcount as

The bitmap has a maximum of 2^32 colors. If the biCompression member is BI_RGB, the bmiColors member is NULL. Each DWORD in the bitmap array represents the relative intensities of blue, green, and red, respectively, for a pixel. The high byte in each DWORD is not used.

So the specification says that the fourth channel should not be used. That means that the image should be rendered as RGB, not RGBA, hence the black.

I can't say why some other software has chosen to ignore this part of the specification.

[mmemoo]

thank you!

[mmemoo]

sorry to bother again but the issue persists on other files like the one i attached
(attached as txt instead of .cur, again)

default_arrow.txt

[radarhere]

Your latest file appears to have a different issue. I still need to investigate this further, but it appears that it contains two images, and one is a mask for the other.

If you would like an immediate fix, this code

from PIL import Image, ImageOps, BmpImagePlugin

def read(path):
  BmpImagePlugin.USE_RAW_ALPHA = True

  im = Image.open(path)
  if im.mode == "1":
    out = Image.new("LA", im.size)
    im._size = im.width, im.height * 2
    im.tile = [im.tile[0]._replace(extents=(0, 0)+im.size)]
    
    and_mask = im.crop((0, out.height, out.width, im.height))
    xor_mask = ImageOps.invert(im.crop((0, 0) + out.size))
    out.paste(and_mask, mask=xor_mask)
    return out
  else:
    return im

read("./default_arrow.cur").save("default_arrow.png")
read("./aero_unavail.cur").save("aero_unavail.png")

gives

[mmemoo]

by "it has another image which is a mask" you mean the single image in the .cur file has a proper and mask no? if that's not the case when i inspect the file with imagemagick i see only 1 image

[radarhere]

it contains two images, and one is a mask for the other.

This sentence was just me describing a technical detail.

If you're not curious about how data is arranged within an image, feel free to completely ignore it.

If you are curious, then I meant that within the image, there is a black-and-white section, and then another black-and-white section. One section actually does render as black-and-white, in the other, the 'white' pixels re the ones that are shown, and the 'black' pixels are the ones that become transparent. It's how the creators of CUR decided to assemble a black/white/transparent image out of only black-and-white data.

In the end, yes, there is only one image that the user should ever see.

[mmemoo]

i actually somewhat know about the structure of cur files and i guess you are talking about xor, and masks.

plus i have some questions afaik, the and mask is used as the transparency mask but your code says otherwise, why?
and when this cur file is used, is there any xor operation which involves the xor mask or is it just used as pixel data?

also a little off-topic but, does bmp's with any bpp store images as and + xor masks or does it work differently