Mirror/zeek
1
0
Fork 0
mirror of https://github.com/zeek/zeek.git synced 2025-10-04 15:48:19 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

Fix a couple of life-time issues when plugin loading fails.

Browse source 
Reported by Coverity.

Follow-up to #1179.
This commit is contained in:
Robin Sommer 2020-12-03 07:55:05 +00:00
parent 946dfd5d16
commit e147692a43
1 changed files with 10 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

16
src/plugin/Manager.cc
View file

@ -142,6 +142,8 @@ void Manager::SearchDynamicPlugins(const std::string& dir)
bool Manager::ActivateDynamicPluginInternal(const std::string& name, bool ok_if_not_found, std::vector<std::string>* errors) bool Manager::ActivateDynamicPluginInternal(const std::string& name, bool ok_if_not_found, std::vector<std::string>* errors)
{ {
errors->clear(); // caller should pass it in empty, but just to be sure
dynamic_plugin_map::iterator m = dynamic_plugins.find(util::strtolower(name)); dynamic_plugin_map::iterator m = dynamic_plugins.find(util::strtolower(name));
if ( m == dynamic_plugins.end() ) if ( m == dynamic_plugins.end() )
@ -193,17 +195,19 @@ bool Manager::ActivateDynamicPluginInternal(const std::string& name, bool ok_if_
current_dir = dir.c_str(); current_dir = dir.c_str();
current_sopath = path; current_sopath = path;
void* hdl = dlopen(path, RTLD_NOW | RTLD_GLOBAL); void* hdl = dlopen(path, RTLD_NOW | RTLD_GLOBAL);
current_dir = nullptr;
current_sopath = nullptr;
if ( ! hdl ) if ( ! hdl )
{ {
const char* err = dlerror(); const char* err = dlerror();
errors->push_back(util::fmt("cannot load plugin library %s: %s", path, err ? err : "<unknown error>")); errors->push_back(util::fmt("cannot load plugin library %s: %s", path, err ? err : "<unknown error>"));
return false; continue;
} }
if ( ! current_plugin ) { if ( ! current_plugin ) {
errors->push_back(util::fmt("load plugin library %s did not instantiate a plugin", path)); errors->push_back(util::fmt("load plugin library %s did not instantiate a plugin", path));
return false; continue;
} }
current_plugin->SetDynamic(true); current_plugin->SetDynamic(true);
@ -223,17 +227,17 @@ bool Manager::ActivateDynamicPluginInternal(const std::string& name, bool ok_if_
if ( util::strtolower(current_plugin->Name()) != util::strtolower(name) ) { if ( util::strtolower(current_plugin->Name()) != util::strtolower(name) ) {
errors->push_back(util::fmt("inconsistent plugin name: %s vs %s", errors->push_back(util::fmt("inconsistent plugin name: %s vs %s",
current_plugin->Name().c_str(), name.c_str())); current_plugin->Name().c_str(), name.c_str()));
return false; continue;
} }
current_dir = nullptr;
current_sopath = nullptr;
current_plugin = nullptr; current_plugin = nullptr;
DBG_LOG(DBG_PLUGINS, " Loaded %s", path); DBG_LOG(DBG_PLUGINS, " Loaded %s", path);
} }
globfree(&gl); globfree(&gl);
if ( ! errors->empty() )
return false;
} }
else else