OpenCascade/occt
1
0
Fork 0
mirror of https://git.dev.opencascade.org/repos/occt.git synced 2025-04-04 18:06:22 +03:00
Code Packages Releases Activity

0023209: Potentially unmatched XOpenDisplay() and XCloseDisplay() calls in some OCCT packages

Browse Source 
Fixed display connection management in Draw_Window
This commit is contained in:
dbv 2013-03-01 11:05:28 +03:00
parent da21902312
commit a6a9658686
1 changed files with 19 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
src/Draw/Draw_Window.cxx
View File

@ -181,7 +181,12 @@ Standard_Boolean Draw_BlackBackGround = Standard_True;
//======================================================
Draw_Window* Draw_Window::firstWindow = NULL;
// X11 specific part
#if !defined(__APPLE__) || defined(MACOSX_USE_GLX)
#include <Aspect_DisplayConnection.hxx>
static Handle(Aspect_DisplayConnection) Draw_DisplayConnection;
//=======================================================================
//function : Draw_Window
//purpose :
@ -1091,17 +1096,21 @@ Standard_Boolean Init_Appli()
Tk_GeometryRequest(mainWindow, 200, 200);
#if !defined(__APPLE__) || defined(MACOSX_USE_GLX)
if (Draw_WindowDisplay == NULL) {
Draw_WindowDisplay = XOpenDisplay(NULL);
// Replaced Tk_Display(mainWindow) with XOpenDisplay; On Mac OS X Tk_Display
// returns a pointer to Display structure defined in system Tcl/Tk libraries.
// This structure differs from structure defined in X11 library and
// this caused DRAWEXE crash on startup.
if (Draw_DisplayConnection.IsNull())
{
try
{
Draw_DisplayConnection = new Aspect_DisplayConnection();
}
catch (Standard_Failure)
{
std::cout << "Cannot open display. Interpret commands in batch mode." << std::endl;
return Standard_False;
}
}
if (Draw_WindowDisplay == NULL) {
cout << "Cannot open display : "<<XDisplayName(NULL)<<endl;
cout << "Interpret commands in batch mode."<<endl;
return Standard_False;
if (Draw_WindowDisplay == NULL)
{
Draw_WindowDisplay = Draw_DisplayConnection->GetDisplay();
}
//
// synchronize the display server : could be done within Tk_Init