diff options
| author | Lassi Marttala <lassi.lm.marttala@partner.bmw.com> | 2012-02-21 10:56:16 +0100 |
|---|---|---|
| committer | Christian Muck <christian.muck@bmw.de> | 2012-04-04 14:30:01 +0200 |
| commit | 83059468ecd8ca6a68901f20477615cb58808c94 (patch) | |
| tree | f0809acf0e0c9d42ce823bb40ef528f924e1de41 /src/lib/dlt_user.c | |
| parent | 71cc80ac25166a65b0cea363c4d77314bbaf7f5e (diff) | |
| download | DLT-daemon-83059468ecd8ca6a68901f20477615cb58808c94.tar.gz | |
GENDLT-15, Rename buffer size constant to avoid confusion.
Better error handling when writing to FIFO.
Remove duplicate msg initialization.
Optimize away multiple uses of strlen for one check.
Signed-off-by: Christian Muck <christian.muck@bmw.de>
Diffstat (limited to 'src/lib/dlt_user.c')
| -rwxr-xr-x | src/lib/dlt_user.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/src/lib/dlt_user.c b/src/lib/dlt_user.c index 568ae86..89daac2 100755 --- a/src/lib/dlt_user.c +++ b/src/lib/dlt_user.c @@ -667,11 +667,12 @@ int dlt_register_context_ll_ts(DltContext *handle, const char *contextid, const if (description!=0) { - dlt_user.dlt_ll_ts[dlt_user.dlt_ll_ts_num_entries].context_description = malloc(strlen(description)+1); - strncpy(dlt_user.dlt_ll_ts[dlt_user.dlt_ll_ts_num_entries].context_description, description, strlen(description)); + int desc_lne = strlen(description); + dlt_user.dlt_ll_ts[dlt_user.dlt_ll_ts_num_entries].context_description = malloc(desc_len+1); + strncpy(dlt_user.dlt_ll_ts[dlt_user.dlt_ll_ts_num_entries].context_description, description, strlen(desc_len)); /* Terminate transmitted string with 0 */ - dlt_user.dlt_ll_ts[dlt_user.dlt_ll_ts_num_entries].context_description[strlen(description)]='\0'; + dlt_user.dlt_ll_ts[dlt_user.dlt_ll_ts_num_entries].context_description[desc_len]='\0'; } if (loglevel!=DLT_USER_LOG_LEVEL_NOT_SET) |