Multiple vulnerabilities in Cube engine 2005_08_29




#######################################################################

Luigi Auriemma

Application: Cube engine
http://www.cubeengine.com
Versions: <= 2005_08_29
Platforms: Windows, *nix, *BSD and MacOS
Bugs: A] sgetstr() buffer-overflow
B] invalid memory access
C] clients crash through invalid map
Exploitation: remote, versus both server and clients
Date: 06 Mar 2006
Author: Luigi Auriemma
e-mail: aluigi@xxxxxxxxxxxxx
web: http://aluigi.altervista.org


#######################################################################


1) Introduction
2) Bugs
3) The Code
4) Fix


#######################################################################

===============
1) Introduction
===============


Cube is an interesting open source game and engine developed by Wouter
van Oortmerssen (http://strlen.com).
It supports both LAN and Internet multiplayer through its master
server.


#######################################################################

=======
2) Bugs
=======

----------------------------
A] sgetstr() buffer-overflow
----------------------------

The game uses an unchecked function for reading the strings from the
incoming data.
The function is sgetstr() located in cube.h:

#define sgetstr() { char *t = text; do { *t = getint(p); } while(*t+
+); }

The problem, which affects both server and clients, is that this code
copies the input data over the text buffer of size MAXTRANS (5000 bytes)
allowing possible malicious code execution.


------------------------
B] invalid memory access
------------------------

sgetstr(), getint() and the instructions which call them don't check
the correct length of the input data.
In short is possible to force the server or the client to read over the
received data reaching unallocated zones of the memory and so crashing
immediately.
The biggest example in the Cube engine is the SV_EXT tag used in the
server where is read a 32 bits number from the input data and then is
performed a reading loop:

for(int n = getint(p); n; n--) getint(p);


------------------------------------
C] clients crash through invalid map
------------------------------------

In the Cube engine the players have the possibility to choose a
specific map on which playing, if there is only one player in the
server the map is changed immediately otherwise will be voted.
When a client tries to load an invalid map file it exits immediately
showing the "while reading map: header malformatted" error.
When the map is choosed all the clients add a .ogz extension to the
mapname received from the server and load the file.
The max size of the mapname is 260 bytes and the function which loads
the file uses a secure sprintf() which truncates the input mapname
(.ogz included) when the limit is reached.
Then the loading of the map is not sanitized versus possible directory
traversal exploitations so if an attacker (a player) specifies a
mapname of about 260 bytes he can force any client which will join the
server (due to the voting problem explained previously which limits the
exploitation of this bug) to load any file which is not a valid map and
so they will exit immediately.
As already said the exploitation happens with any new client which
joins the server since the new mapname will remain active in the server
for all the current match.


#######################################################################

===========
3) The Code
===========


http://aluigi.altervista.org/poc/evilcube.zip


#######################################################################

======
4) Fix
======


No fix.
The author says that the engine is no longer supported.


#######################################################################


---
Luigi Auriemma
http://aluigi.altervista.org



Relevant Pages

  • [Full-disclosure] Multiple vulnerabilities in Cube engine 2005_08_29
    ... C] clients crash through invalid map ... versus both server and clients ...
    (Full-Disclosure)
  • RE: 98 clients and network shares
    ... Since the XP clients in the same subnet can map drives and 98 clients in ... the other subnet can map drives we can assume that routing from the .6 ... network is working and that NetBIOS over TCP/IP is enabled on the server. ... Microsoft Enterprise Network Support ...
    (microsoft.public.win2000.networking)
  • Re: Mapping in a Profile
    ... My LAN consists of a Domain Server and its clients. ... There are shared folders on the server for each user, which the user maps on the client login where he/she logs in. ... How can I map these individual shares as per the concerned login? ...
    (microsoft.public.windows.server.networking)
  • Re: Clients not able to login to Domain
    ... This happens with both the server name and the IP address. ... Can you access Share Folders on SBS Server from problematic client ... Does this issue occur to all clients or several clients? ... > I understand that you cannot map drive to SBS Server. ...
    (microsoft.public.windows.server.sbs)
  • RE: Users Cant Access Documents on Server
    ... Thanks for using the SBS newsgroup. ... As well as we know, if a workstation would not access network shares, then ... Leave the Default Gateway of the internal NIC blank of the server box. ... Clients That Require SMB Signing ...
    (microsoft.public.windows.server.sbs)