summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Updated release notes for 2.17v2.17.0ManikandanC2017-09-152-2/+33
| | | | Signed-off-by: ManikandanC <Manikandan.Chockalingam@in.bosch.com>
* Fix for initialization of buffer settings in DLT user library. (#25)baduras2017-09-131-2/+2
|
* fix various memory leaks (#28)Gordan Markuš2017-09-123-14/+13
| | | | | | | | | | | | | | | | * dlt_common: Fix resource leak Close the file before returning from function. Signed-off-by: Gordan Markuš <gordan.markus@pelagicore.com> * dlt-kpi-process: Fix memory leak Signed-off-by: Gordan Markuš <gordan.markus@pelagicore.com> * dlt-kpi-options: Clean up resources on malloc fail Signed-off-by: Gordan Markuš <gordan.markus@pelagicore.com>
* Merge pull request #24 from baduras/some-minor-fixesRalphNiemeyer2017-09-053-5/+5
|\ | | | | Minor fixes for typo in CMakeLists and merge issue
| * Minor fixes: corrected typo in CMakeLists.txt - WTIH_DLT_ADAPTOR, removed ↵Stefan Badura2017-08-303-5/+5
|/ | | | character from merge - +
* Data stuck in receiver buffer when dlt_daemon_user_send_log_level() fails (#21)Lutz Helwing2017-08-223-12/+23
| | | | | | | | | | | | - When dlt_daemon_user_send_log_level() called in dlt_daemon_process_user_message_register_context() fails -1 was returned which caused dlt_daemon_process_user_messages() to stop processing receiver buffer. Remaining data was stuck until new data arrived over FIFO. - Make debug output of dlt_daemon_user_send_log_level() more verbose. Signed-off-by: Lutz Helwing <lutz_helwing@mentor.com>
* Update dlt_user.h (#22)Jens Lorenz2017-08-071-2/+2
| | | | | | | The dlt_user_log_write_start* function description mention the verbose/non-verbose option. According to my understanding verbose means that the code contains the verbose version of message (message as full string). The non-verbose version uses identifiers in order to reduce the message content and the interpretation of ids will be done by the viewer. Therefore the comments should be modified to capture this correctly.
* Add short explanation for DLT log level (#20)Lutz Helwing2017-08-072-0/+137
| | | Signed-off-by: Lutz Helwing <lutz_helwing@mentor.com>
* Prevent buffer overflow for mount point path in dlt_logstorage_open_log_file ↵pierreN2017-06-161-1/+6
| | | | | (#19) Signed-off-by: Pierre N <pierreN@users.noreply.github.com>
* journald adaptor: test with sudo privileges (#18)Gordan Markuš2017-06-161-2/+2
| | | | | | | | Users in the 'systemd-journal' group can see all journalctl messages. Usually the user is not found in that group and the "start_systemd_journal_test.sh" script might hang because of this. Starting "dlt-system" with sudo privileges will mitigate this risk. Signed-off-by: Gordan Markuš <gordan.markus@pelagicore.com>
* cmake: fix unit tests compilation with systemd (#17)Gordan Markuš2017-06-161-1/+1
| | | | | | | | | By enabling the WITH_SYSTEMD and WITH_DLT_UNIT_TESTS cmake flags the build will fail with an error due to a linking issue: ../src/daemon/libdlt_daemon.so: undefined reference to `sd_booted' collect2: error: ld returned 1 exit status Signed-off-by: Gordan Markuš <gordan.markus@pelagicore.com>
* Input parameter check & Error message modification (#15)Amber Bhardwaj2017-06-166-62/+83
| | | | | | | | | | | | | | | | | | | * Update dlt-kpi-process.c Added input param checking and added return value in non-void functions. * Update dlt-kpi.c Added Input param checking and added return values in non-void functions. * Input parameter check and Error msg modification Added null pointer checking Added input parameter checking Added return values in non-void functions Added proper error message for debugging purpose Signed-off-by: amberbhardwaj <amber.bhardwaj10@gmail.com>
* Tell cmake to use README.md instead of README to fix doc generation (#16)pierreN2017-05-221-1/+1
| | | Signed-off-by: Pierre N <pierreN@users.noreply.github.com>
* dlt-system-process-handling: fix warning (#12)Gordan Markuš2017-05-181-1/+1
| | | | | | | | | | | In function ‘daemonize’: dlt-daemon/src/system/dlt-system-process-handling.c:84:5: warning: this ‘for’ clause does not guard... [-Wmisleading-indentation] for(i = getdtablesize(); i >= 0; i--) ^~~ dlt-daemon/src/system/dlt-system-process-handling.c:87:2: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the ‘for’ int fd = open("/dev/null",O_RDWR); ^~~ Singed-off-by: Gordan Markuš <gordan.markus@pelagicore.com>
* dlt_daemon_connection_types: fix build warnings (#14)Gordan Markuš2017-05-181-1/+1
| | | | | | | | | | | | dlt-daemon/src/daemon/dlt_daemon_event_handler.c: In function ‘dlt_daemon_handle_event’: dlt-daemon/src/daemon/dlt_daemon_event_handler.c:131:30: waring: cast from pointer to integer of different size [-Wpointer-to-int-cast] DltConnectionId id = (DltConnectionId)ev->data.ptr; ^ dlt-daemon/src/daemon/dlt_daemon_event_handler.c: In function ‘dlt_connection_check_activate’: dlt-daemon/src/daemon/dlt_daemon_event_handler.c:381:27: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] ev.data.ptr = (void *)con->id; ^ Signed-off-by: Gordan Markuš <gordan.markus@pelagicore.com>
* Updated README (#10)Jeremiah C. Foster2017-05-101-10/+3
| | | | Changed formatting by removing excess horizontal lines Updated source code URLs
* dlt-adaptor-udp, dlt-adaptor-stdin: implement get of verbosity level from ↵Gianfranco Costamagna2017-03-292-4/+86
| | | | | | | | | | | input (#9) * dlt-adaptor-udp, dlt-adaptor-stdin: implement get of verbosity level from input, defaulting to INFO when wrong type or none is provided Signed-off-by: Gianfranco Costamagna <gianfranco.costamagna@abinsula.com> * Fix minor typo (from gunnarx)
* Merge pull request #8 from amberbhardwaj/masterRalphNiemeyer2017-03-231-0/+5
|\ | | | | Added Description in dlt-system.conf
| * Added Description in dlt-system.confAmber Bhardwaj2017-02-161-0/+5
| | | | | | | | | | | | | | | | | | | | Added additional information about how to enable syslog feature properly. If still facing issue while capturing syslogs through client (e.g. dlt_viewer) after following given steps then install listed packages: build-essential build-essential checkinstall pkg-config libsystemd-journal-dev Note: It's is a known issue when you try to capture syslog via dlt_viewer then syslogs wouldn't come in dlt_viewer because of improper configuration (won't route the log to port 47111) of dlt-system.
* | Merge pull request #7 from clipka/devel_dlt_client_cleanup_fixRalphNiemeyer2017-03-239-16/+119
|\ \ | | | | | | dlt-client: fix dlt_client_cleanup memory handling
| * | dlt-client: fix dlt_client_cleanup memory handlingOnkar Palkar2017-02-069-16/+119
| |/ | | | | | | | | | | | | Setter function introduced for seting up string parameters which are free'd in dlt_client_cleanup Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
* | Merge pull request #6 from clipka/devel_fatal_trapRalphNiemeyer2017-03-232-0/+20
|\ \ | | | | | | CMake Option: Trigger segmentation fault in case of FATAL log
| * | CMake Option: Trigger segmentation fault in case of FATAL logChristoph Lipka2017-02-022-0/+20
| |/ | | | | | | | | | | | | | | When the user library receives a log with log level DLT_LOG_FATAL it triggers a segmentation fault to provide information to the developer via tools like coredump. Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
* | Merge pull request #5 from clipka/devel_connection_handling_fixesRalphNiemeyer2017-03-235-3/+56
|\ \ | |/ |/| Daemon connection handling fixes
| * dlt-daemon: Fix use after free potential issueFrederic Berat2017-02-015-3/+48
| | | | | | | | | | | | | | | | | | | | | | | | | | In dlt_daemon_send_all_multiple, if the connection was broken, we closed it before getting the next available connection. This must be avoided by having a temporary next pointer. The same kind of problem is valid for pointers coming from the epoll interface. The kernel can provide back connection pointer that are not valid any longer. Therefore, we need to use an ID instead of the pointer value to retrieve the connections. Signed-off-by: Frederic Berat <fberat@de.adit-jv.com> Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
| * Event handling: Fix connection destroy bugChristoph Lipka2017-02-012-0/+8
|/ | | | | | | | | It might happen that an event is part of the epoll event queue that belongs to a connection which was destroyed before the event is handled. Due to this, the event handling main loop might stop and the daemon exits. This misbehavior is fixed with this patch. Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
* Remove duplicate README (#4)Gianpaolo Macario2017-02-011-86/+0
| | | | | | | | README.md exists and is displayed by default in GitHub. Remove duplicate README to avoid confusion in case only one of the two is updated in the future. Signed-off-by: Gianpaolo Macario <gianpaolo_macario@mentor.com>
* daemon: check payload length before cast to structChristoph Lipka2017-01-311-15/+66
| | | | Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
* - Added missing build steps to INSTALLmetal4people2016-11-171-2/+9
| | | | | | | I've faced with cmake error while generating make files for dlt-daemon. After searching a bit, I've found out that zlib and dbus packages needed to be installed for default dlt-daemon configuration. I used Ubuntu 16.04 for build and after I installed the packages cmake generated project successfully. I think that is good idea to mention required packages in INSTALL notes in order to save the time of someone who will try to build dlt-daemon as I did.
* Switch to working version 2.17 UNSTABLEChristoph Lipka2016-11-161-2/+2
| | | | Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
* pkg-config: fix library directory.Mathieu Duponchelle2016-11-151-1/+1
| | | | | The install destination might be lib64, and a CMake variable exists that contains this information, CMAKE_INSTALL_LIBDIR.
* Update version to 2.16.0 STABLEv2.16.0Christoph Lipka2016-10-242-2/+62
| | | | | | Update Version in CMake Version and release notes. Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
* doc: Documenatation updateChristoph Lipka2016-10-2411-96/+433
| | | | Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
* Systemd-journal-test: Add WITH_DLT_UNIT_TEST flag when building sourcesChristoph Lipka2016-10-241-1/+1
| | | | Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
* Smoketest: Offline LogstorageOnkar Palkar2016-10-241-0/+206
| | | | | | | This test script checks basic functionality of DLT Offline Logstorage feature. Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
* Smoketest: MultinodeOnkar Palkar2016-10-241-0/+302
| | | | | | This test script checks basic functionality of DLT Multinode feature Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
* Unit Test: Event handlingChristoph Lipka2016-10-246-6/+701
| | | | | | Unit tests for DLT Daemon connection and event handling Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
* Provision to test static functionOnkar Palkar2016-10-241-0/+9
|
* Unit Test: MultiNodeOnkar Palkar2016-10-245-14/+631
| | | | | | Unit test for DLT MultiNode extension Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
* Unit Test: Multinode Unit test preparation scriptS. Hameed2016-10-241-0/+137
| | | | | Signed-off-by: S. Hameed <shameed@jp.adit-jv.com> Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
* CMake: Add option to build unit test binariesChristoph Lipka2016-10-241-2/+10
| | | | | | | Adds a configuration option for building gtest framework and unittests. Default setting: OFF Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
* dlt-system-filetransfer: fix bug caused by malloc assertManikandan C2016-10-241-6/+11
| | | | | | | Remove malloc assert in inappropriate location and add error logs in case of failures. Signed-off-by: Manikandan C <Manikandan.Chockalingam@in.bosch.com>
* Environment variables for library ringbufferChristoph Lipka2016-10-242-1/+56
| | | | | | | | | | | | This patch adds environment variables to configure user library ringbuffer and step size. The following variables can be set: DLT_USER_BUFFER_MIN - for minimal size DLT_USER_BUFFER_MAX - for maximal size DLT_USER_BUFFER_STEP - for step size Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
* DLT_PTR macro: Improve implementation and function API addedChristoph Lipka2016-10-243-7/+43
| | | | | | | | The original implementation result in a compile error when DLT_PTR is used with other macros like DLT_STRING. This patch overcomes that problem. Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
* MultiNode: Specify config file location in dlt.confChristoph Lipka2016-10-244-9/+33
| | | | Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
* dlt-client: Use correct port on connectChristoph Lipka2016-10-242-25/+31
| | | | Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
* process user message: Fix bound handlingFrederic Berat2016-10-243-26/+114
| | | | | | | | | | | | | | | | | | | | | | | | 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-system-filetransfer: Fix compiler warningsChristoph Lipka2016-10-241-1/+2
| | | | Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
* Remove C99 style comments in include directoryS. Hameed2016-10-242-21/+19
| | | | Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
* Dlt-Receive: Use PRIxxx macros for printf variablesChristoph Lipka2016-10-241-2/+3
| | | | | | This change is done to remove compiler warnings. Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>