Bug #5995 for GD: Core dump in newFromPng

Tue Apr 13 08:35:54 2004 SREZIC [...] cpan.org - Ticket created

Subject:

Core dump in newFromPng

GD::Image::newFromPng (may) dump core if the the supplied file is not a PNG file. Example (reproducible on FreeBSD 4.9 and RedHat Linux 8.0, both with perl5.8.0): #!/usr/bin/perl -w use GD; $image = GD::Image->newFromPng("/etc/passwd"); __END__ A gdb backtrace shows: (gdb) bt #0 0x402573ae in gd_chkimagefmt (image=0x0, truecolor=0) at GD.xs:499 #1 0x40257cfc in XS_GD__Image__newFromPng (cv=0x8251bb8) at GD.xs:566 #2 0x080cd4f0 in Perl_pp_entersub () at pp_hot.c:2854 #3 0x080afc71 in Perl_runops_debug () at dump.c:1442 #4 0x080624ca in S_run_body (oldscope=1) at perl.c:1921 #5 0x08062059 in perl_run (my_perl=0x816df68) at perl.c:1840 #6 0x0805e501 in main (argc=2, argv=0xbfffdc54, env=0xbfffdc60) at perlmain.c:86 #7 0x420158d4 in __libc_start_main () from /lib/i686/libc.so.6 Regards, Slaven

Tue Apr 13 11:10:40 2004 LDS [...] cpan.org - Status changed from 'new' to 'resolved'

Tue Apr 13 11:10:40 2004 LDS [...] cpan.org - Correspondence added

This is an issue with libgd. Please refer the bug report to Tom Boutell.

Tue Apr 13 11:20:57 2004 The RT System itself - Status changed from 'resolved' to 'open'

Tue Apr 13 11:20:57 2004 SREZIC [...] cpan.org - Correspondence added

From:

slaven [...] rezic.de

[LDS - Tue Apr 13 11:10:40 2004]: Show quoted text

> This is an issue with libgd. Please refer the bug report to Tom Boutell.

I don't think so. gd_chkimagefmt() is defined in GD.xs. Probably it should be changed to (not tested): void gd_chkimagefmt(GD__Image image, int truecolor) { if (image && !truecolor) { /* return a palette image */ if (gdImageTrueColor(image)) { gdImageTrueColorToPalette(image,1,gdMaxColors); } } } Regards, Slaven

Mon Mar 07 13:20:13 2005 LDS [...] cpan.org - Status changed from 'open' to 'resolved'

Mon Mar 07 13:20:13 2005 LDS [...] cpan.org - Correspondence added 1 min

I'm guessing that this was a bug in libgd (the C library). I can't reproduce it with libgd version 2.0.28, and so I hope it has been fixed since the original report was filed. If the problem is persistent, I might be able to work around this by checking the file types before passing the file names to the gd layer. Lincoln [SREZIC - Tue Apr 13 08:35:54 2004]: Show quoted text

> GD::Image::newFromPng (may) dump core if the the supplied file is not > a > PNG file. Example (reproducible on FreeBSD 4.9 and RedHat Linux 8.0, > both with perl5.8.0): > > #!/usr/bin/perl -w > use GD; > $image = GD::Image->newFromPng("/etc/passwd"); > __END__ > > A gdb backtrace shows: > > (gdb) bt > #0 0x402573ae in gd_chkimagefmt (image=0x0, truecolor=0) at

GD.xs:499 Show quoted text

> #1 0x40257cfc in XS_GD__Image__newFromPng (cv=0x8251bb8) at

GD.xs:566 Show quoted text

> #2 0x080cd4f0 in Perl_pp_entersub () at pp_hot.c:2854 > #3 0x080afc71 in Perl_runops_debug () at dump.c:1442 > #4 0x080624ca in S_run_body (oldscope=1) at perl.c:1921 > #5 0x08062059 in perl_run (my_perl=0x816df68) at perl.c:1840 > #6 0x0805e501 in main (argc=2, argv=0xbfffdc54, env=0xbfffdc60) > at perlmain.c:86 > #7 0x420158d4 in __libc_start_main () from /lib/i686/libc.so.6 > > Regards, > Slaven

Mon Mar 07 13:20:13 2005 LDS [...] cpan.org - Given to LDS

Tue Mar 08 04:36:11 2005 The RT System itself - Status changed from 'resolved' to 'open'

Tue Mar 08 04:36:11 2005 SREZIC [...] cpan.org - Correspondence added

From:

srezic [...] cpan.org

[LDS - Mon Mar 7 13:20:13 2005]: Show quoted text

> I'm guessing that this was a bug in libgd (the C library). I can't > reproduce it with libgd version 2.0.28, and so I hope it has been fixed > since the original report was filed. If the problem is persistent, I > might be able to work around this by checking the file types before > passing the file names to the gd layer. >

I cannot reproduce the segfault with GD 2.19, libgd 2.0.28 on the Linux machine, so it seems really to be fixed. Regards, Slaven Show quoted text

> Lincoln > > [SREZIC - Tue Apr 13 08:35:54 2004]: >

> > GD::Image::newFromPng (may) dump core if the the supplied file is not > > a > > PNG file. Example (reproducible on FreeBSD 4.9 and RedHat Linux 8.0, > > both with perl5.8.0): > > > > #!/usr/bin/perl -w > > use GD; > > $image = GD::Image->newFromPng("/etc/passwd"); > > __END__ > > > > A gdb backtrace shows: > > > > (gdb) bt > > #0 0x402573ae in gd_chkimagefmt (image=0x0, truecolor=0) at

> GD.xs:499

> > #1 0x40257cfc in XS_GD__Image__newFromPng (cv=0x8251bb8) at

> GD.xs:566

> > #2 0x080cd4f0 in Perl_pp_entersub () at pp_hot.c:2854 > > #3 0x080afc71 in Perl_runops_debug () at dump.c:1442 > > #4 0x080624ca in S_run_body (oldscope=1) at perl.c:1921 > > #5 0x08062059 in perl_run (my_perl=0x816df68) at perl.c:1840 > > #6 0x0805e501 in main (argc=2, argv=0xbfffdc54, env=0xbfffdc60) > > at perlmain.c:86 > > #7 0x420158d4 in __libc_start_main () from /lib/i686/libc.so.6 > > > > Regards, > > Slaven

> > >

Fri Apr 21 18:58:36 2017 RURBAN [...] cpan.org - Correspondence added

I've added proper error handling after libgd errors with 2.59 -- Reini Urban

Fri Apr 21 18:58:36 2017 RURBAN [...] cpan.org - Status changed from 'open' to 'resolved'