Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PerlIO::encoding can croak, aborting cleanup #12551

Open
p5pRT opened this issue Nov 12, 2012 · 4 comments
Open

PerlIO::encoding can croak, aborting cleanup #12551

p5pRT opened this issue Nov 12, 2012 · 4 comments

Comments

@p5pRT
Copy link

p5pRT commented Nov 12, 2012

Migrated from rt.perl.org#115692 (status was 'open')

Searchable as RT115692$

@p5pRT
Copy link
Author

p5pRT commented Nov 12, 2012

From @cpansprout

A badly-behaved encoding implementation can cause PerlIO​::encoding to croak during flush, which can happen during stack unwinding, when the handle is freed, resulting in cleanup being aborted. This can cause HEKs to leak, resulting in unbalanced string table warnings.

I don’t know what the best solution is. Right now we have a test in encoding.t that triggers this. It was meant to be a test for a crashing bug, now fixed, but it triggers this other bug.

See also <http​://www.nntp.perl.org/group/perl.perl5.porters/;msgid=CANgJU+ULEx2NZMwyuT9jnwKq+Zw1Bmdk_sDEL-e3XQQWnBvWVA@​mail.gmail.com>.


Flags​:
  category=core
  severity=low


Site configuration information for perl 5.17.6​:

Configured by sprout at Tue Nov 6 10​:02​:52 PST 2012.

Summary of my perl5 (revision 5 version 17 subversion 6) configuration​:
  Local Commit​: b69c1f1694a02073567c19b0768e7ce81e09913b
  Ancestor​: dea1691
  Platform​:
  osname=darwin, osvers=10.5.0, archname=darwin-thread-multi-2level
  uname='darwin pint.local 10.5.0 darwin kernel version 10.5.0​: fri nov 5 23​:20​:39 pdt 2010; root​:xnu-1504.9.17~1release_i386 i386 '
  config_args='-de -DDEBUGGING -Duseithreads -Dusedevel'
  hint=recommended, useposix=true, d_sigaction=define
  useithreads=define, usemultiplicity=define
  useperlio=define, d_sfio=undef, uselargefiles=define, usesocks=undef
  use64bitint=undef, use64bitall=undef, uselongdouble=undef
  usemymalloc=n, bincompat5005=undef
  Compiler​:
  cc='cc', ccflags ='-fno-common -DPERL_DARWIN -DDEBUGGING -fno-strict-aliasing -pipe -fstack-protector -I/usr/local/include',
  optimize='-O3 -g',
  cppflags='-fno-common -DPERL_DARWIN -DDEBUGGING -fno-strict-aliasing -pipe -fstack-protector -I/usr/local/include'
  ccversion='', gccversion='4.2.1 (Apple Inc. build 5664)', gccosandvers=''
  intsize=4, longsize=4, ptrsize=4, doublesize=8, byteorder=1234
  d_longlong=define, longlongsize=8, d_longdbl=define, longdblsize=16
  ivtype='long', ivsize=4, nvtype='double', nvsize=8, Off_t='off_t', lseeksize=8
  alignbytes=8, prototype=define
  Linker and Libraries​:
  ld='env MACOSX_DEPLOYMENT_TARGET=10.3 cc', ldflags =' -fstack-protector -L/usr/local/lib'
  libpth=/usr/local/lib /usr/lib
  libs=-ldbm -ldl -lm -lutil -lc
  perllibs=-ldl -lm -lutil -lc
  libc=, so=dylib, useshrplib=false, libperl=libperl.a
  gnulibc_version=''
  Dynamic Linking​:
  dlsrc=dl_dlopen.xs, dlext=bundle, d_dlsymun=undef, ccdlflags=' '
  cccdlflags=' ', lddlflags=' -bundle -undefined dynamic_lookup -L/usr/local/lib -fstack-protector'

Locally applied patches​:
 


@​INC for perl 5.17.6​:
  /usr/local/lib/perl5/site_perl/5.17.6/darwin-thread-multi-2level
  /usr/local/lib/perl5/site_perl/5.17.6
  /usr/local/lib/perl5/5.17.6/darwin-thread-multi-2level
  /usr/local/lib/perl5/5.17.6
  /usr/local/lib/perl5/site_perl
  .


Environment for perl 5.17.6​:
  DYLD_LIBRARY_PATH (unset)
  HOME=/Users/sprout
  LANG=en_US.UTF-8
  LANGUAGE (unset)
  LD_LIBRARY_PATH (unset)
  LOGDIR (unset)
  PATH=/usr/bin​:/bin​:/usr/sbin​:/sbin​:/usr/local/bin​:/usr/X11/bin​:/usr/local/bin
  PERL_BADLANG (unset)
  SHELL=/bin/bash

@p5pRT
Copy link
Author

p5pRT commented Dec 4, 2012

From @cpansprout

On Sun Nov 11 22​:10​:12 2012, sprout wrote​:

I don’t know what the best solution is. Right now we have a test in
encoding.t that triggers this. It was meant to be a test for a
crashing bug, now fixed, but it triggers this other bug.

I’ve added a skip for the offending test under -DDEBUGGING +
PERL_DESTRUCT_LEVEL in commit 668a862.

--

Father Chrysostomos

@p5pRT
Copy link
Author

p5pRT commented Dec 4, 2012

From [Unknown Contact. See original ticket]

On Sun Nov 11 22​:10​:12 2012, sprout wrote​:

I don’t know what the best solution is. Right now we have a test in
encoding.t that triggers this. It was meant to be a test for a
crashing bug, now fixed, but it triggers this other bug.

I’ve added a skip for the offending test under -DDEBUGGING +
PERL_DESTRUCT_LEVEL in commit 668a862.

--

Father Chrysostomos

@p5pRT
Copy link
Author

p5pRT commented Dec 4, 2012

@cpansprout - Status changed from 'new' to 'open'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants