Free requests after processing them in cmd line tool

App installation and user management requests were not freed after their
processing is finished in security-manager-cmd. Pointers wrapped in
std::unique_ptr.

Change-Id: I689833dea78ccedb5aaac9267d3c0a06895f0568

diff --git a/src/cmd/security-manager-cmd.cpp b/src/cmd/security-manager-cmd.cpp
index 0ed8bb0..c2c12c5 100644 (file)
--- a/src/cmd/security-manager-cmd.cpp
+++ b/src/cmd/security-manager-cmd.cpp
@@ -33,6 +33,7 @@
 #include <dpl/singleton.h>
 #include <protocols.h>
 #include <security-manager.h>
+#include <utils.h>
 
 #include <boost/program_options.hpp>
 namespace po = boost::program_options;
@@ -357,6 +358,7 @@ int main(int argc, char *argv[])
             LogDebug("Install command.");
             if (security_manager_app_inst_req_new(&req) != SECURITY_MANAGER_SUCCESS)
                 return EXIT_FAILURE;
+            auto req_ptr = makeUnique(req, security_manager_app_inst_req_free);
             parseInstallOptions(argc, argv, *req, vm);
             return installApp(*req);
         } else if (vm.count("manage-users")) {
@@ -365,6 +367,8 @@ int main(int argc, char *argv[])
             LogDebug("Manage users command.");
             if (security_manager_user_req_new(&req) != SECURITY_MANAGER_SUCCESS)
                 return EXIT_FAILURE;
+            auto req_ptr = makeUnique(req, security_manager_user_req_free);
+
             parseUserOptions(argc, argv, *req, vm);
             return manageUserOperation(*req, operation);
         } else {