diff options
| author | Nikola Pajkovsky <npajkovs@redhat.com> | 2010-08-18 10:38:23 +0200 |
|---|---|---|
| committer | Nikola Pajkovsky <npajkovs@redhat.com> | 2010-08-18 10:38:23 +0200 |
| commit | 9c2587f867c6d903efdfd001e977b83030bb1a2c (patch) | |
| tree | 4c9ee70da38af0b93b483e84b9c71fdbd68ef7dc /lib/plugins | |
| parent | e0b0da307a78b038045e2cb86934d60befd74339 (diff) | |
| download | abrt-9c2587f867c6d903efdfd001e977b83030bb1a2c.tar.gz abrt-9c2587f867c6d903efdfd001e977b83030bb1a2c.tar.xz abrt-9c2587f867c6d903efdfd001e977b83030bb1a2c.zip | |
assignment segfaults if right hand is a NULL pointer
Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
Diffstat (limited to 'lib/plugins')
| -rw-r--r-- | lib/plugins/Bugzilla.cpp | 2 | ||||
| -rw-r--r-- | lib/plugins/RHTSupport.cpp | 6 |
2 files changed, 4 insertions, 4 deletions
diff --git a/lib/plugins/Bugzilla.cpp b/lib/plugins/Bugzilla.cpp index bcdb7730..815abffa 100644 --- a/lib/plugins/Bugzilla.cpp +++ b/lib/plugins/Bugzilla.cpp @@ -979,7 +979,7 @@ void CReporterBugzilla::SetSettings(const map_plugin_settings_t& pSettings) /* Should not be deleted (why?) */ const map_plugin_settings_t& CReporterBugzilla::GetSettings() { - m_pSettings["BugzillaURL"] = m_bugzilla_url; + m_pSettings["BugzillaURL"] = (m_bugzilla_url)? m_bugzilla_url: ""; m_pSettings["Login"] = (m_login)? m_login: ""; m_pSettings["Password"] = (m_password)? m_password: ""; m_pSettings["SSLVerify"] = m_ssl_verify ? "yes" : "no"; diff --git a/lib/plugins/RHTSupport.cpp b/lib/plugins/RHTSupport.cpp index 313d426d..d13ff467 100644 --- a/lib/plugins/RHTSupport.cpp +++ b/lib/plugins/RHTSupport.cpp @@ -314,9 +314,9 @@ void CReporterRHticket::SetSettings(const map_plugin_settings_t& pSettings) /* Should not be deleted (why?) */ const map_plugin_settings_t& CReporterRHticket::GetSettings() { - m_pSettings["URL"] = m_strata_url; - m_pSettings["Login"] = m_login; - m_pSettings["Password"] = m_password; + m_pSettings["URL"] = (m_strata_url)? m_strata_url: ""; + m_pSettings["Login"] = (m_login)? m_login: ""; + m_pSettings["Password"] = (m_password)? m_password: ""; m_pSettings["SSLVerify"] = m_ssl_verify ? "yes" : "no"; return m_pSettings; |