fix msm-plugin.c svace issue: make sure dupPath is not NULL before strchr()

fix msmmanifest.c svace issue: make sure ac_domain->name not NULL before sprintf()
Change-Id: I2bf7a71e478569fec4d101e41c42776d9c7916a3
Signed-off-by: wangbiao <biao716.wang@samsung.com>

diff --git a/plugins/msm-plugin.c b/plugins/msm-plugin.c
index 4fe1c1a..9db346b 100644 (file)
--- a/plugins/msm-plugin.c
+++ b/plugins/msm-plugin.c
@@ -598,6 +598,7 @@ rpmRC msm_plugin_fsm_file_init(const char* path, mode_t mode)
     if (!path) return RPMRC_FAIL; 
 
     dupPath = strdup(path);
+    if (!dupPath) return RPMRC_FAIL;
     cleanedPath = strchr(dupPath, ';');
     if (cleanedPath)
         *cleanedPath = '\0';

diff --git a/plugins/msmmanifest.c b/plugins/msmmanifest.c
index 73f7c4b..76bfbd9 100644 (file)
--- a/plugins/msmmanifest.c
+++ b/plugins/msmmanifest.c
@@ -452,6 +452,7 @@ static int msmProcessProvide(xmlTextReaderPtr reader, provide_x *provide, sw_sou
                 if (mfx && !provide->name) {
                     ac_domain->name = malloc(strlen(mfx->name) + 2 +
                                              strlen(name) + 1);
+                   if (!ac_domain->name) return -1;
                     sprintf((char *)ac_domain->name, "%s::%s", mfx->name, name);
                     msmFreePointer((void**)&name);
                }