| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
|
|
| |
Fix error in suggested best-practice code snippet
|
|
|
|
|
|
| |
Signed-off-by: Andrei-Mircea Rusu <andrei-mircea.rusu@continental-corporation.com>
Signed-off-by: Andrei-Mircea Rusu <andrei-mircea.rusu@continental-corporation.com>
Co-authored-by: Andrei-Mircea Rusu <andrei-mircea.rusu@continental-corporation.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Up to now the default for logging is stdout (DLT_LOG_TO_CONSOLE).
Scenario:
program_a arg1 | dd of=/my-beauty-case
If program_a wants to log to dlt, but it fails to connect to the socket:
socket /run/dlt-files/dlt cannot be opened, error:Resource temporarily
unavailable. Retrying later...
The error will be logged by default to stdout leading in the best case
to an error and in the worst to damaged partitions.
To solve this log to STDERR (DLT_LOG_TO_STDERR) by default.
Co-authored-by: Tobias Kaufmann <Tobias.KA.Kaufmann@bmw.de>
|
|
|
|
|
|
|
|
| |
dlt-daemon allows configuration of multiple addresses.
Before this commit the daemon exits when the first binding fails.
This commit changes the behavior so that the daemon only exits
if none of the configured addresses can be opened.
Signed-off-by: Alexander Mohr <alexander.m.mohr@mercedes-benz.com>
|
|
|
|
|
|
| |
if an application sends a message with a level below context level
and the level enforcement is on, the message will be dropped
Signed-off-by: Alexander Mohr <alexander.m.mohr@mercedes-benz.com>
|
|
|
| |
Make the user find the static library by pkg-config, when BUILD_SHARED_LIBS=OFF.
|
|
|
|
|
|
|
| |
Re-add install adaptor-udp service for WITH_DLT_ADAPTOR=ON.
Signed-off-by: Daniel Weber <daniel.w.weber@mercedes-benz.com>
Co-authored-by: Daniel Weber <daniel.w.weber@mercedes-benz.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* common: Handle returned value
- The returned value of supporting APIs should be checked correctly.
- In case of extended header, the buffer of header must be checked
before proceeding further.
Signed-off-by: Bui Nguyen Quoc Thanh <thanh.buinguyenquoc@vn.bosch.com>
* console: Handle returned value
In case of parsing dlt file, the failed messages
should be skipped by evaluating returned value
Signed-off-by: Bui Nguyen Quoc Thanh <thanh.buinguyenquoc@vn.bosch.com>
|
|
|
|
|
| |
Set link to github actions in COVESA/dlt-daemon.
Signed-off-by: Saya Sugiura <ssugiura@jp.adit-jv.com>
|
|
|
|
|
|
|
| |
- Add ctest in Actions
- Remove travis ci
- Update Build Status in README.md
Signed-off-by: Bui Nguyen Quoc Thanh <thanh.buinguyenquoc@vn.bosch.com>
|
|
|
|
|
|
|
|
| |
In the dlt_config_file_set_section function of dlt_config_file_parser.c:
s-name is not set to null after free.
It will be freed again in the dlt_config_file_release function.
Signed-off-by: Zhongyang.Bao <Zhongyang.Bao@zeekrlife.com>
|
|\
| |
| | |
filetransfer: Fix getFileCreationDate2 stat check
|
| |
| |
| |
| | |
Signed-off-by: Andrei-Mircea Rusu <andrei-mircea.rusu@continental-corporation.com>
|
|\ \
| | |
| | | |
cmake: Add options to enable/disable each dlt console tool
|
| |/
| |
| |
| |
| |
| |
| | |
This commits adds several cmake options to enable or disable
each console tool separately.
Signed-off-by: Alexander Mohr <alexander.m.mohr@daimler.com>
|
|\ \
| | |
| | | |
cmake: Add option to enable each adaptor by itself
|
| |/
| |
| |
| |
| |
| |
| |
| |
| |
| | |
by itself
This commit adds two options to enable the udp and stdin
adaptor indendent from each other.
Therefore only the necessary component can be enabled
Signed-off-by: Alexander Mohr <alexander.m.mohr@daimler.com>
|
|\ \
| |/
|/| |
Fix the target name in documentation
|
|/
|
|
|
|
| |
Closes #365
Signed-off-by: Martin Willers <M.Willers@gmx.net>
|
|
|
| |
Signed-off-by: Saya Sugiura <ssugiura@jp.adit-jv.com>
|
|
|
|
|
|
| |
- Fix DLT Test Client expected string in Test 5 (for Better formatting of RAWD #291)
- Fix DLT Test User Disable Macro usage: #cmakedefine01 DLT_DISABLE_MACRO (from dlt_user.h.in) still defines the macro even though DLT_DISABLE_MACRO is not set inside CMake.
Signed-off-by: Andrei-Mircea Rusu <andrei-mircea.rusu@continental-corporation.com>
|
|
|
|
|
| |
Closes #352
Signed-off-by: Saya Sugiura <ssugiura@jp.adit-jv.com>
|
|
|
|
|
| |
Closes #353
Signed-off-by: Saya Sugiura <ssugiura@jp.adit-jv.com>
|
|
|
|
|
|
| |
Also tested with the latest version running on Windows XP (gcc-5.4.0)
(http://www.crouchingtigerhiddenfruitbat.org/cygwin/timemachine.html).
Signed-off-by: Adder <adder_2003@yahoo.com>
|
|
|
| |
Signed-off-by: Andrei-Mircea Rusu <andrei-mircea.rusu@continental-corporation.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* some previous dlt_vlog was using %lu specifier to display an uint64_t
variable. However, this isn't portable as depending on the system you're
targeting (32bits, 64bits, Linux, android, etc.), the uint64_t can be an
unsigned long long. This prevents compilation on system where -Werror
and -Wformat is activated coupled with a less tolerant compiler
(e.g clang). Instead, PRIxN macros (e.g PRIu64 for uint64_t) are now
used as specifier to ensure compatibility for each platforms
* for bytes_read which is of type ssize_t, zd specifier is used as
indicated in C99 standard
* for st_size (from struct stat) which is of type off_t, POSIX says that
it should be a signed integer. To be safe, we now use %jd specifier to
ensure that it will fit correctly
|
|
|
|
|
|
|
|
|
| |
Some QNX components when bringing up send messages of severity type "FATAL" instead of "INFO" via slogger2.
QNX component send messages of payload data_type = SLOG2_TYPE_ONLINE. The buffer is new and just came online.
dlt-qnx-slogger2 does not support info->data_type = SLOG2_TYPE_ONLINE and therefore sets the wrong log level.
This commit forcibly change the severity level for all info->data_type = SLOG2_TYPE_ONLINE to SLOG2_INFO.
Signed-off-by: Dinidrol <vamovitch@gmail.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
Scan of port 3490 from a Linux machine by
'nmap -v -p 3490 -Pn IP_address_of_QNX_board' leads to dlt-daemon crash.
It is necessary to parse the code of the returned error in errno.
If errno = ECONNABORTED it means there was an attempt to scan port 3490
and on this error there is no need to exit the application by error.
Fix for the issue https://github.com/GENIVI/dlt-daemon/issues/341
Signed-off-by: Ruslan Valovyi <ruslan.valovyi@volvocars.com>
|
|
|
|
|
|
|
|
|
| |
when using default const string.
this location is free()'ed in the cleanup routine, leading to bugcheck upon
exit. Use strdup to allocate memory, that can be free()'ed.
Signed-off-by: Marc TITINGER <marc.titinger@non.se.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
dlt_mkdir_recursive runs mkdir for directories which already exist.
This fails if the directory already exists and the dlt-daemon
does not have the permissions to change the mode on the directory.
On many systems the directories are not created by dlt-daemon
itself but rather through the build system or while the
root filesystem is created.
At this stage the permissions are already set properly
and they might be less permissive than what dlt-daemon expects
but still sufficient.
For example is rwx not necessary for each directory in the tree.
Before this commit the following scenario would fail:
* Path is set to /mnt/dlt/
* mnt has 755 permissions for root
* dlt has 755 for dlt user
As dlt daemon would try to run mkdir on /mnt
dlt_mkdir_recursive would fail due to lack of permissions
although the permissions are sufficient for dlt-daemon to
work and write its files into /mnt/dlt/
With this commit /mnt would not be created anymore because it
already exists.
If the permissions are not sufficient for dlt-daemon to write in
it will fail when trying to create files and exit at this point
Signed-off-by: Alexander Mohr <alexander.m.mohr@daimler.com>
|
|
|
|
|
|
|
| |
* function atomic_compare_exchange_strong expect a pointer to an _Atomic type. dlt_user_initialised is now an atomic_bool
* function dlt_set_filesize_max returned a DLT_LOG_ERROR which isn't a DltReturnValue. This commit replaces DLT_LOG_ERROR by DLT_RETURN_ERROR which is the expected return type
Signed-off-by: Sebastien RAILLET <sebastien.raillet@marelli.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Before this commit, dlt-daemon sockets were created inside
/data/local/tmp. This works but have many drawbacks:
- /data/local/tmp isn't always a tmpfs depending of the android
system you have. Means sockets are potentially created on a
filesystem which binds to a real device
- as the sockets are created by the dlt-daemon itself, this
prevent the usage of specific SELinux labels / contexts as
they will inherit the label / context from its parent
location (e.g the ones from /data/local/tmp). This prevent a
fine control of the SELinux label / context that you would like
to apply on them
* This commit adapts the dlt-daemon in such way:
- application and control sockets are now created inside
/dev/socket which is the standard path for sockets on android
- these sockets are now created by init (see dlt-daemon.rc) and
their fds are recovered by dlt-daemon through a specific android
API (dlt_daemon_unix_android_get_socket). If the fds can't be
recovered, we fallback to the previous mechanism by creating
by ourself the sockets (even if this will prevent SELinux
label / context on this socket)
- all these modifications have been put under compilation flag
for android
Signed-off-by: Sebastien RAILLET <sebastien.raillet@marelli.com>
|
|
|
|
|
|
| |
rather self-exp: 32bits native hint will not hold 8 bits.
Signed-off-by: Marc TITINGER <marc.titinger@hotmail.com>
|
|
|
|
| |
Signed-off-by: Saya Sugiura <ssugiura@jp.adit-jv.com>
|
|
|
|
| |
Signed-off-by: Saya Sugiura <ssugiura@jp.adit-jv.com>
|
|
|
|
|
|
|
|
| |
Add gtest and gtest_main libraries if we don't install gtest package in system
Create a new message fillter config file when running gtest gateway shell script
set type of control message is CONTROL_MESSAGE_ON_DEMAND when testing dlt_gateway_send_control_message function
Signed-off-by: Le-Tin <Tin.Le@vn.bosch.com>
|
|
|
|
|
|
|
| |
Extern logging_mode and logging_handle variables from commom.c
Reusing dlt_log_init() in cases dlt library internal log file is not opened and logging_mode is DLT_LOG_TO_FILE
Signed-off-by: Le-Tin <Tin.Le@vn.bosch.com>
|
|
|
|
|
|
|
|
|
| |
Adding stdlib.h into dlt-test-cpp-extension.cpp solves the compile issue:
dlt-test-cpp-extension.cpp:89:47: error: 'malloc' was not declared in this scope
char *non_const_string = (char *)malloc(17);
This occoured while building for QNX.
Signed-off-by: Darian Biastoch <dbiastoch@de.adit-jv.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This commit introduces the use of atomic_compare_exchange at beginning
of both dlt_init functions. This function checks in an atomic way, if
'dlt_user_initialised' is 'false' (first run through dlt_init) and directly
sets it 'true'. If a second thread also entered dlt-init at this point of time,
its call of atomic_compare_exchange will return false and dlt_init will be
immediately left due to that. This is only interesting for startup, because
before each call of dlt_init, there is a check on 'dlt_user_initialised' that
will fail for all later executions.
Signed-off-by: Darian Biastoch <dbiastoch@de.adit-jv.com>
|
|
|
|
|
|
|
|
|
|
| |
Removed duplicate "payload" pointer in dlt_client_send functions inside dlt_client.c.
This pointer was only needed to avoid one cast within a function call.
Now this cast is performed to get rid of clang-tidy analyzer warnings.
In addition to that, malloc calls were replaced through calloc to avoid a subsequent
memset with zero.
Signed-off-by: Darian Biastoch <dbiastoch@de.adit-jv.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
Currently, when building dlt debian package, our libdlt.so can not be linked to
other binary consoles.
Adding build option of dh_shlibdeps to specific the path of libdlt.so, and
linking step should not be broken.
With default enabling flag WITH_MAN in debian rules, pandoc package must be
added in Build_Depends option for debian control.
Signed-off-by: Dinh Cong Toan <toan.dinhcong@vn.bosch.com>
|
|
|
|
|
|
|
|
| |
Before cancelling the listening thread, try to shutdown
and close the socket so that the listening thread is
terminated intentionally
Signed-off-by: Bui Nguyen Quoc Thanh <thanh.buinguyenquoc@vn.bosch.com>
|
|
|
|
|
|
|
|
| |
The describtion of dlt_init_common was pretty meaningless,
that's why an additional sentence and a list of all parsed
evironment variables was added.
Signed-off-by: Darian Biastoch <dbiastoch@de.adit-jv.com>
|
|
|
|
| |
Signed-off-by: Saya Sugiura <ssugiura@jp.adit-jv.com>
|
|
|
|
|
|
|
| |
testfile.dlt was not detected correctly with ctest, so corrected
WORKING_DIRECTORY accordingly.
Signed-off-by: Saya Sugiura <ssugiura@jp.adit-jv.com>
|
|
|
|
|
|
|
| |
gtest_dlt_common was failing when ring buffer size was modified.
Amount of data to be written is corrected in the test code.
Signed-off-by: Saya Sugiura <ssugiura@jp.adit-jv.com>
|
|
|
|
| |
Signed-off-by: Saya Sugiura <ssugiura@jp.adit-jv.com>
|
|
|
|
|
|
|
|
| |
This also removed following tests to more appropriate place:
- dlt-test-preregister-context.c
- dlt-test-cpp-extension.cpp
Signed-off-by: Saya Sugiura <ssugiura@jp.adit-jv.com>
|
|
|
|
| |
Signed-off-by: Saya Sugiura <ssugiura@jp.adit-jv.com>
|
|
|
|
| |
Signed-off-by: Saya Sugiura <ssugiura@jp.adit-jv.com>
|