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 2011/11/08 09:45:13 UTC
svn commit: r1199158 - /tomcat/jk/trunk/native/iis/jk_isapi_plugin.c
Author: mturk
Date: Tue Nov 8 08:45:13 2011
New Revision: 1199158
URL: http://svn.apache.org/viewvc?rev=1199158&view=rev
Log:
Skip useless config read in case of failure
Modified:
tomcat/jk/trunk/native/iis/jk_isapi_plugin.c
Modified: tomcat/jk/trunk/native/iis/jk_isapi_plugin.c
URL: http://svn.apache.org/viewvc/tomcat/jk/trunk/native/iis/jk_isapi_plugin.c?rev=1199158&r1=1199157&r2=1199158&view=diff
==============================================================================
--- tomcat/jk/trunk/native/iis/jk_isapi_plugin.c (original)
+++ tomcat/jk/trunk/native/iis/jk_isapi_plugin.c Tue Nov 8 08:45:13 2011
@@ -2891,6 +2891,8 @@ static int read_registry_init_data(void)
ok = ok && get_config_parameter(src, EXTENSION_URI_TAG, extension_uri, sizeof(extension_uri));
ok = ok && get_config_parameter(src, JK_WORKER_FILE_TAG, worker_file, sizeof(worker_file));
ok = ok && get_config_parameter(src, JK_MOUNT_FILE_TAG, worker_mount_file, sizeof(worker_mount_file));
+ if (!ok)
+ goto cleanup;
get_config_parameter(src, URI_REWRITE_TAG, rewrite_rule_file, sizeof(rewrite_rule_file));
if (get_config_parameter(src, URI_SELECT_TAG, tmpbuf, sizeof(tmpbuf))) {
int opt = parse_uri_select(tmpbuf);
@@ -2899,6 +2901,7 @@ static int read_registry_init_data(void)
}
else {
ok = JK_FALSE;
+ goto cleanup;
}
}
shm_config_size = (size_t) get_config_int(src, SHM_SIZE_TAG, 0);
@@ -2913,7 +2916,7 @@ static int read_registry_init_data(void)
if (get_config_parameter(src, ERROR_PAGE_TAG, error_page_buf, sizeof(error_page_buf))) {
error_page = error_page_buf;
}
-
+cleanup:
if (using_ini_file) {
jk_map_free(&map);
}
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org