Legacy ImageMagick Discussions Archive

Ah. so the latest version of libjasper does in fact use tmpnam(), so I assume the one included with Imagemagick is well out of date.

It's not obvious to me why tmpnam() isn't used.

So i've replaced with in jas_stream_tmpfile()
and we're cooking with gas now.

Having looked at it a bit more, I wouldn't bother :) jas_image_decode does that for you when you pass -1 in as the fmt, so I don't think it's relevant. What is relevant is your original response - this seems to be a problem with JasPer, specifically, jas_stream_tmpfile(). It seems to create a temp ...

that is to say, instead of hard coding the format to -1 there, should it not do something like this? /* Initialize JPEG 2000 API. */ jp2_stream=JP2StreamManager(image); if (jp2_stream == (jas_stream_t *) NULL) ThrowReaderException(DelegateError,"UnableToManageJP2Stream"); fmt = jas_image_getfmt(jp2 ...

OK, that's interesting. So I had a look at jas_image_decode(jas_stream_t *in, int fmt, char *optstr) and there's this: /* Is it possible to decode an image represented in this format? */ if (!(fmtinfo = jas_image_lookupfmtbyid(fmt))) goto error; And in imagemagick, jp2.c: jp2_image=jas_image_decode ...

Confirmed for me, example file here https://github.com/openplanets/jpylyzer ... alloon.jp2