imagemagick: Header files kill attributes

Bug #7687 reported by Debian Bug Importer
4
Affects Status Importance Assigned to Milestone
imagemagick (Debian)
Fix Released
Unknown
imagemagick (Ubuntu)
Invalid
High
Unassigned

Bug Description

Automatically imported from Debian bug report #269085 http://bugs.debian.org/269085

Revision history for this message
In , Ryuichi Arafune (arafune) wrote : Bug#265540: fixed in imagemagick 5:6.0.5.2-1
Download full text (3.5 KiB)

Source: imagemagick
Source-Version: 5:6.0.5.2-1

We believe that the bug you reported is fixed in the latest version of
imagemagick, which is due to be installed in the Debian FTP archive:

imagemagick_6.0.5.2-1.diff.gz
  to pool/main/i/imagemagick/imagemagick_6.0.5.2-1.diff.gz
imagemagick_6.0.5.2-1.dsc
  to pool/main/i/imagemagick/imagemagick_6.0.5.2-1.dsc
imagemagick_6.0.5.2-1_i386.deb
  to pool/main/i/imagemagick/imagemagick_6.0.5.2-1_i386.deb
imagemagick_6.0.5.2.orig.tar.gz
  to pool/main/i/imagemagick/imagemagick_6.0.5.2.orig.tar.gz
libmagick++6-dev_6.0.5.2-1_i386.deb
  to pool/main/i/imagemagick/libmagick++6-dev_6.0.5.2-1_i386.deb
libmagick++6_6.0.5.2-1_i386.deb
  to pool/main/i/imagemagick/libmagick++6_6.0.5.2-1_i386.deb
libmagick6-dev_6.0.5.2-1_i386.deb
  to pool/main/i/imagemagick/libmagick6-dev_6.0.5.2-1_i386.deb
libmagick6_6.0.5.2-1_i386.deb
  to pool/main/i/imagemagick/libmagick6_6.0.5.2-1_i386.deb
perlmagick_6.0.5.2-1_i386.deb
  to pool/main/i/imagemagick/perlmagick_6.0.5.2-1_i386.deb

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed. If you
have further comments please address them to <email address hidden>,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Ryuichi Arafune <email address hidden> (supplier of updated imagemagick package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing <email address hidden>)

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.7
Date: Tue, 17 Aug 2004 14:38:53 +0900
Source: imagemagick
Binary: perlmagick libmagick++6-dev libmagick6-dev libmagick6 imagemagick libmagick++6
Architecture: source i386
Version: 5:6.0.5.2-1
Distribution: unstable
Urgency: low
Maintainer: Ryuichi Arafune <email address hidden>
Changed-By: Ryuichi Arafune <email address hidden>
Description:
 imagemagick - Image manipulation programs
 libmagick++6 - The object-oriented C++ API to the ImageMagick library
 libmagick++6-dev - The object-oriented C++ API to the ImageMagick library--developme
 libmagick6 - Image manipulation library (free version)
 libmagick6-dev - Image manipulation library (free version) -- development
 perlmagick - A perl interface to the libMagick graphics routines
Closes: 260702 261768 265540
Changes:
 imagemagick (5:6.0.5.2-1) unstable; urgency=low
 .
   * New upstream release
   * Fixed by the upstream : closes:#265540, #261768, #260702
Files:
 089251d17fc978a48138f15680db57c8 877 graphics optional imagemagick_6.0.5.2-1.dsc
 df60a30ef781fea256e257a3eb34e585 6821654 graphics optional imagemagick_6.0.5.2.orig.tar.gz
 326d0b8619fa66f6513cc9d350681118 133111 graphics optional imagemagick_6.0.5.2-1.diff.gz
 bdd2b2201cec12af42a0c1412fe8ae98 1462746 graphics optional imagemagick_6.0.5.2-1_i386.deb
 c6ef7b33b691b078c6fa84f307da4151 1162014 libs optional libmagick6_6.0.5.2-1_i386.deb
 dd581b65960d908b32f83041a32bd364 1496466 libdevel optional libmagick6-dev_6.0.5.2-1_i386.deb
 863bce65f7b80b209e711f770df24dcd 162744 libs optional libmagick++6_6.0.5.2-1_...

Read more...

Revision history for this message
In , Daniel Kobras (kobras) wrote : Re: Bug#265540: marked as done (convert produces broken output)

reopen 265540 =
thanks

On Mon, Aug 16, 2004 at 11:33:11PM -0700, Debian Bug Tracking System wrote:
> Package: imagemagick
> Version: 5:6.0.3.5-2
> Severity: important
> Architecture: ia64
>
> Please compare http://www.parisc-linux.org/~willy/wot/cnl20040623/graph.ps
> and http://www.parisc-linux.org/~willy/wot/cnl20040623/graph.jpg
> The .jpg was produced from the .ps using:
>
> convert graph.ps graph.jpg
>
> As you can see, the results (while entertaining) are not useful.
(...)
> imagemagick (5:6.0.5.2-1) unstable; urgency=low
> .
> * New upstream release
> * Fixed by the upstream : closes:#265540, #261768, #260702

