Merge pull request #167 from dboltovskyi/fix/security_init

Move initialization of mobileSecurities to Security Manager constructor
smartdevicelink · May 10, 2019 · 109d1cb · 109d1cb
2 parents b99f18d + 5452dfc
commit 109d1cb
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/modules/mobile_session_impl.lua b/modules/mobile_session_impl.lua
@@ -281,6 +281,7 @@ function mt.__index:StopRPC()
   return ret
     :Do(function(_, _)
       self.security:unregisterAllSecureServices()
+      self.security:unregisterSessionSecurity()
     end)
 end
 

diff --git a/modules/security/security_manager.lua b/modules/security/security_manager.lua
@@ -133,6 +133,10 @@ function security_mt.__index:registerSessionSecurity()
   end
 end
 
+function security_mt.__index:unregisterSessionSecurity()
+  SecurityManager.mobileSecurities[self.session.sessionId.get()] = nil
+end
+
 --- Register service into mobile session security. Service assumed as secure
 -- @tparam number service Service number
 function security_mt.__index:registerSecureService(service)