Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | nxcomp: use new macro SAFE_FREE(ptr) | Ulrich Sibiller | 2017-12-09 | 4 | -39/+33 |
| | | | | | Should be used instead of free() calls and will clear the pointer after calling free(). This can prevent double-free or use-after-free errors. | ||||
* | Loop.cpp: fix more memory leaks | Ulrich Sibiller | 2017-12-09 | 1 | -1/+15 |
| | | | | | | | The thread specific stringstream objects on the stack need to be deleted, not just pop()ed. Fixes ArcticaProject/nx-libs#573 (partially) | ||||
* | Loop.cpp: fix two memleaks | Ulrich Sibiller | 2017-12-09 | 1 | -4/+19 |
| | |||||
* | ChannelEndPoint.cpp: fix two memleaks | Ulrich Sibiller | 2017-12-09 | 1 | -4/+10 |
| | |||||
* | nxcomp: simplify free calls | Ulrich Sibiller | 2017-12-09 | 1 | -3/+3 |
| | | | | free() can handle NULL itself | ||||
* | nxcomp: fix double free | Ulrich Sibiller | 2017-12-09 | 1 | -0/+2 |
| | | | | Fixes ArcticaProject/nx-libs#569 | ||||
* | nxproxy/nxcomp: document logging options | Ulrich Sibiller | 2017-12-07 | 1 | -0/+14 |
| | |||||
* | nxcomp/nxproxy: slightly improve the usage message of nxproxy | Ulrich Sibiller | 2017-12-07 | 1 | -6/+8 |
| | |||||
* | nxcomp/nxproxy: add missing hint for keyconv option | Ulrich Sibiller | 2017-12-07 | 1 | -0/+1 |
| | |||||
* | nxcomp: use lowercase abbreviations for seconds and milliseconds | Ulrich Sibiller | 2017-12-02 | 9 | -71/+71 |
| | |||||
* | nxcomp: remove unused return values | Ulrich Sibiller | 2017-12-02 | 1 | -19/+12 |
| | | | | Some functions' return values are never used anywhere, so drop them. | ||||
* | nxcomp: Fix setting of global tcpFD | Ulrich Sibiller | 2017-12-02 | 1 | -1/+2 |
| | | | | | | | This corrects a flaw that stayed undetected for 2.5 years, introduced via c5a2cf82f6baf7a9e64ec2328a5bac0dfee453ef Fixes ArcticaProject/nx-libs#573 | ||||
* | nxcomp/src/Misc.cpp: Update explanation for the NX_TEMP env variable. | Mike Gabriel | 2017-11-22 | 1 | -2/+1 |
| | |||||
* | nxcomp/src/Loop.cpp: hardcode X11 and (legacy/xfs) font UNIX paths. | Mihai Moldovan | 2017-11-04 | 1 | -15/+7 |
| | | | | | | | | Fixes: ArcticaProject/nx-libs#536 Allowing overrides via NX_TEMP and TEMP breaks system using pam_tmpdir.so and most server infrastructure expects the location to be fixed anyway. | ||||
* | nxcomp/src/Auth.cpp: handle launchd sockets in DISPLAY variable correctly ↵ | Mihai Moldovan | 2017-11-03 | 1 | -3/+23 |
| | | | | when fetching X cookie. | ||||
* | nxcomp/m4/ax_pthread.m4: move to top-level m4/ directory, use symlink. | Mihai Moldovan | 2017-10-27 | 1 | -485/+1 |
| | |||||
* | Regression fix for f855b3b6. Caused FTBFS with Loop.cpp:2669:0: error: ↵ | Mike Gabriel | 2017-10-27 | 1 | -1/+0 |
| | | | | unterminated #ifdef (#ifdef TIME). | ||||
* | nxcomp/src/Loop.cpp: fix whitespace errors on continued lines. | Mihai Moldovan | 2017-10-27 | 1 | -810/+810 |
| | |||||
* | Remove leading \n in NXTrans* logging | Vadim Troshchinskiy | 2017-10-27 | 1 | -3/+3 |
| | |||||
* | Fix whitespace and line splitting issues reported by ionic | Vadim Troshchinskiy | 2017-10-27 | 1 | -10/+19 |
| | |||||
* | Conversion of Loop.cpp to the new log system | Vadim Troshchinskiy | 2017-10-27 | 1 | -2560/+1285 |
| | |||||
* | nxcomp: add basic logging_test utility in test subdirectory. | Mihai Moldovan | 2017-10-27 | 6 | -1/+373 |
| | | | | Needs more sophisticated features to actually test stuff correctly. | ||||
* | nxcomp/src/Log.h: prepare for logger testing application. | Mihai Moldovan | 2017-10-27 | 1 | -0/+4 |
| | | | | DO NOT USE! | ||||
* | nxcomp/src/Log.cpp: add PID to thread ID output if requested. | Mihai Moldovan | 2017-10-27 | 1 | -1/+2 |
| | | | | | | | | The default function we use (if a thread name is not specified explicitly) is pthread_self(). This function returns a number that is guaranteed to be unique for each thread within a process, but this assertion doesn't hold globally. Hence only using the thread ID is ambiguous when logging from multiple processes. | ||||
* | nxcomp/src/Log.h: block signals while writing out data. | Mihai Moldovan | 2017-10-27 | 1 | -0/+20 |
| | | | | | Prevents race conditions caused by signal handlers while flushing out our log queue. | ||||
* | nxcomp/src/Log.{cpp,h}: port to std::stack as internal buffer structure. | Mihai Moldovan | 2017-10-27 | 2 | -23/+37 |
| | | | | | | | | This has one drawback: after flushing log data to its underlying output, a new NXLogStamp object MUST be written to the NXLog object in order to create a new entry within the stack. This can be changed if necessary. For now I'd like to keep it as-is. | ||||
* | nxcomp/{configure.ac,Log.cpp}: implement configure-time std::put_time check ↵ | Mihai Moldovan | 2017-10-27 | 2 | -1/+19 |
| | | | | and use macro value in Log.cpp. | ||||
* | nxcomp/Log.h: delete stream_ member before class object destruction. | Mihai Moldovan | 2017-10-27 | 1 | -0/+4 |
| | |||||
* | nxcomp/Log.h: also clear the buffer after setting it to an empty string. | Mihai Moldovan | 2017-10-27 | 1 | -1/+2 |
| | |||||
* | Set default log level to WARNING | Vadim Troshchinskiy | 2017-10-27 | 1 | -1/+1 |
| | | | | | This is to ensure the log output is the same as previous releases, which didn't have INFO defined by default. | ||||
* | New logging implementation | Vadim Troshchinskiy | 2017-10-27 | 4 | -0/+719 |
| | | | | | | | | Features: * Works without ifdefs * Configurable with commandline arguments * Log level configurable per file * Thread safe | ||||
* | nxcomp: add AX_PTHREAD code from autoconf-archive and check for pthread support. | Mihai Moldovan | 2017-10-27 | 3 | -1/+494 |
| | |||||
* | nx{comp{,shad},proxy}: add configure flag --enable-cxx11 to enable and setup ↵ | Mihai Moldovan | 2017-10-27 | 3 | -0/+8 |
| | | | | | | | | | | optional C++11 support. Use AX_CXX_COMPILE_STDCXX(_11) from autotools archive. Note: should only be used in tandem. Make sure that each component either uses the C++11 ABI or none uses it. Mixing and matching libraries and binaries with different C++ ABIs might lead to weird errors. | ||||
* | nxcomp/configure.ac: rename --{en,dis}able-info to --{en,dis}able-debug, as ↵ | Mihai Moldovan | 2017-10-27 | 1 | -1/+1 |
| | | | | originally planned. | ||||
* | Fix bug #525: PANIC! No port specified for multimedia connections | Vadim Troshchinskiy | 2017-10-10 | 3 | -11/+19 |
| | | | | | | | Commit 5f1eb0dc3eb89e84a3b146975e147b995fb7673f introduced a regression that made the use of the multimedia port mandatory. Fixes ArcticaProject/nx-libs#525 | ||||
* | nxcomp/src/Loop.cpp: Reduce control->TokenSize slightly again in order to ↵ | Simon Matter | 2017-10-09 | 1 | -2/+2 |
| | | | | fix fragmentation of packages when on VPN (or otherwise encapsulated) connections. | ||||
* | .gitignore: Improve all .gitignore files, adapt to autotools build-flow and ↵ | Mike Gabriel | 2017-10-09 | 1 | -2/+0 |
| | | | | its build cruft. | ||||
* | nxcomp/src/Loop.cpp: whitespace fixes only. | Mihai Moldovan | 2017-09-29 | 1 | -6/+6 |
| | |||||
* | nxcomp/configure.ac: prettify AC_ARG_ENABLE() calls, use more quotes and ↵ | Mihai Moldovan | 2017-09-29 | 1 | -13/+13 |
| | | | | AS_IF() where appropriate. | ||||
* | Fix nxagent argument parsing logic | Vadim Troshchinskiy | 2017-09-29 | 2 | -57/+51 |
| | | | | | | | | | This makes it so that slave=1 works again as it should. The changes introduced earlier resulted in the unintentional requirement to specify the actual port number, breaking backwards compatibility. Fixes: ArcticaProject/nx-libs#518 | ||||
* | nxcomp/src/{Loop,Proxy}.cpp: On Debian/kFreeBSD (and other *BSD variants) ↵ | Mike Gabriel | 2017-09-06 | 2 | -0/+16 |
| | | | | | | the sockaddr_un.sun_path property is 104 chars long, not 108. Hard-coding Unix domain sockets in nxcomp the string length 104. Fixes ArcticaProject/nx-libs#507. | ||||
* | nx*/configure.ac: Use newly introduced macro NX_DEFAULT_OPTIONS instead of ↵ | Mike Gabriel | 2017-08-11 | 1 | -1/+1 |
| | | | | NX_COMPILER_FLAGS. | ||||
* | nxcomp/configure.ac: fix copy-and-paste typo (nxcompshad.pc.) | Mihai Moldovan | 2017-08-09 | 1 | -1/+1 |
| | |||||
* | nxcomp/configure.ac: we don't need to specify empty optional parameters. | Mihai Moldovan | 2017-08-09 | 1 | -2/+2 |
| | |||||
* | nxcomp/configure.ac: Don't rely on pkg-config alone when it comes to testing ↵ | Mike Gabriel | 2017-08-08 | 1 | -4/+4 |
| | | | | | | | | | | for presence of libjpeg shared lib and header files. Especially systems still using the IJG libjpeg implementation are likely to lack the libjpeg.pc file. . So this adds some alternative way of detecting if all libjpeg related build-dependencies are in place. | ||||
* | nxcomp: Add __attribute__((noreturn)) to more functions where appropriate. | Mike Gabriel | 2017-07-26 | 3 | -4/+4 |
| | |||||
* | nxcomp/src/Loop.cpp: Don't mix using global proxyFD and a local p(roxy)FD ↵ | Mike Gabriel | 2017-07-26 | 1 | -16/+16 |
| | | | | variable (in WaitForRemote() and ConnectToRemote()). Rename local variable name. | ||||
* | nxcomp/src/Loop.cpp: Don't use global variables as function paramters in ↵ | Mike Gabriel | 2017-07-26 | 1 | -4/+4 |
| | | | | handleLogReopenInLoop(). | ||||
* | nxcomp/src/Loop.cpp: Drop global variable diffTs, declare it in every ↵ | Mike Gabriel | 2017-07-26 | 1 | -5/+2 |
| | | | | funtion where it is needed. | ||||
* | nxcomp/src/Transport.cpp: Don't redeclare local variable if it can be reused. | Mike Gabriel | 2017-07-26 | 1 | -1/+1 |
| |