summaryrefslogtreecommitdiff
path: root/src/core/hle/service/service.cpp
diff options
context:
space:
mode:
authorGravatar Yuri Kunde Schlesner2015-01-07 16:55:44 -0200
committerGravatar Yuri Kunde Schlesner2015-01-07 17:32:31 -0200
commita6ad68190fb02af9c89dc461f451bb75c8a10a94 (patch)
treea1fd04b7356286f49d1624d81be53af5b9609145 /src/core/hle/service/service.cpp
parentMerge pull request #438 from lioncash/swp (diff)
downloadyuzu-a6ad68190fb02af9c89dc461f451bb75c8a10a94.tar.gz
yuzu-a6ad68190fb02af9c89dc461f451bb75c8a10a94.tar.xz
yuzu-a6ad68190fb02af9c89dc461f451bb75c8a10a94.zip
Fix double-free in Service manager during shutdown
Fixes #423.
Diffstat (limited to 'src/core/hle/service/service.cpp')
-rw-r--r--src/core/hle/service/service.cpp14
1 files changed, 0 insertions, 14 deletions
diff --git a/src/core/hle/service/service.cpp b/src/core/hle/service/service.cpp
index c5233e687..0c5597283 100644
--- a/src/core/hle/service/service.cpp
+++ b/src/core/hle/service/service.cpp
@@ -46,36 +46,22 @@ Manager* g_manager = nullptr; ///< Service manager
46//////////////////////////////////////////////////////////////////////////////////////////////////// 46////////////////////////////////////////////////////////////////////////////////////////////////////
47// Service Manager class 47// Service Manager class
48 48
49Manager::Manager() {
50}
51
52Manager::~Manager() {
53 for(Interface* service : m_services) {
54 DeleteService(service->GetPortName());
55 }
56}
57
58/// Add a service to the manager (does not create it though)
59void Manager::AddService(Interface* service) { 49void Manager::AddService(Interface* service) {
60 // TOOD(yuriks): Fix error reporting 50 // TOOD(yuriks): Fix error reporting
61 m_port_map[service->GetPortName()] = Kernel::g_handle_table.Create(service).ValueOr(INVALID_HANDLE); 51 m_port_map[service->GetPortName()] = Kernel::g_handle_table.Create(service).ValueOr(INVALID_HANDLE);
62 m_services.push_back(service); 52 m_services.push_back(service);
63} 53}
64 54
65/// Removes a service from the manager, also frees memory
66void Manager::DeleteService(const std::string& port_name) { 55void Manager::DeleteService(const std::string& port_name) {
67 Interface* service = FetchFromPortName(port_name); 56 Interface* service = FetchFromPortName(port_name);
68 m_services.erase(std::remove(m_services.begin(), m_services.end(), service), m_services.end()); 57 m_services.erase(std::remove(m_services.begin(), m_services.end(), service), m_services.end());
69 m_port_map.erase(port_name); 58 m_port_map.erase(port_name);
70 delete service;
71} 59}
72 60
73/// Get a Service Interface from its Handle
74Interface* Manager::FetchFromHandle(Handle handle) { 61Interface* Manager::FetchFromHandle(Handle handle) {
75 return Kernel::g_handle_table.Get<Interface>(handle); 62 return Kernel::g_handle_table.Get<Interface>(handle);
76} 63}
77 64
78/// Get a Service Interface from its port
79Interface* Manager::FetchFromPortName(const std::string& port_name) { 65Interface* Manager::FetchFromPortName(const std::string& port_name) {
80 auto itr = m_port_map.find(port_name); 66 auto itr = m_port_map.find(port_name);
81 if (itr == m_port_map.end()) { 67 if (itr == m_port_map.end()) {