dlt-daemon icon indicating copy to clipboard operation
dlt-daemon copied to clipboard

dlt_daemon_client.c Adding NULL check for tok

Open Manojcv55 opened this issue 5 years ago • 7 comments

dlt_daemon_offline_logstorage.c : Adding NULL check for application dlt_daemon_common.c : fix printf format %d to %ld that formats a long dlt_common.c: fix printf format %d to %ld that formats a long dlt_user.c : Fix for Memory Leak dlt-daemon.c : Fix for Resource Leak dlt_offline_trace.c : Fix for Resource Leak

Signed-off-by: Mvaradaraj2 [email protected]

Manojcv55 avatar Jul 14 '20 04:07 Manojcv55

Fix for Resource, Memory Leak

Manojcv55 avatar Jul 14 '20 04:07 Manojcv55

This pull request introduces 3 alerts when merging af6c958cca46a3325fad56be95aef68e82c6ccd1 into f1ac087c766827b1d0ed9c3a814b3cc052e948f2 - view on LGTM.com

new alerts:

  • 3 for Wrong type of arguments to formatting function

lgtm-com[bot] avatar Jul 14 '20 04:07 lgtm-com[bot]

@Manojcv55 Can you fix the alerts reported from LGTM? Also, the first line of commit message should have the summary of the change. Please update as such. Thank you!

ssugiura avatar Jul 14 '20 23:07 ssugiura

@ssugiura Changes has been made

Manojcv55 avatar Jul 17 '20 06:07 Manojcv55

This pull request introduces 3 alerts when merging 78643c3176f446a0d3fb908cdfddbe2101f261a5 into 5843a3f6211d002c4859533ff7ce1791f09105a4 - view on LGTM.com

new alerts:

  • 3 for Wrong type of arguments to formatting function

lgtm-com[bot] avatar Sep 01 '20 15:09 lgtm-com[bot]

@Manojcv55 Any updates? I guess you can close this one and take care of review feedbacks in https://github.com/GENIVI/dlt-daemon/pull/235.

ssugiura avatar Nov 10 '20 09:11 ssugiura

@Manojcv55 Friendly reminder to update PR. Also please resolve merge conflicts. Thanks in advance.

ssugiura avatar Jul 15 '21 02:07 ssugiura

Replaced by https://github.com/COVESA/dlt-daemon/pull/418

michael-methner avatar Oct 26 '22 08:10 michael-methner