Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | nxcomp/src/ChannelEndPoint.cpp: don't use C-style casting in C++ code... | Mihai Moldovan | 2017-12-24 | 1 | -3/+3 | |
| | ||||||
* | nxcomp/src/Loop.cpp: fix small memory leak. | Mihai Moldovan | 2017-12-24 | 1 | -0/+2 | |
| | ||||||
* | nxcomp/src/Loop.cpp: log message fixups only. | Mihai Moldovan | 2017-12-18 | 1 | -2/+2 | |
| | ||||||
* | nxcomp/src/Log.cpp: only create a new queue entry if we actually intend to ↵ | Mihai Moldovan | 2017-12-11 | 1 | -3/+3 | |
| | | | | | | write it out later. Fixes: ArcticaProject/nx-libs#596 | |||||
* | nxcomp: fix shadow warning | Ulrich Sibiller | 2017-12-11 | 1 | -2/+2 | |
| | | | | | | | | | | | | | | Fixes this: CXX Loop.lo Loop.cpp: In function 'void handleCheckResultInLoop(int&, int&, int&, fd_set&, fd_set&, timeval&, timeval&)': Loop.cpp:14080:68: warning: declaration of 'startTs' shadows a global declaration [-Wshadow] struct timeval &startTs) ^ Loop.cpp:1020:13: note: shadowed declaration is here T_timestamp startTs; Fixes ArcticaProject/nx-libs#545 | |||||
* | 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. | |||||
* | 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/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 | 1 | -1/+2 | |
| | | | | 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 | 1 | -1/+7 | |
| | ||||||
* | 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. | |||||
* | nxcomp/src/Loop.cpp: whitespace fixes only. | Mihai Moldovan | 2017-09-29 | 1 | -6/+6 | |
| | ||||||
* | 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. | |||||
* | 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 | |
| | ||||||
* | nxcomp/src/Loop.cpp: Don't reuse global variable names as function ↵ | Mike Gabriel | 2017-07-26 | 1 | -18/+18 | |
| | | | | parameters in SetupDisplaySocket(). | |||||
* | nxcomp/src/Loop.cpp: Rename local variable 'handler' to 'signalHandler'. | Mike Gabriel | 2017-07-26 | 2 | -5/+5 | |
| | ||||||
* | nxcomp/src/Loop.cpp: Drop duplicate prototypes (which already exist in Misc.h). | Mike Gabriel | 2017-07-26 | 1 | -7/+2 | |
| | ||||||
* | nxcomp/src/Proxy.cpp: Fix index usage of nested loops. | Mike Gabriel | 2017-07-26 | 1 | -5/+5 | |
| | ||||||
* | nxcomp/src/Loop.cpp: Avoid duplicate usage of proxyFD (global variable) and ↵ | Mike Gabriel | 2017-07-26 | 1 | -19/+19 | |
| | | | | *proxyFD (function parameter). |