protonscr

[HL1/CS] Admin slowhacking

steamclosed Feature RequestHigh priorityreviewedMOVED: Counter‐Strike
ValveSoftware/steam-for-linux#1536 · opened 2013-02-10 by AnAkkk · updated 2013-02-19 · 17 comments · github
AAnAkkk 2013-02-10 github

It's been known since a long time now that admins are able to send any commands to the clients, thus mess up their config.
It's quite annoying to join a server and find out that all of your binds have been changed, etc.
Servers can even open/close the players cd tray with the "cd" command.
I don't want to download annoying sounds from servers, and I am setting cl_allowdownload to 0. Guess what? The server forces cl_allowdownload to 1 before it starts downloading the sounds.

The fix shouldn't be to make your config read only and restart the game every time this happens. Servers just shouldn't be able to execute any commands on the clients, as it's the case on Source engine. Only the few commands that are required for the game to work should be allowed to be executed.

Bbogdyutzu 2013-02-10 github

+1 for this

CConnorMcLeod 2013-02-10 github

+1 "developer" command is one of commands that should be blocked, because then, config.cfg can be written even in read only mode.

Ttxdv 2013-02-10 github

what about connect/redirects?

AAnAkkk 2013-02-10 github

The "connect" command is already blocked AFAIK, the client won't do anything if it's received from a server. If there are other redirect exploits they're probably done in an other way.

Ttxdv 2013-02-10 github

so this means death to xredict?

Aalfred-valve maintainer 2013-02-11 github

yes, a horrible horrible death to all those honey pot servers. #1189 is a bug to allow a legit method for redirect. Blocking random commands from a server is something that we will do.

XXalus 2013-02-11 github

I think a dialog for everything the server wanna change of client settings and also redirecting to another server and everything, would be the best result.

AAnAkkk 2013-02-11 github

A dialog for redirection would probably be good, but I don't think there should be a dialog for other client settings. Players that never used the console/commands wouldn't understand what's happening. There is no reason a server would need to change client settings.

LLaurentiuAndrei 2013-02-12 github

Think about plugins like amx_ss which use send the "snapshot" command to the player's console.
But! I suggest to block only the one with "cl_" in the beginning.

AAnAkkk 2013-02-12 github

Valve isn't allowing admins to take screenshots on the Source engine and don't want to, I don't know see why they would on CS 1.6. An admin could loop the snapshot command to annoy a player and fill his disk of screenshots.

EDIT: Anyway this is pointless, hacks just disable themselves when the snapshot command is called.

LLaurentiuAndrei 2013-02-12 github

Well then it would be just awesome to block this communication between server and client.

EDIT : This will actually solve my problem which consists in the fact that the server is FLOODING me with the fps_101 command at every 1.0 seconds. I just hate it.

LLevShisterov 2013-02-13 github

There is at least one widely used client command that is sent from server: spk.

AAnAkkk 2013-02-19 github

@alfred-valve, am I supposed to reopen this in the halflife tracker, or do you still look at issues in this one?

Ttxdv 2013-02-19 github

general statement of whether we are supposed to reopen or not would be nice.

LLevShisterov 2013-02-19 github

There is also "echo" command which is widely used for printing info to client.

Aalfred-valve maintainer 2013-02-19 github

I am actively watching the halflife tracker, not this one, so re-opening a bug in that tracker and pointing to this bug will increase the chances of your bug being looked at :)

Nothing extracted yet.