Fix static analysis issue
authorJunghyun Yeon <jungh.yeon@samsung.com>
Thu, 25 Feb 2021 06:28:59 +0000 (15:28 +0900)
committerJunghyun Yeon <jungh.yeon@samsung.com>
Thu, 25 Feb 2021 06:28:59 +0000 (15:28 +0900)
Signed-off-by: Junghyun Yeon <jungh.yeon@samsung.com>
src/manager/pkginfo_manager.cc

index ffc696f..0231f02 100644 (file)
@@ -160,9 +160,10 @@ extern "C" EXPORT_API int _appinfo_get_datacontrol_info(
 
        pkgmgr_client::PkgInfoClient client(parcelable, uid,
                        pkgmgr_common::ReqType::QUERY);
-       if (!client.SendRequest())
+       if (!client.SendRequest()) {
+               sqlite3_free(query);
                return PMINFO_R_ERROR;
-       // TODO: deliver rawdata to reqhandler directly if server is not working
+       }
 
        std::shared_ptr<pkgmgr_common::parcel::ResultParcelable> return_parcel(
                        std::static_pointer_cast<pkgmgr_common::parcel::ResultParcelable>(
@@ -206,8 +207,10 @@ extern "C" EXPORT_API int _appinfo_get_datacontrol_appid(
 
        pkgmgr_client::PkgInfoClient client(parcelable, uid,
                        pkgmgr_common::ReqType::QUERY);
-       if (!client.SendRequest())
+       if (!client.SendRequest()) {
+               sqlite3_free(query);
                return PMINFO_R_ERROR;
+       }
        // TODO: deliver rawdata to reqhandler directly if server is not working
 
        std::shared_ptr<pkgmgr_common::parcel::ResultParcelable> return_parcel(
@@ -253,8 +256,10 @@ extern "C" EXPORT_API int _appinfo_get_datacontrol_trusted_info(
 
        pkgmgr_client::PkgInfoClient client(parcelable, uid,
                        pkgmgr_common::ReqType::QUERY);
-       if (!client.SendRequest())
+       if (!client.SendRequest()) {
+               sqlite3_free(query);
                return PMINFO_R_ERROR;
+       }
        // TODO: deliver rawdata to reqhandler directly if server is not working
 
        std::shared_ptr<pkgmgr_common::parcel::ResultParcelable> return_parcel(
@@ -301,8 +306,10 @@ extern "C" EXPORT_API int _appinfo_get_datacontrol_privileges(
 
        pkgmgr_client::PkgInfoClient client(parcelable, uid,
                        pkgmgr_common::ReqType::QUERY);
-       if (!client.SendRequest())
+       if (!client.SendRequest()) {
+               sqlite3_free(query);
                return PMINFO_R_ERROR;
+       }
        // TODO: deliver rawdata to reqhandler directly if server is not working
 
        std::shared_ptr<pkgmgr_common::parcel::ResultParcelable> return_parcel(
@@ -348,8 +355,10 @@ extern "C" EXPORT_API int _appinfo_get_appcontrol_privileges(
 
        pkgmgr_client::PkgInfoClient client(parcelable, uid,
                        pkgmgr_common::ReqType::QUERY);
-       if (!client.SendRequest())
+       if (!client.SendRequest()) {
+               sqlite3_free(query);
                return PMINFO_R_ERROR;
+       }
        // TODO: deliver rawdata to reqhandler directly if server is not working
 
        std::shared_ptr<pkgmgr_common::parcel::ResultParcelable> return_parcel(