Mailing List Archive: ModPerl: Dev

Fwd: [rt.cpan.org #66894] Apache::SizeLimit: Linux::Smaps detection broken

Index | Next | Previous | View Threaded

fred at redhotpenguin

Mar 25, 2011, 9:58 AM

Post #1 of 2 (573 views)
Permalink

Fwd: [rt.cpan.org #66894] Apache::SizeLimit: Linux::Smaps detection broken

---------- Forwarded message ----------
From: Christian Ruppert via RT <bug-mod_perl [at] rt>
Date: Fri, Mar 25, 2011 at 9:46 AM
Subject: [rt.cpan.org #66894] Apache::SizeLimit: Linux::Smaps detection broken
To:

Fri Mar 25 12:46:38 2011: Request 66894 was acted upon.
Transaction: Ticket created by idl0r
Queue: mod_perl
Subject: Apache::SizeLimit: Linux::Smaps detection broken
Broken in: 2.0.5
Severity: Normal
Owner: Nobody
Requestors: idl0r [at] qasl
Status: new
Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=66894 >

Hey guys,

it looks like the Linux::Smaps detection should be improved to *not* die
if /proc/self/smaps doesn't exist. It's quite easy by just moving the
curly bracket.
Please take a look at my patch where I it has been fixed already.

Attachments:

0001-Fix-Linux-Smaps-detection-to-not-die-if-proc-self-sm.patch (0.83 KB)

torsten.foertsch at gmx

Mar 25, 2011, 10:07 AM

Post #2 of 2 (560 views)
Permalink

Re: Fwd: [rt.cpan.org #66894] Apache::SizeLimit: Linux::Smaps detection broken [In reply to]

On Friday, March 25, 2011 17:58:26 Fred Moyer wrote:
> - if (eval { require Linux::Smaps } && Linux::Smaps->new($$)) {
> + if (eval { require Linux::Smaps && Linux::Smaps->new($$) }) {

+1

Torsten Förtsch

--
Need professional modperl support? Hire me! (http://foertsch.name)

Like fantasy? http://kabatinte.net

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe [at] perl
For additional commands, e-mail: dev-help [at] perl

Index | Next | Previous | View Threaded

Interested in having your list archived? Contact Gossamer Threads