Lucene search

K
securityvulnsSecurityvulnsSECURITYVULNS:DOC:18788
HistoryJan 03, 2008 - 12:00 a.m.

Buffer-overflow and format string in White_Dune 0.29beta791

2008-01-0300:00:00
vulners.com
9

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

                         Luigi Auriemma

Application: White_Dune
http://vrml.cip.ica.uni-stuttgart.de/dune/
Versions: <= 0.29beta791
Platforms: Unix/Linux/MacOSX and Windows
Bugs: A] buffer-overflow in Scene::errorf
B] format string in ImportFile
Exploitation: local
Date: 02 Jan 2008
Author: Luigi Auriemma
e-mail: [email protected]
web: aluigi.org

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

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

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

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

White_Dune is an open source editor/viewer for the VRML97 files.

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

=======
2) Bugs


A] buffer-overflow in Scene::errorf

A buffer-overflow vulnerability is located in the function which builds
the error messages for the problems happened during the parsing of the
WRL file.

From Scene.cpp:

void
Scene::errorf(const char *fmt, …)
{
va_list ap;
char buf[1024], buf2[1024];
const char *url = "";

va_start&#40;ap, fmt&#41;;
vsprintf&#40;buf, fmt, ap&#41;;
if &#40;TheApp-&gt;getImportURL&#40;&#41; != NULL&#41;
    url = TheApp-&gt;getImportURL&#40;&#41;;
mysnprintf&#40;buf2, 1024, &quot;&#37;s &#37;d: &#37;s&quot;, url, lineno, buf&#41;;
_compileErrors += buf2;

}


B] format string in ImportFile

Another problem related to the handling of the errors.
After the building of the error message the parse() function returns
immediately and swDebugf() is called for visualizing it to stderr or to
the debugger without using the needed format argument required by the
function.

From DuneApp.cpp:

DuneApp::ImportFile(const char openpath, Scene scene, bool protoLibrary,
Node *node, int field)

if (errors[0]) {
swMessageBox(_mainWnd, errors, "Parse Errors", SW_MB_OK, SW_MB_WARNING);
swDebugf(errors);

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

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

http://aluigi.org/poc/whitedunboffs.zip

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

======
4) Fix

Version 0.29beta795

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


Luigi Auriemma
http://aluigi.org