| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
| |
Unit test for DLT MultiNode extension
Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
|
|
|
|
|
|
|
| |
Remove malloc assert in inappropriate location and add error logs in case of
failures.
Signed-off-by: Manikandan C <Manikandan.Chockalingam@in.bosch.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
| |
Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
|
|
|
|
| |
Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
| |
Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
|
|
|
|
|
|
| |
This change is done to remove compiler warnings.
Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
|
|
|
|
| |
Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
|
|
|
|
|
|
|
|
| |
Currently, allocated memory for context description is not freed in dlt_free().
free() for each member variables of dlt_user.dlt_ll_ts[i] is added in the for block in dlt_free().
Signed-off-by: Yusuke Sato <yusuke-sato@apn.alpine.co.jp>
Change-Id: I52a81cf2918cb0014fdcfc666cb8bb93ca7cb525
|
|
|
|
|
|
|
|
|
|
|
| |
ECU ID that is not default value
dlt-daemon overwrites ECU ID even if user log message already has the ECU ID that is not default value.
This phenomenon happens when application uses dlt_forward_msg().
Therefore, dlt-daemon is modified not to overwrite ECU ID if log message has already has the ECU ID.
Signed-off-by: Yusuke Sato <yusuke-sato@apn.alpine.co.jp>
Change-Id: Icbc6304e5571242bf75ff0d32cbe4b004f1d7f9d
|
|
|
|
|
|
|
|
|
|
| |
registration
Option of get log info response during context registration was fixed to 7.
This modification enables to change that option.
Signed-off-by: Yusuke Sato <yusuke-sato@apn.alpine.co.jp>
Change-Id: Ia75f15cdb33e0bffd77c5ba71a1960e820714e61
|
|
|
|
|
|
|
| |
Configuration of daemon FIFO (default: /tmp/dlt) size is added to dlt.conf.
Signed-off-by: Yusuke Sato <yusuke-sato@apn.alpine.co.jp>
Change-Id: I05cc56b9a05e4c3a0a1af49c374f35ae10f4674d
|
|
|
|
|
|
|
|
|
|
| |
user buffer
Handle of /tmp/dlt cannot be reset if dlt-daemon is killed during output user buffer because dlt_user_log_resend_buffer() never returns DLT_RETURN_PIPE_ERROR.
Return value of dlt_user_log_resend_buffer() is modified to pass return value of dlt_user_log_out3() without change.
Signed-off-by: Yusuke Sato <yusuke-sato@apn.alpine.co.jp>
Change-Id: I506103ebebcdb3c145d1f0719a3c8caffc8d89c1
|
|
|
|
|
|
|
| |
Output debug log: File name contained in inotify event message.
Signed-off-by: Yusuke Sato <yusuke-sato@apn.alpine.co.jp>
Change-Id: I276b1373ae3b60347b4af44681f5c601f62dde4c
|
|
|
|
|
|
|
|
| |
Segfault is caused by dlt_user_check_buffer() if this function is called during increasing the user buffer.
To avoid this issue, semaphore has to be added.
Signed-off-by: Yusuke Sato <yusuke-sato@apn.alpine.co.jp>
Change-Id: Ie56f2fb73aeacb835180468e57c9338d2ffdb072
|
|
|
|
|
|
|
|
| |
Currently, dlt-system just wait for free space of user buffer > 50%.
To speed up the file transfer, dlt-system should try to resend user buffer contents actively.
Signed-off-by: Yusuke Sato <yusuke-sato@apn.alpine.co.jp>
Change-Id: I09d76acacf65b2c68559a05d45e00f35f402316a
|
|
|
|
|
|
|
|
| |
When file transfer feature of dlt-system is used on the 64 bit OS, file name is overwritten at the unique name creation phase.
File serial number should be maximum 10-digit decimal number.
Signed-off-by: Yusuke Sato <yusuke-sato@apn.alpine.co.jp>
Change-Id: I18fac363a103b6806ed9eb003ebbb305af7e7f58
|
|
|
|
|
|
|
|
| |
There is possibility that dlt-dbus continues to allocate memory for each filtered DBus message.
To solve this issue, free() is added after output the DBus log.
Signed-off-by: Yusuke Sato <yusuke-sato@apn.alpine.co.jp>
Change-Id: I393552c97035b6b06673cd5a9fd05c9158f47cdc
|
|
|
|
|
|
|
|
|
|
| |
This function allows an application developer to check if the DLT
LogLevel is enabled before starting any log string creation. It can be
used in application specific logging wrapper functions.
Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
Change-Id: I4c269b12bfdeae29a4e795385cdbc54a579ba621
|
|
|
|
|
|
|
| |
Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
Topic: GENDLT-425
Change-Id: I4ec56d714804d9ee4fa904b71c58dafd00990b4f
|
|
|
|
|
|
| |
Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
Change-Id: Id01c9a11e4b5ccf4166f10c5e45363f3a7c00535
|
|
|
|
|
|
| |
Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
Change-Id: I211222d04350820098bf56dfc5b7aee50e2529f0
|
|
|
|
|
| |
Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
Change-Id: I504664fa8d2a4c6d9449a1911bf139dfe6005d76
|
|
|
|
|
|
|
|
| |
This commit resloves a bug introduced from the commit:
CommonControl: Unix socket path and ecuid parsing for control applications
Signed-off-by: S. Hameed <shameed@jp.adit-jv.com>
Change-Id: Ifd4af24cab3ef41fda86fdf63a559438b51eca6e
|
|
|
|
|
|
| |
Signed-off-by: Manikandan C <Manikandan.Chockalingam@in.bosch.com>
Change-Id: I023c6bccbe3977d50b77bb601df80643d8e2d82c
|
|
|
|
|
|
|
|
| |
1. support for setting log level using wildcards for both app and context
2. support for setting entire system log level
Signed-off-by: Manikandan C <Manikandan.Chockalingam@in.bosch.com>
Change-Id: I92f8c5461903f092cd50f05f644013432940a87b
|
|
|
|
|
|
|
|
| |
This macro internally checks the size of a void pointer to decide if the
pointer address is sent with DLT_HEX32 or DLT_HEX64 macro.
Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
Change-Id: Ie316ac411f2174e0760e09244d04b83c8006f55d
|
|
|
|
|
|
|
|
| |
The user log message handler was returning 1 instead of -1 in case of
failure, which was not expected.
Signed-off-by: Frederic Berat <fberat@de.adit-jv.com>
Change-Id: Id03b2879f631e804f9939a9f52132be26491f888
|
|
|
|
|
|
|
|
| |
The activation of the connection is now centralized in one common
function.
Signed-off-by: Frederic Berat <fberat@de.adit-jv.com>
Change-Id: I86703eab411088905a85f526d5102875e2c92b7e
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
| |
Incremented client connections in case of serial connection also
Signed-off-by: Manikandan C <Manikandan.Chockalingam@in.bosch.com>
Change-Id: I3eb56ca67d46a7474cb750769c65c6548d40554f
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
| |
Some configuration elements are mandatory, some other optionals.
The Gateway configuration handler now supports both of them.
If a mandatory configuration is missing, the Section is ignored, and
next one is handled.
If all sections are wrong, the Gaeteway fails its initialization.
Signed-off-by: Frederic Berat <fberat@de.adit-jv.com>
Change-Id: I41177783e7c5cae528b54b48f558032e672c9451
|
|
|
|
|
|
|
|
|
|
|
| |
The New SendSerialHeader configuration allows to define if a gateway
needs this serial header to be sent. It can be configured differently
for each gateway.
If the configuration is not available, the DLT global configuration is
used.
Signed-off-by: Frederic Berat <fberat@de.adit-jv.com>
Change-Id: If37cb0dfc7e765a1752e438e61c1622ffb7dbc4f
|
|
|
|
|
|
| |
Signed-off-by:Frederic Berat <fberat@de.adit-jv.com>
Change-Id: I9b387d5713f5ae7b6fcfadb22d804fdb97534e44
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
A list of control messages can be specified in the dlt_gateway.conf.
These messages then are send immediately to the passive node after the
connection has been established.
Currently supported control messages are
- GET_LOG_INFO
- GET_SOFTWARE_VERSION
Having the "SendControl" property is now mandatory for a passive node
configuration, but the list of control messages might be empty.
Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
Change-Id: I0ae4501449bf264c2d6f32724b0f035926115611
|
|
|
|
|
|
| |
Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
Change-Id: Iffbfb4693b172065d6000ecbf23342498341fa73
|
|
|
|
|
|
|
|
| |
support long options
Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
Change-Id: I2bac5e48a5d210b544a96fe96dc322f28ac472fe
|
|
|
|
|
|
| |
Signed-off-by: S. Hameed <shameed@jp.adit-jv.com>
Change-Id: I6d4e893b0d2d76edf6235040ffd215a9305b7a22
|
|
|
|
|
|
|
|
| |
properties
Signed-off-by: S. Hameed <shameed@jp.adit-jv.com>
Change-Id: I52c70f051c407e7b7e0281b1f48cdaf2f9dc5a47
|
|
|
|
|
|
| |
Signed-off-by: S. Hameed <shameed@jp.adit-jv.com>
Change-Id: I4466364e6340c526c8ce6f22d80c3ffc1b8be4ef
|
|
|
|
|
|
|
|
|
|
|
|
| |
- double free SIGSEGV caused by uninitialized free'd value in case
of uncomplete Logstorage filter configuration
- load configuratation returns error when the last configuration is
invalid, but previous configurations are valid which results in
memory leaks
Signed-off-by: Christoph Lipka <clipka@jp.adit-jv.com>
Change-Id: I44103ed6b17e4b02eb35bdb575c234ff12bb5208
|
|
|
|
|
|
|
|
|
| |
A new return value DLT_RETURN_USER_BUFFER_FULL is introduced which is
returned by all dlt_user_log_write_* and dlt_log_* functions when the
data to be added to the user buffer exceeds its size (datasize >
DLT_USER_BUF_MAX_SIZE)
Signed-off-by: Lutz Helwing <lutz_helwing@mentor.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Commit d319ebf4d2d449d70befae0eebbdd82c19c5315f introduced new handling
of the situation when writing data to the user buffer would exceed its
size. This was achieved by truncating the data to fit. This solution had
a problem:
It could lead to a buffer overflow when the updated arg_size gets negative
(actually undefined value because it is unsigned).
As a quick fix the behaviour is reverted to the state before the
aforementioned commit.
Signed-off-by: Lutz Helwing <lutz_helwing@mentor.com>
|
|
|
|
|
|
|
|
| |
With commit d319ebf4d2d449d70befae0eebbdd82c19c5315f a bug was
introduces which lead to some messages getting not transmitted.
Change-Id: I9f20177fbc7db9d7caf527e14159444b5fec168a
Signed-off-by: Lutz Helwing <lutz_helwing@mentor.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Previously if messages containing strings or raw data were exceeding the
maximum message size (DLT_USER_BUF_MAX_SIZE, currently set to 1390) they
were discarded completely.
This behaviour is changed so that too large data is truncated and
sent anyway.
Commented out part of unit test for dlt_log_raw abnormal where
negative values for unsigned length were passed.
Change-Id: I4f6bf152e7125924717c941ad7ebb8fd79787715
Signed-off-by: Lutz Helwing <lutz_helwing@mentor.com>
|
|
|
|
|
|
|
|
|
|
|
| |
If a file get written to after it appeared in a folder that is watched
by the dlt-system filetransfer thread, the application tries to send
this file multiple times, causing it to crash. This patch fixes this by
canceling the sending process more gracefully if the file cannot be
found.
Change-Id: Id19b72061bed41642d12cd33afeaa41fb444b2c8
Signed-off-by: Lutz Helwing <lutz_helwing@mentor.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
For each file that is supposed to be sent, dlt-system will check if a
client is connected to the daemon. If not, it waits and keeps checking
once per second, until a client is connected.
Files will still be compressed and moved to the .tosend subdirectory,
regardless of the client connection.
Changed the default startup timeout in the configuration file to 0.
Change-Id: I5850adb51c381bc6279ac59f5bd985491ba5caf3
Signed-off-by: Lutz Helwing <lutz_helwing@mentor.com>
|