You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by mt...@apache.org on 2008/04/18 15:51:07 UTC

svn commit: r649541 - /tomcat/connectors/trunk/jk/native/common/jk_lb_worker.c

Author: mturk
Date: Fri Apr 18 06:51:05 2008
New Revision: 649541

URL: http://svn.apache.org/viewvc?rev=649541&view=rev
Log:
Free resources if failed and updated OK status

Modified:
    tomcat/connectors/trunk/jk/native/common/jk_lb_worker.c

Modified: tomcat/connectors/trunk/jk/native/common/jk_lb_worker.c
URL: http://svn.apache.org/viewvc/tomcat/connectors/trunk/jk/native/common/jk_lb_worker.c?rev=649541&r1=649540&r2=649541&view=diff
==============================================================================
--- tomcat/connectors/trunk/jk/native/common/jk_lb_worker.c (original)
+++ tomcat/connectors/trunk/jk/native/common/jk_lb_worker.c Fri Apr 18 06:51:05 2008
@@ -1062,6 +1062,7 @@
         jk_log(l, JK_LOG_ERROR,
                "Failed allocating AJP message");
         JK_TRACE_EXIT(l);
+        free(states);
         return JK_SERVER_ERROR;
     }
     if (jk_b_set_buffer_size(s->reco_buf, p->worker->max_packet_size)) {
@@ -1069,6 +1070,7 @@
         jk_log(l, JK_LOG_ERROR,
                "Failed allocating AJP message buffer");
         JK_TRACE_EXIT(l);
+        free(states);
         return JK_SERVER_ERROR;
     }
     jk_b_reset(s->reco_buf);
@@ -1231,7 +1233,8 @@
                 if (rec->s->busy)
                     rec->s->busy--;
                 if (service_stat == JK_TRUE) {
-                    rec->s->state = JK_LB_STATE_OK;
+                    rec->s->state  = JK_LB_STATE_OK;
+                    states[rec->i] = JK_LB_STATE_OK;
                     rec->s->error_time = 0;
                     rc = JK_TRUE;
                     recoverable = JK_UNSET;



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org