diff options
| author | Simon Brandner <simon.brandner@partner.bmw.de> | 2013-01-28 12:14:06 +0100 |
|---|---|---|
| committer | Alexander Wenzel <Alexander.AW.Wenzel@bmw.de> | 2013-07-19 16:54:38 +0200 |
| commit | e2943ff4cf2d7d2da4000e2ca35663a7c83675d7 (patch) | |
| tree | 1a710a70a5190dacdb26c02da6cb570271f5d506 /src/shared/dlt_offline_trace.c | |
| parent | 733a1111254882aec6ebc9c90bb690aaee3f2773 (diff) | |
| download | DLT-daemon-e2943ff4cf2d7d2da4000e2ca35663a7c83675d7.tar.gz | |
Semaphores and Pointer passing insteasd by value and otehr coverity issue fixes
Signed-off-by: Alexander Wenzel <Alexander.AW.Wenzel@bmw.de>
Diffstat (limited to 'src/shared/dlt_offline_trace.c')
| -rw-r--r-- | src/shared/dlt_offline_trace.c | 23 |
1 files changed, 15 insertions, 8 deletions
diff --git a/src/shared/dlt_offline_trace.c b/src/shared/dlt_offline_trace.c index 72b31ad..80c328a 100644 --- a/src/shared/dlt_offline_trace.c +++ b/src/shared/dlt_offline_trace.c @@ -101,8 +101,10 @@ unsigned long dlt_offline_trace_get_total_size(DltOfflineTrace *trace) { while ((dp=readdir(dir)) != NULL) { if(strstr(dp->d_name,".dlt")) { sprintf(filename,"%s/%s",trace->directory,dp->d_name); - stat(filename,&status); - size += status.st_size; + if ( 0 == stat(filename,&status) ) + size += status.st_size; + else + printf("Offline trace file %s cannot be stat-ed",filename); } } closedir(dir); @@ -127,12 +129,17 @@ int dlt_offline_trace_delete_oldest_file(DltOfflineTrace *trace) { while ((dp=readdir(dir)) != NULL) { if(strstr(dp->d_name,".dlt")) { sprintf(filename,"%s/%s",trace->directory,dp->d_name); - stat(filename,&status); - if(time_oldest == 0 || status.st_mtime < time_oldest) { - time_oldest = status.st_mtime; - size_oldest = status.st_size; - strcpy(filename_oldest,filename); - } + if (0 == stat(filename,&status)) + { + if(time_oldest == 0 || status.st_mtime < time_oldest) { + time_oldest = status.st_mtime; + size_oldest = status.st_size; + strcpy(filename_oldest,filename); + } + } + else + printf("Old offline trace file %s cannot be stat-ed",filename); + } } closedir(dir); |