DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG· RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT <http://issues.apache.org/bugzilla/show_bug.cgi?id=44402>. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND· INSERTED IN THE BUG DATABASE.
http://issues.apache.org/bugzilla/show_bug.cgi?id=44402 [EMAIL PROTECTED] changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |NEEDINFO ------- Additional Comments From [EMAIL PROTECTED] 2008-02-13 13:03 ------- First guess for your last crash in comment #4 (all line numbers 2.2.8): lr->accept_func(&csd, lr, ptrans); (line 742 in worker.c) fails with rv != APR_SUCCESS, but with a non NULL value for csd. In contrast to the code in prefork we don't check this situation: Lines 621 - 631 of prefork.c: status = lr->accept_func(&csd, lr, ptrans); SAFE_ACCEPT(accept_mutex_off()); /* unlock after "accept" */ if (status == APR_EGENERAL) { /* resource shortage or should-not-occur occured */ clean_child_exit(1); } else if (status != APR_SUCCESS) { continue; } Maybe we need to do a continue in the worker case as well or we need to do something like the following: Index: server/mpm/worker/worker.c =================================================================== --- server/mpm/worker/worker.c (Revision 627576) +++ server/mpm/worker/worker.c (Arbeitskopie) @@ -743,6 +743,9 @@ /* later we trash rv and rely on csd to indicate success/failure */ AP_DEBUG_ASSERT(rv == APR_SUCCESS || !csd); + if (rv != APR_SUCCESS) { + csd = NULL; + } if (rv == APR_EGENERAL) { /* E[NM]FILE, ENOMEM, etc */ resource_shortage = 1; -- Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee. --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
