审查各种版本lua的调试问题

Author: CppCXY

emmy_debugger/src/emmy_debugger_manager.cpp

@@ -8,7 +8,8 @@ EmmyDebuggerManager::EmmyDebuggerManager()
 	  stateStepOver(std::make_shared<HookStateStepOver>()),
 	  stateStop(std::make_shared<HookStateStop>()),
 	  stateStepIn(std::make_shared<HookStateStepIn>()),
-	  stateStepOut(std::make_shared<HookStateStepOut>())
+	  stateStepOut(std::make_shared<HookStateStepOut>()),
+	  isRunning(false)
 {
 }
 
@@ -49,7 +50,18 @@ std::shared_ptr<Debugger> EmmyDebuggerManager::AddDebugger(lua_State* L)
 		}
 		else
 		{
-			debugger = std::make_shared<Debugger>(nullptr, shared_from_this());
+			// 如果首次add 的state不是main state，则main state视为空指针
+			// 但不影响luajit附加调试和远程调试
+			lua_State* mainState = nullptr;
+
+			int ret = lua_pushthread(L);
+			lua_pop(L, 1);
+			if (ret == 1)
+			{
+				mainState = L;
+			}
+
+			debugger = std::make_shared<Debugger>(mainState, shared_from_this());
 		}
 		debuggers.insert({identify, debugger});
 	}
@@ -216,20 +228,36 @@ void EmmyDebuggerManager::Eval(std::shared_ptr<EvalContext> ctx)
 
 void EmmyDebuggerManager::OnDisconnect()
 {
+	SetRunning(false);
 	std::lock_guard<std::mutex> lock(debuggerMtx);
 	for (auto it : debuggers)
 	{
 		it.second->Stop();
 	}
-	if (luaVersion == LuaVersion::LUA_JIT)
+}
+
+void EmmyDebuggerManager::SetRunning(bool value)
+{
+	std::lock_guard<std::mutex> lock(isRuningMtx);
+	isRunning = value;
+	if(isRunning)
 	{
-		debuggers.clear();
+		for(auto debugger: GetDebuggers())
+		{
+			debugger->Start();
+		}
 	}
 }
 
+bool EmmyDebuggerManager::IsRunning()
+{
+	std::lock_guard<std::mutex> lock(isRuningMtx);
+	return isRunning;
+}
+
 EmmyDebuggerManager::UniqueIdentifyType EmmyDebuggerManager::GetUniqueIdentify(lua_State* L)
 {
-	if (luaVersion != LuaVersion::LUA_JIT)
+	if (luaVersion == LuaVersion::LUA_JIT)
 	{
 		// 我们认为luajit只会有一个debugger
 		return 0;

emmy_debugger/src/emmy_facade.cpp

@@ -318,6 +318,8 @@ void EmmyFacade::OnInitReq(const rapidjson::Document& document)
 		emmyDebuggerManager->extNames = extNames;
 	}
 
+	emmyDebuggerManager->SetRunning(true);
+
 	//TODO 这里有个线程安全问题，消息线程和lua 执行线程不是相同线程，但是没有一个锁能让我做同步
 	// 所以我不能在这里访问lua state 指针的内部结构
 	// 
@@ -328,11 +330,7 @@ void EmmyFacade::OnInitReq(const rapidjson::Document& document)
 	auto debuggers = emmyDebuggerManager->GetDebuggers();
 	for (auto debugger : debuggers)
 	{
-		debugger->Start();
-		if (luaVersion != LuaVersion::LUA_JIT)
-		{
-			debugger->Attach(false);
-		}
+		debugger->Attach(false);
 	}
 }
 
@@ -611,10 +609,10 @@ void EmmyFacade::Hook(lua_State* L, lua_Debug* ar)
 		{
 			debugger = emmyDebuggerManager->AddDebugger(L);
 			install_emmy_core(L);
-			// attach的时候能进入这里必然是 connected，所以可以执行start
-			debugger->Start();
-			debugger->Attach();
-
+			if (emmyDebuggerManager->IsRunning()) {
+				debugger->Start();
+				debugger->Attach();
+			}
 			// send attached notify
 			rapidjson::Document rspDoc;
 			rspDoc.SetObject();

emmy_debugger/src/hook_state.cpp

@@ -179,9 +179,11 @@ bool HookStateStop::Start(std::shared_ptr<Debugger> debugger, lua_State* current
 	if (current == nullptr)
 		return false;
 
-	// 停止调试时,不在更新hook,
-	// 因为lua_state* 可能已经失效了
+	// 不能取消当前hook
+	// 当unity 停止play之后附加调试依然运行，此时lua虚拟机可能已经销毁
+	// 所以以下操作会导致进程崩溃
 	// debugger->UpdateHook(0, current);
+	
 	// 此处会引发递归加锁而报错，而如果使用递归锁对调试体验影响
 	// debugger->DoAction(DebugAction::Continue);
 	debugger->SetHookState(debugger->GetEmmyDebuggerManager()->stateContinue);

include/emmy_debugger/emmy_debugger_manager.h

@@ -66,6 +66,10 @@ class EmmyDebuggerManager : public std::enable_shared_from_this<EmmyDebuggerMana
 
 	void OnDisconnect();
 
+	void SetRunning(bool value);
+
+	bool IsRunning();
+
 	// public 成员放下面
 	std::shared_ptr<HookStateBreak> stateBreak;
 	std::shared_ptr<HookStateStepOver> stateStepOver;
@@ -93,4 +97,7 @@ class EmmyDebuggerManager : public std::enable_shared_from_this<EmmyDebuggerMana
 	std::vector<std::shared_ptr<BreakPoint>> breakpoints;
 
 	std::set<int> lineSet;
+
+	std::mutex isRuningMtx;
+	bool isRunning;
 };