DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org 933E03840C25
Date: Wed, 24 Jun 2020 18:46:54 +0000 (UTC)
To: cygwin@cygwin.com, Marco Atzeri <marco.atzeri@gmail.com>
Message-ID: <1697153505.2549743.1593024414719@mail.yahoo.com>
In-Reply-To: <7577f575-9a66-bdb6-1cb9-30a6f9e305ad@gmail.com>
References: <241354771.1752406.1592876221936.ref@mail.yahoo.com>
 <241354771.1752406.1592876221936@mail.yahoo.com>
 <92b51d45-55b5-33e1-c1c7-883094b6b09b@cornell.edu>
 <134f371b-8989-4ed7-2209-acc486960d65@cornell.edu>
 <0dd00cea6dd675142ca617d2caf5d818@mhoenicka.de>
 <3d65999c-c82c-0b75-925b-d799a7e8f98c@gmail.com>
 <18a9c001-03cc-1538-99aa-7ea87630abe5@cornell.edu>
 <df040576-8ce9-51ac-8c92-97e44ac9dedd@cornell.edu>
 <6744da4c-ed95-ca9b-3e07-2adc7ea9f36f@gmail.com>
 <a1918811-4766-4383-7469-3f6e6921dc3b@gmail.com>
 <094cb004-e88e-4b4c-89ce-e8c8791c3f9e@cornell.edu>
 <7577f575-9a66-bdb6-1cb9-30a6f9e305ad@gmail.com>
Subject: Re: ImageMagick maybe broken by libgs9 upgrade
MIME-Version: 1.0
Precedence: list
From: Emily via Cygwin <cygwin@cygwin.com>
Reply-To: Emily <emilyw@yahoo.com>
Content-Type: text/plain; charset="utf-8"
Errors-To: cygwin-bounces@cygwin.com
Sender: "Cygwin" <cygwin-bounces@cygwin.com>
Content-Transfer-Encoding: 8bit

Hi Marco, I don't see that one, do I need to add a special download site?  Thanks!


On Wednesday, June 24, 2020, 11:21:59 AM PDT, Marco Atzeri <marco.atzeri@gmail.com> wrote: 

On 24.06.2020 12:24, Ken Brown via Cygwin wrote:
> On 6/24/2020 1:30 AM, Marco Atzeri via Cygwin wrote:
>> On 23.06.2020 23:02, Marco Atzeri wrote:
>>> On 23.06.2020 22:56, Ken Brown via Cygwin wrote:
>>>>
>>>> I think I've found the problem, although it will take patching and 
>>>> rebuilding ImageMagick (which I haven't done) to confirm that I'm 
>>>> right.
>>>>
>>>> In the ImageMagick source file coders/ps.c:237, there's a call to 
>>>> ghost_info->new_instance, a.k.a. gsapi_new_instance (see line 214). 
>>>> The documentation of the latter in the ghostscript sources 
>>>> (psi/iapi.c:57) says that the first argument pinstance should 
>>>> satisfy *pinstance == NULL in the first call to that function.
>>>>
>>>> But *pinstance in this call is the variable 'interpreter', defined 
>>>> without initialization in ps.c:191.  As a result, **pinstance 
>>>> contains garbage, and the program eventually crashes when it tries 
>>>> to dereference a garbage pointer.
>>>>
>>>> The fix, if I'm right, is to initialize interpreter to NULL in 
>>>> ps.c:191.
>>>>
>>>> Ken
>>>
>>> Noted.
>>>
>>> Regards
>>> MArco
>>>
>>
>> a test ImageMagick-6.9.10.11-3 is going up.
>> Let me know if it solves the problem
> 
> That fixes it for my .eps file.  I didn't think of it before, but a 
> similar patch to coders/pdf.c:201 will be needed for .pdf files.  I 
> won't be able to test that because I haven't found a .pdf file that 
> exhibits the problem.
> 
> Ken
> 


test version 6.9.10.11-4 is going up.

If Emily tests with her pdf files, it will be nice

Regards
Marco

--
Problem reports:      https://cygwin.com/problems.html
FAQ:                  https://cygwin.com/faq/
Documentation:        https://cygwin.com/docs.html
Unsubscribe info:     https://cygwin.com/ml/#unsubscribe-simple