summaryrefslogtreecommitdiff
path: root/src/shared
Commit message (Collapse)AuthorAgeFilesLines
* process user message: Fix bound handlingFrederic Berat2016-10-241-10/+10
| | | | | | | | | | | | | | | | | | | | | | | | On context and application registration, the data needs to be read in 2 parts. If the first part was at the end of the receiver buffer, it was wrongly removed whereas the complete data is not full received and therefore interpretable. This has to be fixed by not removing the data before everything is checked. Thus, while readjusting the buffer to the first found header, the offset was not properly updated. Signed-off-by: Frederic Berat <fberat@de.adit-jv.com> Process User Messages: Fix buffer handling In case application and context registration the buffer need to be read two times. But before removing the data, the data need to be readjusted to its origin. This was missing in a previous fix. Furthermore, in case of corrupted description field, applications and contexts will be registered anyway with the dummy description "Unknown". Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
* dlt-daemon: receiver reworkFrederic Berat2016-04-282-0/+40
| | | | | | | | | | The receiver structures have been removed from the dlt-daemon structure, they are now part of the connection. The overall usage of the receiver structrure has also been reviewed in the daemon. Signed-off-by: Frederic Berat <fberat@de.adit-jv.com> Change-Id: I7cf80d79ed73bd6d4f370bb3f278d26ccc9d8d7a
* Offline trace: Make search more preciseS. Hameed2016-04-281-1/+1
| | | | | | | To avoid collisions with Logstorage Offlinetrace code checks for Offlinetrace log file base name Signed-off-by: S. Hameed <shameed@jp.adit-jv.com> Change-Id: I9719dd116cc72dd61f41211caba805e865be9539
* Parse INI filesChristoph Lipka2015-11-242-0/+722
| | | | | | | | | Offline Logstorage, Multinode and potentially other DLT extensions retrieve their configuration from a configuration file in INI file format. To avoid code duplications, this helper functionality should be used to read configuration files. Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
* offline trace: supported trace file name configuration based on indexManikandan C2015-11-181-10/+164
| | | | | | | | | Changes made for trace file name configuration based on index and made it configurable from dlt.conf file. Using this configuration one can select either timestamp based or index based log file creation for offline trace. Signed-off-by: Manikandan C <Manikandan.Chockalingam@in.bosch.com>
* Purged all warnings for -Wall -WextraLutz Helwing2015-10-281-1/+6
| | | | Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* DLT daemon improvement - dlt_init()-checkLutz Helwing2015-10-076-1295/+1388
| | | | | | | | | | DLT daemon improvement - parameter value range check Adapt unit tests to check for enum return values Changed C version to gnu99 and C++ version to gnu++0c Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* Allow multiple instances of dlt-daemonStefan Vacek2015-10-072-6/+53
| | | | | | | | | | | | | | | | - Make dlt-daemon configurable to specify directory of fifos and port of dlt-daemon, this allows to run multiple instances of dlt-daemon at one node at the same time. This is useful in testing environment where simultanous tests should not interfere due to the same instance of dlt-daemon - dlt-daemon: add option -t <dir> to specify a directory where all fifos will be created - dlt-daemon: add option -p <port> to specify the port under which dlt-daemon can be connected - client-library: add environment variable DLT_PIPE_DIR to specify a non-default directory of fifos for logging applications - client-library: add environment variable DLT_DAEMON_TCP_PORT to specify the port under which dlt-daemon can be reached (especially when using dlt-receive) Signed-off-by: Stefan Vacek <stefan.vacek@intel.com> Signed-off-by: Lutz Helwing <lutz_helwing@mentor.com>
* Cleanup masterStefan Vacek2015-10-075-0/+0
| | | | | | | - remove compiler warnings - fix file permissions Signed-off-by: Stefan Vacek <stefan.vacek@intel.com>
* Removed all trailing whitespacesLutz Helwing2015-07-214-60/+60
| | | | Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* Rework of unit testsStefan Held2015-07-211-53/+7
| | | | | Signed-off-by: Stefan Held <stefan_held@mentor.com> Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* Fix compilation warnings and possible misuse of snprintStefan Vacek2015-07-211-12/+13
| | | | | | | | | - sprintf was used were snprint was intended - make proper usage of 64bit values in printf-formatting - remove some not used variables - make some casts to make printf happy Signed-off-by: Stefan Vacek <stefan.vacek@intel.com>
* Rework of unit tests in gtest_dlt_commonStefan Held2015-07-211-42/+104
| | | | | | | Fix all nullpointer errors in dlt_common.c Signed-off-by: Stefan Held <stefan_held@mentor.com> Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* Fixed license headers and copyright doxygen commentsLutz Helwing2015-06-177-14/+14
| | | | Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* Updated license headers to latest GENIVI license policy. Fixed further LRT ↵Lutz Helwing2015-06-177-78/+105
| | | | | | Scan findings. Renamed and cleanup further files. Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* dlt-daemon: Explicitly set the default loggingLevel to LOG_INFOSid Heroor2015-06-171-1/+1
| | | | Signed-off-by: Sid Heroor <heroor@heptaxel.com>
* Unified all line endings to UNIX style.Jens Bocklage2015-06-173-443/+443
| | | | Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* added dlt_log_* unit tests rework of dlt_buffer_* and dlt_messages_* unit testsStefan Held2015-06-171-1/+52
| | | | | Signed-off-by: Stefan Held <stefan_held@mentor.com> Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* Add Service ID Last entry to avoid further modifications in dependent codeS. Hameed2015-04-011-1/+1
| | | | | Signed-off-by: S. Hameed <shameed@jp.adit-jv.com> Signed-off-by: Christoph Lipka <clipka@de.adit-jv.com>
* Reworked internal outputSascha Philipp2015-03-252-154/+89
| | | | | | Change-Id: I371c1b234929fda46608f563f08cae141b97d0a9 Signed-off-by: Ralf Anton Beier <ralf.anton.beier@continental-corporation.com> Signed-off-by: Sascha Philipp <sascha.philipp@continental-corporation.com>
* adding support for new macros to the daemon. new macros: DLT_HEX8(VAR) 8bits ↵Jens Bocklage2015-03-121-1/+73
| | | | | | | | variable displayed in hexadecimal with "0x" prefix DLT_HEX16(VAR) 16bits displayed in hexadecimal with "0x" prefix DLT_HEX32(VAR) 32bits displayed in hexadecimal with "0x" prefix DLT_HEX64(VAR) 64bits displayed in hexadecimal with "0x" prefix DLT_BIN8(VAR) 8bits variable displayed in binary with "0b" prefix DLT_BIN16(VAR) 16bits variable displayed in binary with "0b" prefix plus typo fix Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* Added new API to send marker message from application.Alexander Wenzel2014-09-111-0/+1
| | | | Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* Fixed: all possible malloc, sprintf and strcpy problemsAlexander Wenzel2014-04-012-115/+128
| | | | Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* Updated authors information.Alexander Wenzel2014-04-011-46/+5
| | | | Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* Cygwin port: cygwin patch, signal handling patch and cppcheck and install ↵Alexander Wenzel2014-04-011-1/+1
| | | | | | lib dll to correct location on Windows. Originally from Mikko Rapeli <mikko.rapeli@bmw.de>. Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* Fixed compiler warnings with 32Bit gcc compiler.Alexander Wenzel2014-04-011-3/+3
| | | | Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* Defined return value for dlt_message_read().Alexander Wenzel2014-01-101-8/+8
| | | | Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* Cleanup of send return values. Further cleanup of send restructure.Alexander Wenzel2014-01-101-1/+1
| | | | Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* When using DLT in console mode on a 64-bit machine, timestamps are corrupted ↵Alexander Wenzel2014-01-101-1/+2
| | | | | | due to an address of a 32-bit value being cast to a 64-bit pointer. Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* Fix potential buffer overflow in offline trace.Alexander Wenzel2014-01-101-20/+39
| | | | Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* Environement variables added to configure internal logging in library.Alexander Wenzel2014-01-101-6/+43
| | | | Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* Added further checks to dlt_buffer.Alexander Wenzel2014-01-101-4/+18
| | | | Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* Security fix on DLT pipes.Alexander Wenzel2014-01-101-2/+2
| | | | Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* Changes after review.Alexander Wenzel2013-08-071-1/+1
| | | | Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* Bug 44 - Don't print "Buffer full" message from DLT daemon for each trace.Alexander Wenzel2013-08-072-1/+10
| | | | Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* Bug 84 - Adding utf8 support to dlt-daemon, dlt-viewer. Modified patch, ↵Alexander Wenzel2013-07-251-2/+2
| | | | | | originally provided by Stefan Vacek. Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* Invalid Sock is set to -1 instead 0 (stdin). Consequent error handling in ↵Simon Brandner2013-07-191-6/+17
| | | | | | fseek failure cases Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* Removed possible recursive call of dlt_user_log_file_errorMessage. Added ↵Simon Brandner2013-07-191-2/+7
| | | | | | | | | | some return value checks for system calls (stat,fseek) and error logging in those cases. Conflicts: src/lib/dlt_filetransfer.c Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* added DLT_SEM_LOCK around init of dlt user bufferSimon Brandner2013-07-191-0/+1
| | | | Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* some coverity fixesSimon Brandner2013-07-191-5/+27
| | | | | | fixed some merge problems Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* Semaphores and Pointer passing insteasd by value and otehr coverity issue fixesSimon Brandner2013-07-192-12/+26
| | | | Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* Fix compiler warningsLassi Marttala2013-07-191-6/+1
| | | | Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* fix overflow after ~35 minutes in dlt_uptime function. Should now ocurr ↵Simon Brandner2013-07-191-2/+2
| | | | | | after >100 hrs Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* appended null termination to version stringSimon Brandner2013-07-192-9/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | adding closure of socket in syslog_thread and read_socket fixed missing free of filepath in compressed file transfer fixed missing closedir in added in send_one error case fixed missing fclose of src_file in ferror case dlt-daemon.c fixed missing fclose in ferror case dlt_common.c using snprintf for filename instead sprintf in dlt_file_open modified some magix numbers - replaced by limits.h values. added strncpy instead strcpy to make cp more safe. split up a function to avoid coverity complaint when computing size of a write buffer length dlt_daemon_common: added strncpy instead strcpy to make cp more safe. Fixes some more Coverity issues in dlt-daemon.c dlt_user.c dlt_common.c and dlt_offline_trace.c. Conflicts: src/daemon/dlt-daemon.c src/lib/dlt_user.c Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* Removed most warnings caused by: Wunused-but-set-variable flag. Eome were ↵Simon Brandner2013-07-191-3/+6
| | | | | | kept, because they were the nicer variant to eliminate them, or they seemed reserved for an commandline argument Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* Fix: Parasoft DLT complaints about missing Null pointer checkSimon Brandner2013-07-191-1/+3
| | | | Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* [GDLT-120]: Segmented Network Trace: Prepare for rewrite.Lassi Marttala2012-11-262-314/+0
| | | | | | | | | | | | | This is a combination of 2 commits. Original descriptions below. [GDLT-120]: Remove dlt_queue. Only usage replace with POSIX queue. [GDLT-120]: Improve documentation. Add a note about context having to be pesistent thorough application lifetime. [GDLT-120]: Add macro interface. Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* [GDLT-120]: Improved Network trace. Rewrite to POSIX queue.Lassi Marttala2012-11-263-5/+320
| | | | | | | | | | | | | | | | | | | | | | | | | This is a squash of multiple commits. The original commit messages follow. [GDLT-120]: Segmented network trace: Correct error condition checks. [GDLT-120]: Add documentation for extended network trace [GDLT-120]: Add dlt_queue. Linked list based dynamic queue. [GDLT-120]: Recode to using queue for segmented messages [GDLT-120]: Improved thread start handling. Add error handling. Remove libm from linkage. [GDLT-120]: Avoid sending an extraneous empty segment, when size is aligned [GDLT-120]: More expansive return values from dlt_user_log_send_log [GDLT-120]: Change default behaviour of dlt_user_trace_network to truncated, instead of discard. [GDLT-120]: Convert to POSIX queue. Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
* [GENDLT-26]: Check for description length sanity.Lassi Marttala2012-06-131-1/+1
| | | | Signed-off-by: Christian Muck <christian.muck@bmw.de>
* [GDLT-90] Implemented systemd watchdog concept in dlt-systemChristian Muck2012-06-131-2/+2
| | | | Signed-off-by: Christian Muck <christian.muck@bmw.de>