diff options
| author | Jiri Moskovcak <jmoskovc@redhat.com> | 2009-10-05 21:42:16 +0200 |
|---|---|---|
| committer | Jiri Moskovcak <jmoskovc@redhat.com> | 2009-10-05 21:42:16 +0200 |
| commit | a47026057cd99a0e1eb34d51ed644cc66ae20bc6 (patch) | |
| tree | 5a2b250f50aa8239ac98377817410cdae6a267e5 /src/Daemon/PluginManager.cpp | |
| parent | a56b047e24bbf96879e77f51867b96d515cdbfa6 (diff) | |
| download | abrt-a47026057cd99a0e1eb34d51ed644cc66ae20bc6.tar.gz abrt-a47026057cd99a0e1eb34d51ed644cc66ae20bc6.tar.xz abrt-a47026057cd99a0e1eb34d51ed644cc66ae20bc6.zip | |
removed unsecure reading/writting from ~HOME directory
- daemon eunning under root can't read/write from ~HOME, so
reading/writting config files have been moved to client (GUI)
- uses gnome keyring to store the credentials
Diffstat (limited to 'src/Daemon/PluginManager.cpp')
| -rw-r--r-- | src/Daemon/PluginManager.cpp | 13 |
1 files changed, 12 insertions, 1 deletions
diff --git a/src/Daemon/PluginManager.cpp b/src/Daemon/PluginManager.cpp index 4c1dae5c..bd5727e0 100644 --- a/src/Daemon/PluginManager.cpp +++ b/src/Daemon/PluginManager.cpp @@ -420,12 +420,18 @@ void CPluginManager::SetPluginSettings(const std::string& pName, return; } + /** we don't want to save it from daemon if it's running under root + but wi might get back to this once we make the daemon to not run + with root privileges + */ + /* SavePluginSettings(confPath, pSettings); if (chown(confPath.c_str(), uid, gid) == -1) { perror_msg("Can't change '%s' ownership to %u:%u", confPath.c_str(), (int)uid, (int)gid); return; } + */ } } } @@ -443,7 +449,11 @@ map_plugin_settings_t CPluginManager::GetPluginSettings(const std::string& pName if (plugin != m_mapPlugins.end()) { ret = plugin->second->GetSettings(); - + /** we don't want to load it from daemon if it's running under root + but wi might get back to this once we make the daemon to not run + with root privileges + */ + /* if (abrt_plugin->second->GetType() == REPORTER) { std::string home = get_home_dir(atoi(pUID.c_str())); @@ -452,6 +462,7 @@ map_plugin_settings_t CPluginManager::GetPluginSettings(const std::string& pName LoadPluginSettings(home + "/.abrt/" + pName + "."PLUGINS_CONF_EXTENSION, ret); } } + */ return ret; } } |
