Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | nxcomp/src/Log.{cpp,h}: provide infrastructure for appending to already ↵ | Mihai Moldovan | 2017-12-25 | 2 | -23/+89 |
| | | | | | | | | | existing log lines. Also handle errors due to missing buffers gracefully. This error is surprisingly easy to trigger, so we should make sure that the program does not crash. | ||||
* | nxcomp/src/Log.h: NXLogStamp constructor should actually take a size_t-typed ↵ | Mihai Moldovan | 2017-12-25 | 1 | -1/+1 |
| | | | | line parameter. | ||||
* | nxcomp/src/Log.h: use initializer lists instead of initializing member ↵ | Mihai Moldovan | 2017-12-25 | 1 | -17/+4 |
| | | | | variables in-block. | ||||
* | 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 |
| | |||||
* | Support building with legacy zlib versions that do not ship a pkg-config file. | Mihai Moldovan | 2017-12-16 | 1 | -1/+1 |
| | |||||
* | nx{comp{,shad},proxy}: use dist-bzip2 instead of dist-xz. | Mihai Moldovan | 2017-12-16 | 1 | -1/+1 |
| | | | | | Automake older than version 1.11 does not support dist-xz. No workaround possible via a feature check, as far as I can tell. | ||||
* | nxcomp/test/logging_test.cpp: return a value in non-void function. | Mihai Moldovan | 2017-12-15 | 1 | -0/+2 |
| | | | | Failed *SUSE checks. | ||||
* | nxcomp/Makefile.am: call aclocal with the correct include dir argument. | Mihai Moldovan | 2017-12-15 | 1 | -0/+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 | ||||
* | Silence warning: "ar: 'u' modifier ignored since 'D' is the default" | Ulrich Sibiller | 2017-12-11 | 1 | -0/+3 |
| | | | | Fixes ArcticaProject/nx-libs#530 | ||||
* | 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. | ||||
* | 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. |