mirror of
https://github.com/quasar/Quasar.git
synced 2026-04-25 23:35:58 +03:00
[GH-ISSUE #716] Protobuf isn't built into client.bin #459
Labels
No labels
bug
bug
cant-reproduce
discussion
duplicate
easy
enhancement
help wanted
improvement
invalid
need more info
pull-request
question
wont-add
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/Quasar#459
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @ooojustin on GitHub (Oct 16, 2018).
Original GitHub issue: https://github.com/quasar/Quasar/issues/716
Since the last release of the project, you've switched .NET serialization to protobuf.
The code to build assemblies in the Quasar.Client.csproj has not been adapted to this: https://github.com/quasar/QuasarRAT/blob/master/Quasar.Client/Quasar.Client.csproj#L150
The built stub in the server can only be used if it is stored in the same directory as protobuf-net.dll.
Theoretically, this could be fixed by changing the following:
"$(SolutionDir)packages\ILMerge.2.14.1208\tools\ILMerge.exe" /out:"$(TargetDir)$(TargetName).all.exe" "$(TargetPath)" "$(TargetDir)Gma.System.MouseKeyHook.dll" "$(TargetDir)Quasar.Common.dll" /target:WinExe /wildcards /internalizeto
"$(SolutionDir)packages\ILMerge.2.14.1208\tools\ILMerge.exe" /out:"$(TargetDir)$(TargetName).all.exe" "$(TargetPath)" "$(TargetDir)protobuf-net.dll" "$(TargetDir)Gma.System.MouseKeyHook.dll" "$(TargetDir)Quasar.Common.dll" /target:WinExe /wildcards /internalizeI rebuilt the project, and was able to confirm that protobuf was successfully merged into client.bin - but for some reason, after doing this, the connection would not work. So in one scenario, it can connect but crashes without the dll; in the other scenario it doesn't crash but it wont connect.
@MaxXor commented on GitHub (Oct 16, 2018):
Oh, thanks I forgot the DLL.
@MaxXor commented on GitHub (Oct 16, 2018):
I think the crashes could be fixed by changing
github.com/quasar/QuasarRAT@1bc20d38c0/Quasar.Server/Build/Renamer.cs (L60)to
Maybe you could report back and tell if it's working, I can only check this tomorrow.
@ooojustin commented on GitHub (Oct 16, 2018):
The crashing is fixed by merging the protobuf assembly, and checking if the namespace starts with Quasar in renamer seemed to fix the connection issue. All seems to be working normally now, good job.