From 7f903697ecf116edcfc7eca4315eaaed0626401c Mon Sep 17 00:00:00 2001 From: Denys Vlasenko Date: Thu, 19 Nov 2009 14:38:27 +0100 Subject: fix small bug in plugin loading code Signed-off-by: Denys Vlasenko --- lib/Utils/Plugin.h | 1 + src/Daemon/PluginManager.cpp | 7 ++++--- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/lib/Utils/Plugin.h b/lib/Utils/Plugin.h index 9447f45..2d8bba3 100644 --- a/lib/Utils/Plugin.h +++ b/lib/Utils/Plugin.h @@ -84,6 +84,7 @@ typedef enum { ACTION, /**< An action plugin*/ REPORTER, /**< A reporter plugin*/ DATABASE /**< A database plugin*/ + MAX_PLUGIN_TYPE = DATABASE } plugin_type_t; /** diff --git a/src/Daemon/PluginManager.cpp b/src/Daemon/PluginManager.cpp index f8e8fed..2f4d875 100644 --- a/src/Daemon/PluginManager.cpp +++ b/src/Daemon/PluginManager.cpp @@ -166,9 +166,10 @@ void CPluginManager::LoadPlugin(const char *pName) { std::string libPath = ssprintf(PLUGINS_LIB_DIR"/"PLUGINS_LIB_PREFIX"%s."PLUGINS_LIB_EXTENSION, pName); abrtPlugin = new CABRTPlugin(libPath.c_str()); - if (abrtPlugin->GetMagicNumber() != PLUGINS_MAGIC_NUMBER || - (abrtPlugin->GetType() < ANALYZER && abrtPlugin->GetType() > DATABASE)) - { + if (abrtPlugin->GetMagicNumber() != PLUGINS_MAGIC_NUMBER + || abrtPlugin->GetType() < 0 + || abrtPlugin->GetType() > MAX_PLUGIN_TYPE + ) { throw CABRTException(EXCEP_PLUGIN, "CPluginManager::LoadPlugin(): non-compatible plugin"); } log("Plugin %s (%s) succesfully loaded", pName, abrtPlugin->GetVersion()); -- cgit