In writeimage.c in m_WriteImage it uses iargs.compression before initializing it. This was what was causing a coredump when doing WriteImage.

I am not into changing dx sources so could someone please add the following 'patch' or something similar to the current sources?

around line 107 add to the block:

    iargs.image = NULL;
    iargs.pipe = NULL;

initialization of the compression flag:

    iargs.image = NULL;
    iargs.pipe = NULL;
    iargs.compression = NULL;

This was discovered using the excellent valgrind tool. (I created a shellscript which executes dxexec under valgrind and set DXEXEC so it uses that one)

It also shows up another problem which I don't know what it is caused by. In my 4.3.0 sources:

==6514== Conditional jump or move depends on uninitialised value(s)
Begin Execution
==6514==    at 0x8079B51: ExCheckInputAvailability (evalgraph.c:3039)
==6514==    by 0x80748CD: ExStartModules (evalgraph.c:821)
==6514==    by 0x8075444: _execGnode (evalgraph.c:1615)
==6514==    by 0x808ED76: _dxf_ExRQDequeue (rq.c:443)

Regards,

Mattijs

--
/*---------------------------------------------------------*\
| ===========           Mattijs Janssens                    |
| \\        /           Development Engineer                |
|  \\      /                                                |
|   \\    /             Nabla Ltd.                          |
|    \\  /              The Mews, Picketts Lodge            |
|     \\/               Picketts Lane, Salfords,            |
|     F ield            Surrey RH1 5RG.                     |
|     O peration        Tel: +44 (0)1293 821272             |
|     A nd              Email: [EMAIL PROTECTED]       |
|     M anipulation     URL: http://www.Nabla.co.uk         |
\*---------------------------------------------------------*/

Reply via email to