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 ...

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 ...

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