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=42002>.
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=42002

           Summary: Error when no JkWorkersFile  used (JK 1.2.21)
           Product: Tomcat 5
           Version: Unknown
          Platform: All
        OS/Version: Linux
            Status: NEW
          Severity: regression
          Priority: P2
         Component: Native:JK
        AssignedTo: tomcat-dev@jakarta.apache.org
        ReportedBy: [EMAIL PROTECTED]


Hi,
As I upgraded from 1.2.19 to 1.2.21, I hit the following regression :
a Wrong [emerg] "Error in reading worker properties" is thrown because I don't
have any JkWorkersFile defined.
mod_jk.c should silently accept this, while the second part of test about empty
map looks correct.
I suggest following fix (wich works for me but may need more test) :

--- mod_jk.c.orig       2007-02-26 20:09:41.000000000 +0100
+++ mod_jk.c    2007-03-30 22:47:31.000000000 +0200
@@ -2658,17 +2658,16 @@
     jk_set_worker_def_cache_size(mpm_threads);

     /*     if(map_alloc(&init_map)) { */
-    if (!jk_map_read_properties(init_map, conf->worker_file, NULL, 1, 
conf->log)) {
-        if (jk_map_size(init_map) == 0) {
-            ap_log_error(APLOG_MARK, APLOG_STARTUP | APLOG_CRIT,
-                         0, NULL,
-                         "No worker file and no worker options in httpd.conf"
-                         "use JkWorkerFile to set workers");
-        }
+    if ((conf->worker_file!=NULL) && !jk_map_read_properties(init_map,
conf->worker_file, NULL, 1, conf->log)) {
         ap_log_error(APLOG_MARK, APLOG_EMERG | APLOG_NOERRNO, 0, NULL, "Error
in reading worker properties");
         return !OK;
     }
-
+    if (jk_map_size(init_map) == 0) {
+        ap_log_error(APLOG_MARK, APLOG_STARTUP | APLOG_CRIT,
+                     0, NULL,
+                     "No worker file and no worker options in httpd.conf"
+                     "use JkWorkerFile to set workers");
+    }
     if (jk_map_resolve_references(init_map, "worker.", 1, 1, conf->log) ==
JK_FALSE) {
         ap_log_error(APLOG_MARK, APLOG_EMERG | APLOG_NOERRNO, 0, NULL, "Error
in resolving configuration references");
         return !OK;

-- 
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]

Reply via email to