projects
/
metaproxy-moved-to-github.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Fix ref to freed memory in log filter 1line - MP-556
[metaproxy-moved-to-github.git]
/
src
/
router_flexml.cpp
diff --git
a/src/router_flexml.cpp
b/src/router_flexml.cpp
index
ba1afa0
..
e1f723e
100644
(file)
--- a/
src/router_flexml.cpp
+++ b/
src/router_flexml.cpp
@@
-119,6
+119,8
@@
void mp::RouterFleXML::Rep::parse_xml_filters(xmlDocPtr doc,
id_value = value;
else if (name == "type")
type_value = value;
id_value = value;
else if (name == "type")
type_value = value;
+ else if (name == "base")
+ ;// Ignore XInclude base attribute.
else
throw mp::XMLError("Only attribute id or type allowed"
" in filter element. Got " + name);
else
throw mp::XMLError("Only attribute id or type allowed"
" in filter element. Got " + name);
@@
-130,15
+132,14
@@
void mp::RouterFleXML::Rep::parse_xml_filters(xmlDocPtr doc,
type_value.c_str(), m_dl_path.c_str());
m_factory->add_creator_dl(type_value, m_dl_path);
}
type_value.c_str(), m_dl_path.c_str());
m_factory->add_creator_dl(type_value, m_dl_path);
}
- mp::filter::Base* filter_base = m_factory->create(type_value);
+ boost::shared_ptr<mp::filter::Base> fb(m_factory->create(type_value));
- filter_base->configure(node, test_only, file_include_path);
+ fb->configure(node, test_only, file_include_path);
if (m_id_filter_map.find(id_value) != m_id_filter_map.end())
throw mp::XMLError("Filter " + id_value + " already defined");
if (m_id_filter_map.find(id_value) != m_id_filter_map.end())
throw mp::XMLError("Filter " + id_value + " already defined");
- m_id_filter_map[id_value] =
- boost::shared_ptr<mp::filter::Base>(filter_base);
+ m_id_filter_map[id_value] = fb;
node = mp::xml::jump_to_next(node, XML_ELEMENT_NODE);
}
node = mp::xml::jump_to_next(node, XML_ELEMENT_NODE);
}
@@
-176,6
+177,8
@@
void mp::RouterFleXML::Rep::parse_xml_filters1(xmlDocPtr doc,
refid_value = value;
else if (name == "type")
type_value = value;
refid_value = value;
else if (name == "type")
type_value = value;
+ else if (name == "base")
+ ;// Ignore XInclude base attribute.
else
throw mp::XMLError("Only attribute 'refid' or 'type'"
" allowed for element 'filter'."
else
throw mp::XMLError("Only attribute 'refid' or 'type'"
" allowed for element 'filter'."
@@
-238,6
+241,8
@@
void mp::RouterFleXML::Rep::parse_xml_routes(xmlDocPtr doc,
if (name == "id")
id_value = value;
if (name == "id")
id_value = value;
+ else if (name == "base")
+ ;// Ignore XInclude base attribute.
else
throw mp::XMLError("Only attribute 'id' allowed for"
" element 'route'."
else
throw mp::XMLError("Only attribute 'id' allowed for"
" element 'route'."
@@
-248,6
+253,9
@@
void mp::RouterFleXML::Rep::parse_xml_routes(xmlDocPtr doc,
// process <filter> / <filters> nodes in third level
const xmlNode* node3 = mp::xml::jump_to_children(node, XML_ELEMENT_NODE);
// process <filter> / <filters> nodes in third level
const xmlNode* node3 = mp::xml::jump_to_children(node, XML_ELEMENT_NODE);
+
+
+
parse_xml_filters1(doc, node3, test_only, file_include_path, route);
std::map<std::string,RouterFleXML::Route>::iterator it;
parse_xml_filters1(doc, node3, test_only, file_include_path, route);
std::map<std::string,RouterFleXML::Route>::iterator it;
@@
-344,6
+352,8
@@
void mp::RouterFleXML::Rep::parse_xml_config_dom(xmlDocPtr doc,
if (name == "route")
m_start_route = value;
if (name == "route")
m_start_route = value;
+ else if (name == "base")
+ ;// Ignore XInclude base attribute.
else
throw mp::XMLError("Only attribute route allowed"
" in element 'start'. Got " + name);
else
throw mp::XMLError("Only attribute route allowed"
" in element 'start'. Got " + name);