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

Update SDK Code #2110

Closed
SamVanheer opened this issue Mar 22, 2019 · 14 comments
Closed

Update SDK Code #2110

SamVanheer opened this issue Mar 22, 2019 · 14 comments
Assignees
Labels

Comments

@SamVanheer
Copy link

The latest update added a new engine function to the client:
pfnEngSrc_pfnFilteredClientCmd_t pfnFilteredClientCmd;

Please add it to the SDK along with other missing changes.

@metita
Copy link

metita commented Mar 22, 2019

Source? I would like to know where are you looking to find that info.

@SamVanheer
Copy link
Author

I reverse engineered the Counter-Strike client.dll to see how they fixed the Director Message exploit.

@metita
Copy link

metita commented Mar 22, 2019

Do you mind sharing the content from "pfnEngSrc_pfnFilteredClientCmd_t pfnFilteredClientCmd;"?

@SamVanheer
Copy link
Author

typedef int (*pfnEngSrc_pfnFilteredClientCmd_t)(char *);

It has been added to the end of the engine functions interface.

@tschumann
Copy link

Thanks - it goes right after pfnVguiWrap2_GetMouseDelta?
And what are the other missing changes?

@SamVanheer
Copy link
Author

Yeah it's at the very end so the interface isn't broken for existing games.

Other changes include missing flags to indicate if a cvar is a privileged or cl_filterstuffcmd privileged command, missing bug fixes like the weapon select server crash exploit, possibly others that haven't been merged in.

@mikela-valve
Copy link
Member

Hi @SamVanheer, I'm going to co-opt this issue into being a general tracker for getting the SDK updated to include all changes to it that I can find since the last commit in 2014. I may just break it up into several commits as I have time to audit files and include them in the update so I'll make notes here when I update files.

@mikela-valve mikela-valve changed the title Add new engine function to SDK Update SDK Code Apr 23, 2019
@mikela-valve mikela-valve self-assigned this Apr 23, 2019
@SamVanheer
Copy link
Author

Alright, see also:
#2101
#1839

@mikela-valve
Copy link
Member

Thanks, I’ll close out #2101 first and close #1839 in favour of this one.

Sent with GitHawk

@SamVanheer
Copy link
Author

SamVanheer commented Apr 24, 2019

If possible, can support for using VGUI2 be added to the SDK? Mods can use it but the VGUI2 controls library and required interfaces are missing so everything would need to be reverse engineered or adapted from Source.

See #1639 and #1739

A template for making a VGUI2 based HUD like Counter-Strike would be nice to have, since it's still a lot of work otherwise.

See also #1402

@SamVanheer
Copy link
Author

SamVanheer commented Feb 23, 2021

I've been updating a copy of the SDK to include all of the changes that have been made so far: https://github.com/Solokiller/halflife-updated

I've gone through the list of bugs listed in the milestones to make sure each fix has been applied. Additionally there are also fixes that have not yet been applied to the official games.

The fix for #1377 is temporary pending a full fix with engine-level support. The code that is currently in my version of the SDK exists in the engine but is not used for reasons explained in the issue.

Here's the changelog containing all changes made: https://github.com/Solokiller/halflife-updated/wiki/Changelog

@coderamen666
Copy link

Can you merge all the November 2023 changes to the sdk so that modders have corresponding source code to the 25th anniversary release?

@SamVanheer
Copy link
Author

@shawns-valve since all changes seem to have been merged this issue can probably be closed, unless there are plans to include VGUI2 or other functionality in the SDK as well?

@kisak-valve
Copy link
Member

Closing per the last comment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants