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

fix svlua exploit #2008

Merged
merged 3 commits into from
Feb 24, 2025
Merged

fix svlua exploit #2008

merged 3 commits into from
Feb 24, 2025

Conversation

meetric1
Copy link
Contributor

make sure to wrap your ents yall...

@spankyyy
Copy link
Contributor

no fun allowed

@StrawWagen
Copy link

the funny

@ax255
Copy link
Contributor

ax255 commented Feb 24, 2025

epic

@wrefgtzweve
Copy link
Contributor

monthly RCE at this point

@Vurv78
Copy link

Vurv78 commented Feb 24, 2025

Theres no way to actually access that afaik? Also it needs to be unwrapped to run :Fire on in the rest of the file

@NullLikesNothing
Copy link
Contributor

NullLikesNothing commented Feb 24, 2025

Theres no way to actually access that afaik.

image

@Vurv78
Copy link

Vurv78 commented Feb 24, 2025

lool nvm, for some reason he stores the table on the wrapped sf entity when I think it's intended to store it on the unwrapped entity (would likely be the actual fix)

shame i don't have perms anymore :D

@NullLikesNothing
Copy link
Contributor

storing it on the unwrapped ent would probably be better yeah

@meetric1
Copy link
Contributor Author

Theres no way to actually access that afaik? Also it needs to be unwrapped to run :Fire on in the rest of the file

oops. good catch

should be fixed

@yogwoggf
Copy link

epic fail

@Xandertron Xandertron mentioned this pull request Feb 24, 2025
@thegrb93 thegrb93 merged commit a724193 into thegrb93:master Feb 24, 2025
@thegrb93
Copy link
Owner

Went ahead and merged but those index names suck ass and will likely clash with other addons. Need to move them to instance data

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.

9 participants