If you try to make an image from an ICO or CUR image string using the Imagemagick driver an Unable to read image from binary data exception is thrown. If you try from a path however it works, because Imagick can determine the image type using the file extension.

I've fixed loading from a string here by checking the first four bytes of the binary to see if the string is an ICO or CUR file, and then set the appropriate format on the Imagick object. I've added test coverage for this scenario, and also for loading ICO and CUR files using paths and Imagick objects too.

One odd thing I noticed is that finfo_open() returns image/vnd.microsoft.icon on my machine, but the Travis CI builds return image/x-icon. I don't know how much this matters though, as this can be dealt with in userland (there are numerous mime types for ICO, so easy enough to check).

I previously opened a bug for this, but I've closed that and opened this fix instead (there's some examples of the issue there): #1007

Thanks!