Commit b49c99eb authored by Marvin Scholz's avatar Marvin Scholz

Reduce unnecessary config locks in XSLT loader

parent 72b77502
Pipeline #250 failed with stage
in 11 seconds
...@@ -269,10 +269,12 @@ static xmlDocPtr custom_loader(const xmlChar *URI, ...@@ -269,10 +269,12 @@ static xmlDocPtr custom_loader(const xmlChar *URI,
/* In case a top stylesheet is loaded */ /* In case a top stylesheet is loaded */
case XSLT_LOAD_START: case XSLT_LOAD_START:
config = config_get_config(); /* Check if the admin URI is already cached */
if (admin_URI != NULL) {
break;
}
/* Check if we need to load the admin path */ config = config_get_config();
if (!admin_URI) {
/* Append path separator to path */ /* Append path separator to path */
size_t len = strlen(config->adminroot_dir); size_t len = strlen(config->adminroot_dir);
xmlChar* admin_path = xmlMalloc(len+2); xmlChar* admin_path = xmlMalloc(len+2);
...@@ -287,7 +289,6 @@ static xmlDocPtr custom_loader(const xmlChar *URI, ...@@ -287,7 +289,6 @@ static xmlDocPtr custom_loader(const xmlChar *URI,
} else { } else {
ICECAST_LOG_DEBUG("Loaded and cached admin_URI \"%s\"", admin_URI); ICECAST_LOG_DEBUG("Loaded and cached admin_URI \"%s\"", admin_URI);
} }
}
config_release_config(); config_release_config();
break; break;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment