Skip to content

Commit 1c253bc

Browse files
author
Kevin Gross
authored
Fix SMods escalation attack (#1627)
* Security Fix: SMods can perform escalation attack Fixes a vulnerability where a supermoderator can delete a admin account an re-register it, gaining admin access in a server * Addendum to previous commit * Remove SMod ability to chgpass Remove supermoderators ability to change an admin password
1 parent 5fd3811 commit 1c253bc

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

Server/mods/deathmatch/acl.xml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -177,10 +177,7 @@
177177
<right name="command.unloadmodule" access="true" />
178178
<right name="command.reloadmodule" access="true" />
179179
<right name="command.addaccount" access="true" />
180-
<right name="command.delaccount" access="true" />
181-
<right name="command.chgpass" access="true" />
182180
<right name="function.addAccount" access="true" />
183-
<right name="function.removeAccount" access="true" />
184181
<right name="function.setAccountName" access="true" />
185182
<right name="function.setAccountPassword" access="true" />
186183
</acl>
@@ -199,6 +196,8 @@
199196
<right name="command.authserial" access="true" />
200197
<right name="command.reloadacl" access="true" />
201198
<right name="command.stopall" access="true" />
199+
<right name="command.delaccount" access="true" />
200+
<right name="command.chgpass" access="true" />
202201
<right name="function.addBan" access="true" />
203202
<right name="function.setUnbanTime" access="true" />
204203
<right name="function.setBanAdmin" access="true" />
@@ -233,6 +232,7 @@
233232
<right name="function.updateResourceACLRequest" access="true" />
234233
<right name="function.shutdown" access="true" />
235234
<right name="function.setPlayerScriptDebugLevel" access="true" />
235+
<right name="function.removeAccount" access="true" />
236236
</acl>
237237
<acl name="RPC">
238238
<right name="function.callRemote" access="true" />

0 commit comments

Comments
 (0)