Skip Menu |

This queue is for tickets about the RPC-XML CPAN distribution.

Report information
The Basics
Id: 65616
Status: resolved
Priority: 0/
Queue: RPC-XML
People
Owner: Nobody in particular
Requestors: apodran [...] gmail.com
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in: 0.74
Fixed in: 0.76

History Show all quoted text
  Thu Feb 10 06:20:50 2011 https://odisseyrv.myopenid.com/ - Ticket created
Subject: RPC::XML::Server dies on very slow methods
RPC::XML::Server dies on very slow methods without any message. It happens on methods with execution time more than ~20 seconds on Debian Lenny. When the SIGPIPE is caught in the "process_request" function (as I see it is the fix for the #54183 ticket), RPC::XML::Server dies on execution of the following line: ----------------------------------------------- $conn->close; ----------------------------------------------- Placed it to "eval" block resolved this issue in most cases but not always. I've resolved it by adding a local SIGPIPE handler into this "eval" block. OS: Debian 5.0 Perl: 5.10.0 RPC::XML: 0.74
  Thu Feb 10 06:23:29 2011 https://odisseyrv.myopenid.com/ - Correspondence added
Here is the patch I propose to resolve the issue.
Subject: rpc-xml-server_slow-methods-crash.patch
--- RPC-XML-0.74/lib/RPC/XML/Server.pm 2011-01-22 23:05:19.000000000 +0200 +++ RPC-XML-0.74_patched/lib/RPC/XML/Server.pm 2011-02-10 12:19:11.123903917 +0200 @@ -1592,7 +1592,15 @@ sub server_loop ## no critic (RequireArg } $conn->timeout($self->timeout); $self->process_request($conn); - $conn->close; + + eval { + local $SIG{PIPE} = sub { die "server_loop: Caught SIGPIPE\n"; }; + $conn->close; + }; + if ( $@ ) { + warn "Cannot close conection: $@\n"; + } + undef $conn; # Free up any lingering resources }
  Thu Feb 10 06:23:30 2011 The RT System itself - Status changed from 'new' to 'open'
  Thu Aug 18 22:38:54 2011 rjray [...] blackperl.com - Correspondence added
On Thu Feb 10 06:23:29 2011, https://odisseyrv.myopenid.com/ wrote: Show quoted text
> Here is the patch I propose to resolve the issue.
Patch applied, with a little adjustment to how I check the eval return. This will be in 0.76. Randy -- Randy J. Ray rjray@blackperl.com randy.j.ray@gmail.com
  Sun Aug 21 21:51:55 2011 rjray [...] blackperl.com - Fixed in 0.76 added
  Sun Aug 21 21:51:55 2011 rjray [...] blackperl.com - Correspondence added
Fix delivered in 0.76 release. -- Randy J. Ray rjray@blackperl.com randy.j.ray@gmail.com
  Sun Aug 21 21:51:56 2011 rjray [...] blackperl.com - Status changed from 'open' to 'resolved'