Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: client message. #695

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

jaredliu2018
Copy link

@jaredliu2018 jaredliu2018 commented Nov 28, 2024

Add client messages recv/send.
Server can hook any message(CLC_Messages) from client.
Also server can create a new client message and send it to client.
All complex logic is implemented in c++ side. Use it via simple API like "HookClientMessage()" and "message.Send()".

@KillStr3aK
Copy link
Contributor

Well this pr achieves way more stuff than described

can you regenerate the native methods with respecting the codestyle we use for them? (for e.x.: UnhookClientmessage -> UnhookClientMessage)

@@ -562,4 +582,204 @@ void* CModule::FindSymbol(const std::string& name)
CSSHARP_CORE_ERROR("Cannot find symbol {}", name);
return nullptr;
}

Sections* CModule::GetSection(const std::string_view name)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you can literally get everything you need in dl_iterate_phdr, or looping through PEB, copy-pasting these is making this code base even more bloated, same as SignatureIterator and the rest /shrug

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Implement FindVirutalTable in another way, I think that is necessary.

@jaredliu2018 jaredliu2018 requested a review from Nukoooo January 16, 2025 02:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants