Loop.cpp: free display before leaving SetupDisplaySocket()

fixes a memory leak

Loop.cpp: free display before leaving SetupDisplaySocket()
4107159e · Ulrich Sibiller · 604e0968 · 4107159e
Commit 4107159e authored Dec 27, 2017 by Ulrich Sibiller
Show whitespace changes
Inline Side-by-side

Showing with 5 additions and 0 deletions

Loop.cpp nxcomp/src/Loop.cpp +5 -0

No files found.
--- a/nxcomp/src/Loop.cpp
+++ b/nxcomp/src/Loop.cpp
@@ -3877,6 +3877,8 @@ void SetupDisplaySocket(int &addr_family, sockaddr *&addr,

    cerr << "Error" << ": Invalid display '" << display << "'.\n";

+    delete [] display;
+
    HandleCleanup();
  }

@@ -3949,6 +3951,7 @@ void SetupDisplaySocket(int &addr_family, sockaddr *&addr,

        close(testSocketFD);
        addr = (sockaddr *) xServerAddrABSTRACT;
+        delete [] display;
        return;

    } else {
@@ -4000,6 +4003,7 @@ void SetupDisplaySocket(int &addr_family, sockaddr *&addr,
      cerr << "Error" << ": Error " << EGET() << " '" << ESTR()
           << "' checking '" << unixSocketDir << "'.\n";

+      delete [] display;
      HandleCleanup();
    }

@@ -4048,6 +4052,7 @@ void SetupDisplaySocket(int &addr_family, sockaddr *&addr,
      cerr << "Error" << ": Unknown display host '" << display
           << "'.\n";

+      delete [] display;
      HandleCleanup();
    }