Testing on an ia64 machine reveals that 6.0.5.2-1 does not fix this
issue. (The problem doesn't show up on i386.)

Daniel.

Revision history for this message
In , Sylvain Le Gall (gildor) wrote : Imagemagick semi broken on IA64

severity 265540 grave

thanks

Hello,

It seems that the convert program do worse than outputing bad quality
image, it also produces "Bus error" on IA 64.

Please have a look at this build log :
http://buildd.debian.org/fetch.php?&pkg=cameleon&ver=1.3-4&arch=ia64&stamp=1093353284&file=log&as=raw
http://buildd.debian.org/fetch.php?&pkg=cameleon&ver=1.3-4&arch=ia64&stamp=1092645134&file=log&as=raw

convert main_window.jpg main_window.eps
make[2]: *** [main_window.eps] Bus error

Kind regard
Sylvain Le Gall

Revision history for this message
In , Kyle McMartin (kyle) wrote : imagemagick on ia64

The root cause seems to be,

convert(1130): unaligned access to 0x60000fffffff8c28,
ip=0x200000000104f090
zsh: 1130 bus error (core dumped) convert graph.jpg graph.eps

(gdb) bt
#0 0x200000000104f090 in __sigsetjmp () from /lib/libc.so.6.1
#1 0x0000000000000000 in ?? ()
Previous frame identical to this frame (corrupt stack?)

Yuck.

stat("graph.jpg", {st_mode=S_IFREG|0644, st_size=133363, ...}) = 0
open("graph.jpg", O_RDONLY) = 3
fstat(3, {st_mode=S_IFREG|0644, st_size=133363, ...}) = 0
mmap(NULL, 65536, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS, -1,
0) = 0x20000000012a8000
read(3, "\377\330\377\340\0\20JFIF\0\1\1\1\0\34\0\34\0\0\377\356"...,
16384) = 16384
lseek(3, 0, SEEK_SET) = 0
mmap(NULL, 133363, PROT_READ, MAP_PRIVATE, 3, 0) = 0x20000000012b8000
close(3) = 0
munmap(0x20000000012a8000, 65536) = 0
convert(3167): unaligned access to 0x60000fffffff8c28,
ip=0x200000000104f090
--- SIGBUS (Bus error) @ 200000000104f090 (60000fffffff8c28) ---
+++ killed by SIGBUS (core dumped) +++

Double Yuck.

Will recompile imagemagick with debugging on and take a better look
tomorrow.

Regards,
--
Kyle McMartin

Revision history for this message
In , Sven Luther (sven-luther) wrote : Re: Imagemagick semi broken on IA64

On Fri, Aug 27, 2004 at 01:12:44AM +0200, Sylvain LE GALL wrote:
> severity 265540 grave
>
> thanks
>
> Hello,
>
> It seems that the convert program do worse than outputing bad quality
> image, it also produces "Bus error" on IA 64.
>
> Please have a look at this build log :
> http://buildd.debian.org/fetch.php?&pkg=cameleon&ver=1.3-4&arch=ia64&stamp=1093353284&file=log&as=raw
> http://buildd.debian.org/fetch.php?&pkg=cameleon&ver=1.3-4&arch=ia64&stamp=1092645134&file=log&as=raw
>
>
> convert main_window.jpg main_window.eps
> make[2]: *** [main_window.eps] Bus error

Little followup :

07:09 < kylem> svenl, this error is exceedingly gross.
07:13 < svenl> kylem: huh ?
07:13 < kylem> imagemagick
07:14 < svenl> kylem: yeah, but why you say the error is gross ?
07:15 < kylem> unaligned access.
07:15 < svenl> Oh.
07:15 < kylem> stack corruption in the core dump.

Thanks kylem for having a look at this.

Friendly,

Sven Luther

Revision history for this message
In , Daniel Kobras (kobras) wrote : Re: Bug#265540: imagemagick on ia64

clone 265540 -1 -2
severity -1 important
retitle -2 imagemagick: Header files kill attributes
thanks

There are two issues. The first has to do with conversions to postscript
being messed up. This is most likely a ghostscript bug. Haven't
investigated in detail. The second shows up on ia64 when trying to use
imagemagick on a jpeg file. I'm splitting the bug reports, therefore,
and am going to talk about the latter in the following.

On Fri, Aug 27, 2004 at 01:03:47AM -0400, Kyle McMartin wrote:
>
> The root cause seems to be,
>
> convert(1130): unaligned access to 0x60000fffffff8c28,
> ip=0x200000000104f090
> zsh: 1130 bus error (core dumped) convert graph.jpg graph.eps
>
> (gdb) bt
> #0 0x200000000104f090 in __sigsetjmp () from /lib/libc.so.6.1
> #1 0x0000000000000000 in ?? ()
> Previous frame identical to this frame (corrupt stack?)

The root cause actually is

 #if !defined(__attribute__)
 # define __attribute__(x) /*nothing*/
 #endif

sprinkled across several of imagemagick's header files. Too bad that
__attribute__ is no #define, and that some sources after the imagemagick
headers #include system headers that need to set attributes. The bug
reported here was caused by coders/jpeg.c #including <setjmp.h> that
#includes <bits/setjmp.h>, which on ia64 contains the line

 typedef long __jmp_buf[_JBLEN] __attribute__ ((aligned (16)));
 /* guarantees 128-bit alignment! */

The alignment restriction is killed by the above definition in the
imagemagick headers, and the setjmp call in ReadJPEGImage() triggers the
bus error due to an unaligned access. I'll follow up with the obvious
quick'n'dirty patch to fix the issue for us.

Regards,

Daniel.

Revision history for this message
In , Daniel Kobras (kobras) wrote :

tag 269085 + patch
thanks

Moi Ryuichi!

I have attached a patch that includes a fix for convert bailing out with
a bus error on ia64 machines. It also fixes a potential integer overflow
introduced with the security fix with the current upstream version.
Especially the first issue needs to be fixed urgently becaused it causes
build failures of other packages on ia64 at least. As you can see, I
have already prepared an NMU with these two changes that is tested on
ia64, and that I can upload without further effort. However, I'd be more
comfortable if you either gave me an okay to do the upload, or of course
prepare a maintainer upload including these changes. Full NMU diff to
your previous upload attached.

Regards,

Daniel.

Revision history for this message
Debian Bug Importer (debzilla) wrote :

Automatically imported from Debian bug report #269085 http://bugs.debian.org/269085

Revision history for this message
Debian Bug Importer (debzilla) wrote :

Message-ID: <email address hidden>
Date: Fri, 13 Aug 2004 17:05:47 +0100
From: Matthew Wilcox <email address hidden>
To: <email address hidden>
Subject: convert produces broken output

Package: imagemagick
Version: 5:6.0.3.5-2
Severity: important
Architecture: ia64

Please compare http://www.parisc-linux.org/~willy/wot/cnl20040623/graph.ps
and http://www.parisc-linux.org/~willy/wot/cnl20040623/graph.jpg
The .jpg was produced from the .ps using:

 convert graph.ps graph.jpg

As you can see, the results (while entertaining) are not useful.

--
"Next the statesmen will invent cheap lies, putting the blame upon
the nation that is attacked, and every man will be glad of those
conscience-soothing falsities, and will diligently study them, and refuse
to examine any refutations of them; and thus he will by and by convince
himself that the war is just, and will thank God for the better sleep
he enjoys after this process of grotesque self-deception." -- Mark Twain

Revision history for this message
Debian Bug Importer (debzilla) wrote :
Download full text (3.7 KiB)

Message-Id: <email address hidden>
Date: Tue, 17 Aug 2004 02:17:10 -0400
From: Ryuichi Arafune <email address hidden>
To: <email address hidden>
Subject: Bug#265540: fixed in imagemagick 5:6.0.5.2-1

Source: imagemagick
Source-Version: 5:6.0.5.2-1

We believe that the bug you reported is fixed in the latest version of
imagemagick, which is due to be installed in the Debian FTP archive:

imagemagick_6.0.5.2-1.diff.gz
  to pool/main/i/imagemagick/imagemagick_6.0.5.2-1.diff.gz
imagemagick_6.0.5.2-1.dsc
  to pool/main/i/imagemagick/imagemagick_6.0.5.2-1.dsc
imagemagick_6.0.5.2-1_i386.deb
  to pool/main/i/imagemagick/imagemagick_6.0.5.2-1_i386.deb
imagemagick_6.0.5.2.orig.tar.gz
  to pool/main/i/imagemagick/imagemagick_6.0.5.2.orig.tar.gz
libmagick++6-dev_6.0.5.2-1_i386.deb
  to pool/main/i/imagemagick/libmagick++6-dev_6.0.5.2-1_i386.deb
libmagick++6_6.0.5.2-1_i386.deb
  to pool/main/i/imagemagick/libmagick++6_6.0.5.2-1_i386.deb
libmagick6-dev_6.0.5.2-1_i386.deb
  to pool/main/i/imagemagick/libmagick6-dev_6.0.5.2-1_i386.deb
libmagick6_6.0.5.2-1_i386.deb
  to pool/main/i/imagemagick/libmagick6_6.0.5.2-1_i386.deb
perlmagick_6.0.5.2-1_i386.deb
  to pool/main/i/imagemagick/perlmagick_6.0.5.2-1_i386.deb

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed. If you
have further comments please address them to <email address hidden>,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Ryuichi Arafune <email address hidden> (supplier of updated imagemagick package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing <email address hidden>)

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.7
Date: Tue, 17 Aug 2004 14:38:53 +0900
Source: imagemagick
Binary: perlmagick libmagick++6-dev libmagick6-dev libmagick6 imagemagick libmagick++6
Architecture: source i386
Version: 5:6.0.5.2-1
Distribution: unstable
Urgency: low
Maintainer: Ryuichi Arafune <email address hidden>
Changed-By: Ryuichi Arafune <email address hidden>
Description:
 imagemagick - Image manipulation programs
 libmagick++6 - The object-oriented C++ API to the ImageMagick library
 libmagick++6-dev - The object-oriented C++ API to the ImageMagick library--developme
 libmagick6 - Image manipulation library (free version)
 libmagick6-dev - Image manipulation library (free version) -- development
 perlmagick - A perl interface to the libMagick graphics routines
Closes: 260702 261768 265540
Changes:
 imagemagick (5:6.0.5.2-1) unstable; urgency=low
 .
   * New upstream release
   * Fixed by the upstream : closes:#265540, #261768, #260702
Files:
 089251d17fc978a48138f15680db57c8 877 graphics optional imagemagick_6.0.5.2-1.dsc
 df60a30ef781fea256e257a3eb34e585 6821654 graphics optional imagemagick_6.0.5.2.orig.tar.gz
 326d0b8619fa66f6513cc9d350681118 133111 graphics optional imagemagick_6.0.5.2-1.diff.gz
 bdd2b2201cec12af42a0c1412fe8ae98 1462746 graphics optional imagemagick_6.0.5.2-1_i386.deb
 c6ef7b33b691b078c6fa84f307da4151 1162...

Read more...

Revision history for this message
Debian Bug Importer (debzilla) wrote :

Message-ID: <email address hidden>
Date: Tue, 17 Aug 2004 16:01:55 +0200
From: Daniel Kobras <email address hidden>
To: Debian Bug Tracking System <email address hidden>
Subject: Re: Bug#265540: marked as done (convert produces broken output)

reopen 265540 =
thanks

On Mon, Aug 16, 2004 at 11:33:11PM -0700, Debian Bug Tracking System wrote:
> Package: imagemagick
> Version: 5:6.0.3.5-2
> Severity: important
> Architecture: ia64
>
> Please compare http://www.parisc-linux.org/~willy/wot/cnl20040623/graph.ps
> and http://www.parisc-linux.org/~willy/wot/cnl20040623/graph.jpg
> The .jpg was produced from the .ps using:
>
> convert graph.ps graph.jpg
>
> As you can see, the results (while entertaining) are not useful.
(...)
> imagemagick (5:6.0.5.2-1) unstable; urgency=low
> .
> * New upstream release
> * Fixed by the upstream : closes:#265540, #261768, #260702

Testing on an ia64 machine reveals that 6.0.5.2-1 does not fix this
issue. (The problem doesn't show up on i386.)

Daniel.

Revision history for this message
Debian Bug Importer (debzilla) wrote :

Message-ID: <20040826231243.GA7061@grand>
Date: Fri, 27 Aug 2004 01:12:44 +0200
From: "Sylvain LE GALL" <email address hidden>
To: <email address hidden>
Cc: <email address hidden>
Subject: Imagemagick semi broken on IA64

severity 265540 grave

thanks

Hello,

It seems that the convert program do worse than outputing bad quality
image, it also produces "Bus error" on IA 64.

Please have a look at this build log :
http://buildd.debian.org/fetch.php?&pkg=cameleon&ver=1.3-4&arch=ia64&stamp=1093353284&file=log&as=raw
http://buildd.debian.org/fetch.php?&pkg=cameleon&ver=1.3-4&arch=ia64&stamp=1092645134&file=log&as=raw

convert main_window.jpg main_window.eps
make[2]: *** [main_window.eps] Bus error

Kind regard
Sylvain Le Gall

Revision history for this message
Debian Bug Importer (debzilla) wrote :

Message-ID: <email address hidden>
Date: Fri, 27 Aug 2004 01:03:47 -0400
From: <email address hidden> (Kyle McMartin)
To: <email address hidden>
Subject: imagemagick on ia64

The root cause seems to be,

convert(1130): unaligned access to 0x60000fffffff8c28,
ip=0x200000000104f090
zsh: 1130 bus error (core dumped) convert graph.jpg graph.eps

(gdb) bt
#0 0x200000000104f090 in __sigsetjmp () from /lib/libc.so.6.1
#1 0x0000000000000000 in ?? ()
Previous frame identical to this frame (corrupt stack?)

Yuck.

stat("graph.jpg", {st_mode=S_IFREG|0644, st_size=133363, ...}) = 0
open("graph.jpg", O_RDONLY) = 3
fstat(3, {st_mode=S_IFREG|0644, st_size=133363, ...}) = 0
mmap(NULL, 65536, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS, -1,
0) = 0x20000000012a8000
read(3, "\377\330\377\340\0\20JFIF\0\1\1\1\0\34\0\34\0\0\377\356"...,
16384) = 16384
lseek(3, 0, SEEK_SET) = 0
mmap(NULL, 133363, PROT_READ, MAP_PRIVATE, 3, 0) = 0x20000000012b8000
close(3) = 0
munmap(0x20000000012a8000, 65536) = 0
convert(3167): unaligned access to 0x60000fffffff8c28,
ip=0x200000000104f090
--- SIGBUS (Bus error) @ 200000000104f090 (60000fffffff8c28) ---
+++ killed by SIGBUS (core dumped) +++

Double Yuck.

Will recompile imagemagick with debugging on and take a better look
tomorrow.

Regards,
--
Kyle McMartin

Revision history for this message
Debian Bug Importer (debzilla) wrote :

Message-ID: <20040827054847.GA17634@pegasos>
Date: Fri, 27 Aug 2004 07:48:47 +0200
From: Sven Luther <email address hidden>
To: Sylvain LE GALL <email address hidden>
Cc: <email address hidden>
Subject: Re: Imagemagick semi broken on IA64

On Fri, Aug 27, 2004 at 01:12:44AM +0200, Sylvain LE GALL wrote:
> severity 265540 grave
>
> thanks
>
> Hello,
>
> It seems that the convert program do worse than outputing bad quality
> image, it also produces "Bus error" on IA 64.
>
> Please have a look at this build log :
> http://buildd.debian.org/fetch.php?&pkg=cameleon&ver=1.3-4&arch=ia64&stamp=1093353284&file=log&as=raw
> http://buildd.debian.org/fetch.php?&pkg=cameleon&ver=1.3-4&arch=ia64&stamp=1092645134&file=log&as=raw
>
>
> convert main_window.jpg main_window.eps
> make[2]: *** [main_window.eps] Bus error

Little followup :

07:09 < kylem> svenl, this error is exceedingly gross.
07:13 < svenl> kylem: huh ?
07:13 < kylem> imagemagick
07:14 < svenl> kylem: yeah, but why you say the error is gross ?
07:15 < kylem> unaligned access.
07:15 < svenl> Oh.
07:15 < kylem> stack corruption in the core dump.

Thanks kylem for having a look at this.

Friendly,

Sven Luther

Revision history for this message
Debian Bug Importer (debzilla) wrote :

Message-ID: <email address hidden>
Date: Mon, 30 Aug 2004 20:33:58 +0200
From: Daniel Kobras <email address hidden>
To: Kyle McMartin <email address hidden>, <email address hidden>
Subject: Re: Bug#265540: imagemagick on ia64

clone 265540 -1 -2
severity -1 important
retitle -2 imagemagick: Header files kill attributes
thanks

There are two issues. The first has to do with conversions to postscript
being messed up. This is most likely a ghostscript bug. Haven't
investigated in detail. The second shows up on ia64 when trying to use
imagemagick on a jpeg file. I'm splitting the bug reports, therefore,
and am going to talk about the latter in the following.

On Fri, Aug 27, 2004 at 01:03:47AM -0400, Kyle McMartin wrote:
>
> The root cause seems to be,
>
> convert(1130): unaligned access to 0x60000fffffff8c28,
> ip=0x200000000104f090
> zsh: 1130 bus error (core dumped) convert graph.jpg graph.eps
>
> (gdb) bt
> #0 0x200000000104f090 in __sigsetjmp () from /lib/libc.so.6.1
> #1 0x0000000000000000 in ?? ()
> Previous frame identical to this frame (corrupt stack?)

The root cause actually is

 #if !defined(__attribute__)
 # define __attribute__(x) /*nothing*/
 #endif

sprinkled across several of imagemagick's header files. Too bad that
__attribute__ is no #define, and that some sources after the imagemagick
headers #include system headers that need to set attributes. The bug
reported here was caused by coders/jpeg.c #including <setjmp.h> that
#includes <bits/setjmp.h>, which on ia64 contains the line

 typedef long __jmp_buf[_JBLEN] __attribute__ ((aligned (16)));
 /* guarantees 128-bit alignment! */

The alignment restriction is killed by the above definition in the
imagemagick headers, and the setjmp call in ReadJPEGImage() triggers the
bus error due to an unaligned access. I'll follow up with the obvious
quick'n'dirty patch to fix the issue for us.

Regards,

Daniel.

Revision history for this message
Matt Zimmerman (mdz) wrote :

Not an issue for Warty architectures

Revision history for this message
Debian Bug Importer (debzilla) wrote :
Download full text (6.2 KiB)

Message-ID: <email address hidden>
Date: Mon, 30 Aug 2004 22:04:34 +0200
From: Daniel Kobras <email address hidden>
To: <email address hidden>
Subject: Re: imagemagick: Header files kill attributes

--6sX45UoQRIJXqkqR
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

tag 269085 + patch
thanks

Moi Ryuichi!

I have attached a patch that includes a fix for convert bailing out with
a bus error on ia64 machines. It also fixes a potential integer overflow
introduced with the security fix with the current upstream version.
Especially the first issue needs to be fixed urgently becaused it causes
build failures of other packages on ia64 at least. As you can see, I
have already prepared an NMU with these two changes that is tested on
ia64, and that I can upload without further effort. However, I'd be more
comfortable if you either gave me an okay to do the upload, or of course
prepare a maintainer upload including these changes. Full NMU diff to
your previous upload attached.

Regards,

Daniel.

--6sX45UoQRIJXqkqR
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline; filename=interdiff

diff -u imagemagick-6.0.6.2/debian/changelog imagemagick-6.0.6.2/debian/changelog
--- imagemagick-6.0.6.2/debian/changelog
+++ imagemagick-6.0.6.2/debian/changelog
@@ -1,3 +1,16 @@
+imagemagick (5:6.0.6.2-1.1) unstable; urgency=high
+
+ * Non-maintainer upload.
+ * magick/{deprecate,error,log,utility}.h: Remove bogus __attribute__
+ override that shadowed any wanted attribute settings (like vital
+ alignment restrictions). Cures convert bus errors on IA64.
+ Closes: #269085
+ * coders/{avi,bmp,dib}.c: Fix use of Min() macro to handle large
+ (>2 GB) image files as well.
+ * High urgency because the convert bus errors break builds on IA64.
+
+ -- Daniel Kobras <email address hidden> Mon, 30 Aug 2004 20:59:24 +0200
+
 imagemagick (5:6.0.6.2-1) unstable; urgency=low

   * New upstream release
only in patch2:
unchanged:
--- imagemagick-6.0.6.2.orig/magick/deprecate.h
+++ imagemagick-6.0.6.2/magick/deprecate.h
@@ -11,10 +11,6 @@
 #include "magick/blob.h"
 #include "magick/constitute.h"

-#if !defined(__attribute__)
-# define __attribute__(x) /*nothing*/
-#endif
-
 #define Downscale(quantum) ScaleQuantumToChar(quantum)
 #define False 0
 #define Intensity(color) PixelIntensityToQuantum(color)
only in patch2:
unchanged:
--- imagemagick-6.0.6.2.orig/magick/error.h
+++ imagemagick-6.0.6.2/magick/error.h
@@ -10,10 +10,6 @@

 #include "magick/log.h"

-#if !defined(__attribute__)
-# define __attribute__(x) /*nothing*/
-#endif
-
 #define ThrowBinaryException(severity,tag,context) \
 { \
   if (image != (Image *) NULL) \
only in patch2:
unchanged:
--- imagemagick-6.0.6.2.orig/magick/log.h
+++ imagemagick-6.0.6.2/magick/log.h
@@ -10,10 +10,6 @@

 #include "magick/error.h"

-#if !defined(__attribute__)
-# define __attribute__(x) /*nothing*/
-#endif
-
 #if !defined(GetUnadornedModuleName)
 # if (((defined(__cplusplus) || defined(c_plusplus)) && defined(HAS_CPP__func__)) || \
       (!(defined(__cplusplus) || defined(c_plusplus)) && defined(HAS_C__func__)))
only in patch2:
unchanged:
---...

Read more...

Revision history for this message
In , Daniel Kobras (kobras) wrote : Re: Bug#269085: imagemagick: Header files kill attributes

On Mon, Aug 30, 2004 at 10:04:34PM +0200, Daniel Kobras wrote:
> prepare a maintainer upload including these changes. Full NMU diff to
> your previous upload attached.

Even better, the patch below is incremental to the one I previously sent
out, and removes an illegal RPATH from Magick.so in the perlmagick
package. Furthermore, library dependencies of the various libs and
binaries on libmagick6 are now calculated with the proper shlibs
information from the built libmagick6. Feel free to apply.

Regards,

Daniel.

Revision history for this message
Debian Bug Importer (debzilla) wrote :

Message-ID: <email address hidden>
Date: Tue, 31 Aug 2004 00:21:46 +0200
From: Daniel Kobras <email address hidden>
To: <email address hidden>
Subject: Re: Bug#269085: imagemagick: Header files kill attributes

--sm4nu43k4a2Rpi4c
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

On Mon, Aug 30, 2004 at 10:04:34PM +0200, Daniel Kobras wrote:
> prepare a maintainer upload including these changes. Full NMU diff to
> your previous upload attached.

Even better, the patch below is incremental to the one I previously sent
out, and removes an illegal RPATH from Magick.so in the perlmagick
package. Furthermore, library dependencies of the various libs and
binaries on libmagick6 are now calculated with the proper shlibs
information from the built libmagick6. Feel free to apply.

Regards,

Daniel.

--sm4nu43k4a2Rpi4c
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline; filename=interdiff2

diff -u imagemagick-6.0.6.2/debian/changelog imagemagick-6.0.6.2/debian/changelog
--- imagemagick-6.0.6.2/debian/changelog
+++ imagemagick-6.0.6.2/debian/changelog
@@ -1,3 +1,13 @@
+imagemagick (5:6.0.6.2-1.2) unstable; urgency=low
+
+ * Non-maintainer upload
+ * debian/rules: Override LD_RUN_PATH to get rid of RPATH setting in
+ perl module.
+ * debian/rules: Use shlibs information of the libmagick6 just built
+ to determine library dependencies.
+
+ -- Daniel Kobras <email address hidden> Mon, 30 Aug 2004 23:39:02 +0200
+
 imagemagick (5:6.0.6.2-1.1) unstable; urgency=high

   * Non-maintainer upload.
diff -u imagemagick-6.0.6.2/debian/rules imagemagick-6.0.6.2/debian/rules
--- imagemagick-6.0.6.2/debian/rules
+++ imagemagick-6.0.6.2/debian/rules
@@ -48,6 +48,7 @@
  $(MAKE)
  cd $(CURDIR)/PerlMagick && perl Makefile.PL
  cd $(CURDIR)/PerlMagick && make \
+ LD_RUN_PATH= \
   EXTRALIBS='-L../magick/.libs -L/usr/lib -lMagick -L/usr/X11R6/lib -L/usr/lib -L/usr/lib -llcms -ltiff -lfreetype -ljpeg -lpng -ldpstk -ldps -lXext -lSM -lICE -lX11 -lbz2 -lxml2 -lz' \
   LDLOADLIBS='-L../magick/.libs -L/usr/lib -lMagick -L/usr/X11R6/lib -L/usr/lib -L/usr/lib -llcms -ltiff -lfreetype -ljpeg -lpng -ldpstk -ldps -lXext -lSM -lICE -lX11 -lbz2 -lxml2 -lz -lpthread -lm -lpthread'
  touch build-stamp
@@ -133,7 +134,7 @@
  dh_makeshlibs
  dh_installdeb
  dh_perl
- dh_shlibdeps
+ dh_shlibdeps -L libmagick6 -ldebian/libmagick6/usr/lib
  dh_gencontrol
  dh_md5sums
  dh_builddeb

--sm4nu43k4a2Rpi4c--

Revision history for this message
In , Daniel Kobras (kobras) wrote : Fixed in NMU of imagemagick 5:6.0.6.2-1.2
Download full text (3.1 KiB)

tag 269085 + fixed

quit

This message was generated automatically in response to a
non-maintainer upload. The .changes file follows.

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.7
Date: Mon, 30 Aug 2004 23:39:02 +0200
Source: imagemagick
Binary: perlmagick libmagick++6-dev libmagick6-dev libmagick6 imagemagick libmagick++6
Architecture: i386 ia64 source
Version: 5:6.0.6.2-1.2
Distribution: unstable
Urgency: low
Maintainer: Daniel Kobras <email address hidden>
Changed-By: Daniel Kobras <email address hidden>
Description:
 imagemagick - Image manipulation programs
 libmagick++6 - The object-oriented C++ API to the ImageMagick library
 libmagick++6-dev - The object-oriented C++ API to the ImageMagick library--developme
 libmagick6 - Image manipulation library (free version)
 libmagick6-dev - Image manipulation library (free version) -- development
 perlmagick - A perl interface to the libMagick graphics routines
Closes: 269085
Changes:
 imagemagick (5:6.0.6.2-1.2) unstable; urgency=low
 .
   * Non-maintainer upload
   * debian/rules: Override LD_RUN_PATH to get rid of RPATH setting in
     perl module.
   * debian/rules: Use shlibs information of the libmagick6 just built
     to determine library dependencies.
 .
 imagemagick (5:6.0.6.2-1.1) unstable; urgency=high
 .
   * Non-maintainer upload.
   * magick/{deprecate,error,log,utility}.h: Remove bogus __attribute__
     override that shadowed any wanted attribute settings (like vital
     alignment restrictions). Cures convert bus errors on IA64.
     Closes: #269085
   * coders/{avi,bmp,dib}.c: Fix use of Min() macro to handle large
     (>2 GB) image files as well.
   * High urgency because the convert bus errors break builds on IA64.
Files:
 eb33b2ef806e6de36acb7aabbabeeb8b 881 graphics optional imagemagick_6.0.6.2-1.2.dsc
 1da6599444970afefb41e72c89809457 185338 libs optional libmagick++6_6.0.6.2-1.2_ia64.deb
 28de5ae2f7490db753c599a7da29037b 1463808 graphics optional imagemagick_6.0.6.2-1.2_i386.deb
 338630a02ddc0d70b8f37cbfe1b628bb 206948 libdevel optional libmagick++6-dev_6.0.6.2-1.2_i386.deb
 371ab6530916570968fe0f4d3ae7b404 1466834 graphics optional imagemagick_6.0.6.2-1.2_ia64.deb
 3ef8ca4f3405e4f906e546a0d6562d12 1599828 libs optional libmagick6_6.0.6.2-1.2_ia64.deb
 424c41aed166da04da95fd5b1cbe10d9 1167344 libs optional libmagick6_6.0.6.2-1.2_i386.deb
 7ea911e45a8d60716f143dd92206f8e7 2123184 libdevel optional libmagick6-dev_6.0.6.2-1.2_ia64.deb
 87abf91a86c7e165bec379ea8c636919 141128 perl optional perlmagick_6.0.6.2-1.2_i386.deb
 95aab50e9ae49fbcda5c64508c15e9b8 1501622 libdevel optional libmagick6-dev_6.0.6.2-1.2_i386.deb
 a01593e125b914b73c9f7a9bd6f01d7f 294278 libdevel optional libmagick++6-dev_6.0.6.2-1.2_ia64.deb
 a4077915973e4285bf7a66bf57cbf926 171292 perl optional perlmagick_6.0.6.2-1.2_ia64.deb
 a57b87a2142cc770144213fe5038f8f0 134350 graphics optional imagemagick_6.0.6.2-1.2.diff.gz
 fc27b68b5e83ca23630b166c53bc9bc4 163094 libs optional libmagick++6_6.0.6.2-1.2_i386.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.5 (GNU/Linux)

iD8DBQFBNKwNpOKIA4m/fisRApgxAJ9zepd29ZNl5Z51xDFmdjrsgDM5QQCgr14n
EOlWkpSqoQCOOIj90nHWISY=
=4F56
-----END PGP SIGNATURE-----
...

Read more...

Revision history for this message
Debian Bug Importer (debzilla) wrote :
Download full text (3.4 KiB)

Message-Id: <email address hidden>
Date: Tue, 31 Aug 2004 13:17:09 -0400
From: Daniel Kobras <email address hidden>
To: <email address hidden>
Cc: Daniel Kobras <email address hidden>
Subject: Fixed in NMU of imagemagick 5:6.0.6.2-1.2

tag 269085 + fixed

quit

This message was generated automatically in response to a
non-maintainer upload. The .changes file follows.

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.7
Date: Mon, 30 Aug 2004 23:39:02 +0200
Source: imagemagick
Binary: perlmagick libmagick++6-dev libmagick6-dev libmagick6 imagemagick libmagick++6
Architecture: i386 ia64 source
Version: 5:6.0.6.2-1.2
Distribution: unstable
Urgency: low
Maintainer: Daniel Kobras <email address hidden>
Changed-By: Daniel Kobras <email address hidden>
Description:
 imagemagick - Image manipulation programs
 libmagick++6 - The object-oriented C++ API to the ImageMagick library
 libmagick++6-dev - The object-oriented C++ API to the ImageMagick library--developme
 libmagick6 - Image manipulation library (free version)
 libmagick6-dev - Image manipulation library (free version) -- development
 perlmagick - A perl interface to the libMagick graphics routines
Closes: 269085
Changes:
 imagemagick (5:6.0.6.2-1.2) unstable; urgency=low
 .
   * Non-maintainer upload
   * debian/rules: Override LD_RUN_PATH to get rid of RPATH setting in
     perl module.
   * debian/rules: Use shlibs information of the libmagick6 just built
     to determine library dependencies.
 .
 imagemagick (5:6.0.6.2-1.1) unstable; urgency=high
 .
   * Non-maintainer upload.
   * magick/{deprecate,error,log,utility}.h: Remove bogus __attribute__
     override that shadowed any wanted attribute settings (like vital
     alignment restrictions). Cures convert bus errors on IA64.
     Closes: #269085
   * coders/{avi,bmp,dib}.c: Fix use of Min() macro to handle large
     (>2 GB) image files as well.
   * High urgency because the convert bus errors break builds on IA64.
Files:
 eb33b2ef806e6de36acb7aabbabeeb8b 881 graphics optional imagemagick_6.0.6.2-1.2.dsc
 1da6599444970afefb41e72c89809457 185338 libs optional libmagick++6_6.0.6.2-1.2_ia64.deb
 28de5ae2f7490db753c599a7da29037b 1463808 graphics optional imagemagick_6.0.6.2-1.2_i386.deb
 338630a02ddc0d70b8f37cbfe1b628bb 206948 libdevel optional libmagick++6-dev_6.0.6.2-1.2_i386.deb
 371ab6530916570968fe0f4d3ae7b404 1466834 graphics optional imagemagick_6.0.6.2-1.2_ia64.deb
 3ef8ca4f3405e4f906e546a0d6562d12 1599828 libs optional libmagick6_6.0.6.2-1.2_ia64.deb
 424c41aed166da04da95fd5b1cbe10d9 1167344 libs optional libmagick6_6.0.6.2-1.2_i386.deb
 7ea911e45a8d60716f143dd92206f8e7 2123184 libdevel optional libmagick6-dev_6.0.6.2-1.2_ia64.deb
 87abf91a86c7e165bec379ea8c636919 141128 perl optional perlmagick_6.0.6.2-1.2_i386.deb
 95aab50e9ae49fbcda5c64508c15e9b8 1501622 libdevel optional libmagick6-dev_6.0.6.2-1.2_i386.deb
 a01593e125b914b73c9f7a9bd6f01d7f 294278 libdevel optional libmagick++6-dev_6.0.6.2-1.2_ia64.deb
 a4077915973e4285bf7a66bf57cbf926 171292 perl optional perlmagick_6.0.6.2-1.2_ia64.deb
 a57b87a2142cc770144213fe5038f8f0 134350 graphics optional imagemagick_6.0.6.2-1.2.diff.gz
 fc27b68b5e83ca23630b166c53bc9bc...

Read more...

Revision history for this message
In , Ryuichi Arafune (arafune) wrote : Bug#269085: fixed in imagemagick 6:6.2.3.6-1
Download full text (4.1 KiB)

Source: imagemagick
Source-Version: 6:6.2.3.6-1

We believe that the bug you reported is fixed in the latest version of
imagemagick, which is due to be installed in the Debian FTP archive:

imagemagick_6.2.3.6-1.diff.gz
  to pool/main/i/imagemagick/imagemagick_6.2.3.6-1.diff.gz
imagemagick_6.2.3.6-1.dsc
  to pool/main/i/imagemagick/imagemagick_6.2.3.6-1.dsc
imagemagick_6.2.3.6-1_i386.deb
  to pool/main/i/imagemagick/imagemagick_6.2.3.6-1_i386.deb
imagemagick_6.2.3.6.orig.tar.gz
  to pool/main/i/imagemagick/imagemagick_6.2.3.6.orig.tar.gz
libmagick++6-dev_6.2.3.6-1_i386.deb
  to pool/main/i/imagemagick/libmagick++6-dev_6.2.3.6-1_i386.deb
libmagick++6c2_6.2.3.6-1_i386.deb
  to pool/main/i/imagemagick/libmagick++6c2_6.2.3.6-1_i386.deb
libmagick6-dev_6.2.3.6-1_i386.deb
  to pool/main/i/imagemagick/libmagick6-dev_6.2.3.6-1_i386.deb
libmagick6_6.2.3.6-1_i386.deb
  to pool/main/i/imagemagick/libmagick6_6.2.3.6-1_i386.deb
perlmagick_6.2.3.6-1_i386.deb
  to pool/main/i/imagemagick/perlmagick_6.2.3.6-1_i386.deb

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed. If you
have further comments please address them to <email address hidden>,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Ryuichi Arafune <email address hidden> (supplier of updated imagemagick package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing <email address hidden>)

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.7
Date: Thu, 4 Aug 2005 12:39:54 +0900
Source: imagemagick
Binary: perlmagick libmagick++6c2 libmagick++6-dev libmagick6-dev libmagick6 imagemagick
Architecture: source i386
Version: 6:6.2.3.6-1
Distribution: unstable
Urgency: low
Maintainer: Ryuichi Arafune <email address hidden>
Changed-By: Ryuichi Arafune <email address hidden>
Description:
 imagemagick - Image manipulation programs
 libmagick++6-dev - The object-oriented C++ API to the ImageMagick library--developme
 libmagick++6c2 - The object-oriented C++ API to the ImageMagick library
 libmagick6 - Image manipulation library
 libmagick6-dev - Image manipulation library -- development
 perlmagick - A perl interface to the libMagick graphics routines
Closes: 264033 265540 266146 268357 269085 270882 277775 277795 278401 282173 291033 291118 296084 297990 302093 303765 306424 310690 310812 315629 316475 317299 317628 318255 321208
Changes:
 imagemagick (6:6.2.3.6-1) unstable; urgency=low
 .
   * New upstream release
   * upstream fixes:
      - fix typo in mogrify manpage: closes: #317628, #321208
      - update config.sub/config.guess closes: #317299
      - fix " configure.ac takes wrong assumptions" closes: #303765
   * point to the correct URL in manpages. closes: #318255, #315629
   * man pages are rerwrited. closes: #264033, #316475
   * closing bugs fixed by NMs. closes: #310690, #310812, #268357, #269085, #278401, #291033, #291118, #297990, #302093, #265540, #296084, #277775, #306424, #266146, #270882, #282173, #277795,
Files:
 68c8b4eef9526747...

Read more...

Revision history for this message
Debian Bug Importer (debzilla) wrote :
Download full text (4.3 KiB)

Message-Id: <email address hidden>
Date: Wed, 03 Aug 2005 22:32:09 -0700
From: Ryuichi Arafune <email address hidden>
To: <email address hidden>
Subject: Bug#269085: fixed in imagemagick 6:6.2.3.6-1

Source: imagemagick
Source-Version: 6:6.2.3.6-1

We believe that the bug you reported is fixed in the latest version of
imagemagick, which is due to be installed in the Debian FTP archive:

imagemagick_6.2.3.6-1.diff.gz
  to pool/main/i/imagemagick/imagemagick_6.2.3.6-1.diff.gz
imagemagick_6.2.3.6-1.dsc
  to pool/main/i/imagemagick/imagemagick_6.2.3.6-1.dsc
imagemagick_6.2.3.6-1_i386.deb
  to pool/main/i/imagemagick/imagemagick_6.2.3.6-1_i386.deb
imagemagick_6.2.3.6.orig.tar.gz
  to pool/main/i/imagemagick/imagemagick_6.2.3.6.orig.tar.gz
libmagick++6-dev_6.2.3.6-1_i386.deb
  to pool/main/i/imagemagick/libmagick++6-dev_6.2.3.6-1_i386.deb
libmagick++6c2_6.2.3.6-1_i386.deb
  to pool/main/i/imagemagick/libmagick++6c2_6.2.3.6-1_i386.deb
libmagick6-dev_6.2.3.6-1_i386.deb
  to pool/main/i/imagemagick/libmagick6-dev_6.2.3.6-1_i386.deb
libmagick6_6.2.3.6-1_i386.deb
  to pool/main/i/imagemagick/libmagick6_6.2.3.6-1_i386.deb
perlmagick_6.2.3.6-1_i386.deb
  to pool/main/i/imagemagick/perlmagick_6.2.3.6-1_i386.deb

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed. If you
have further comments please address them to <email address hidden>,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Ryuichi Arafune <email address hidden> (supplier of updated imagemagick package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing <email address hidden>)

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.7
Date: Thu, 4 Aug 2005 12:39:54 +0900
Source: imagemagick
Binary: perlmagick libmagick++6c2 libmagick++6-dev libmagick6-dev libmagick6 imagemagick
Architecture: source i386
Version: 6:6.2.3.6-1
Distribution: unstable
Urgency: low
Maintainer: Ryuichi Arafune <email address hidden>
Changed-By: Ryuichi Arafune <email address hidden>
Description:
 imagemagick - Image manipulation programs
 libmagick++6-dev - The object-oriented C++ API to the ImageMagick library--developme
 libmagick++6c2 - The object-oriented C++ API to the ImageMagick library
 libmagick6 - Image manipulation library
 libmagick6-dev - Image manipulation library -- development
 perlmagick - A perl interface to the libMagick graphics routines
Closes: 264033 265540 266146 268357 269085 270882 277775 277795 278401 282173 291033 291118 296084 297990 302093 303765 306424 310690 310812 315629 316475 317299 317628 318255 321208
Changes:
 imagemagick (6:6.2.3.6-1) unstable; urgency=low
 .
   * New upstream release
   * upstream fixes:
      - fix typo in mogrify manpage: closes: #317628, #321208
      - update config.sub/config.guess closes: #317299
      - fix " configure.ac takes wrong assumptions" closes: #303765
   * point to the correct URL in manpages. closes: #318255, #315629
   * man pages are rerwrited. closes: #264033, #316475
...

Read more...

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.