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

Failed XSLoader::load() leaks memory #15348

Open
p5pRT opened this issue May 22, 2016 · 5 comments
Open

Failed XSLoader::load() leaks memory #15348

p5pRT opened this issue May 22, 2016 · 5 comments
Labels

Comments

@p5pRT
Copy link

p5pRT commented May 22, 2016

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

Searchable as RT128209$

@p5pRT
Copy link
Author

p5pRT commented May 22, 2016

From @cpansprout

$ ./perl -ILib -e 'use XSLoader; warn $$; eval { XSLoader​::load() } while 1'

Run that script and watch the memory usage go up. I found this with Porting/leakfinder.pl, so it is an SV leak.

--

Father Chrysostomos

@p5pRT
Copy link
Author

p5pRT commented May 23, 2016

From @jkeenan

On Sat May 21 17​:33​:17 2016, sprout wrote​:

$ ./perl -ILib -e 'use XSLoader; warn $$; eval { XSLoader​::load() }
while 1'

Run that script and watch the memory usage go up. I found this with
Porting/leakfinder.pl, so it is an SV leak.

1. I suspect that the 'L' in the second term should be lower-case, i.e.​:

./perl -Ilib -e 'use XSLoader; warn $$; eval { XSLoader​::load() } while 1'

2. What is a good way to "watch the memory usage go up"?

--
James E Keenan (jkeenan@​cpan.org)

@p5pRT
Copy link
Author

p5pRT commented May 23, 2016

The RT System itself - Status changed from 'new' to 'open'

@p5pRT
Copy link
Author

p5pRT commented May 23, 2016

From @cpansprout

On Sun May 22 17​:34​:22 2016, jkeenan wrote​:

On Sat May 21 17​:33​:17 2016, sprout wrote​:

$ ./perl -ILib -e 'use XSLoader; warn $$; eval { XSLoader​::load() }
while 1'

Run that script and watch the memory usage go up. I found this with
Porting/leakfinder.pl, so it is an SV leak.

1. I suspect that the 'L' in the second term should be lower-case, i.e.​:

./perl -Ilib -e 'use XSLoader; warn $$; eval { XSLoader​::load() } while 1'

Yes. I often make that typo, but it makes no difference on a case-tolerant file system.

2. What is a good way to "watch the memory usage go up"?

I use top.

--

Father Chrysostomos

@p5pRT
Copy link
Author

p5pRT commented May 23, 2016

From @demerphq

ps $pid
On 22 May 2016 21​:25, "Father Chrysostomos via RT" <
perlbug-followup@​perl.org> wrote​:

On Sun May 22 17​:34​:22 2016, jkeenan wrote​:

On Sat May 21 17​:33​:17 2016, sprout wrote​:

$ ./perl -ILib -e 'use XSLoader; warn $$; eval { XSLoader​::load() }
while 1'

Run that script and watch the memory usage go up. I found this with
Porting/leakfinder.pl, so it is an SV leak.

1. I suspect that the 'L' in the second term should be lower-case, i.e.​:

./perl -Ilib -e 'use XSLoader; warn $$; eval { XSLoader​::load() } while
1'

Yes. I often make that typo, but it makes no difference on a
case-tolerant file system.

2. What is a good way to "watch the memory usage go up"?

I use top.

--

Father Chrysostomos

---
via perlbug​: queue​: perl5 status​: open
https://rt-archive.perl.org/perl5/Ticket/Display.html?id=128209

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

No branches or pull requests

2 participants