diff options
| author | Lassi Marttala <lassi.lm.marttala@partner.bmw.de> | 2012-06-04 11:53:10 +0200 |
|---|---|---|
| committer | Christian Muck <christian.muck@bmw.de> | 2012-06-13 23:53:01 +0200 |
| commit | 54daad6cc95fb66662c894fabb5fd18308eec6e0 (patch) | |
| tree | 5c20a58a362133d3bdde143851209f17a458bbbc /src/system | |
| parent | 85f564f2b4c580bc151d05559e6ed0614919da8a (diff) | |
| download | DLT-daemon-54daad6cc95fb66662c894fabb5fd18308eec6e0.tar.gz | |
[GDLT-93]: Add -Wextra flags for compilation
Fixed all the warnings that popped up with the new flag.
Signed-off-by: Christian Muck <christian.muck@bmw.de>
Diffstat (limited to 'src/system')
| -rw-r--r-- | src/system/dlt-system-filetransfer.c | 6 | ||||
| -rw-r--r-- | src/system/dlt-system-logfile.c | 2 | ||||
| -rw-r--r-- | src/system/dlt-system-process-handling.c | 2 | ||||
| -rw-r--r-- | src/system/dlt-system-processes.c | 6 |
4 files changed, 8 insertions, 8 deletions
diff --git a/src/system/dlt-system-filetransfer.c b/src/system/dlt-system-filetransfer.c index 275818e..7eb6e92 100644 --- a/src/system/dlt-system-filetransfer.c +++ b/src/system/dlt-system-filetransfer.c @@ -102,7 +102,7 @@ char *compress_file(char *src, int level) FILE *src_file; dst_file = gzopen(dst, dst_mode); - if(dst_file < 0) + if(dst_file == Z_NULL) { free(dst); return NULL; @@ -110,7 +110,7 @@ char *compress_file(char *src, int level) src_file = fopen(src, "r"); - if(src_file < 0) + if(src_file == NULL) { gzclose(dst_file); free(dst); @@ -217,7 +217,7 @@ int flush_dir(FiletransferOptions opts, int which) DIR *dir; dir = opendir(sdir); - if(dir > 0) + if(dir != NULL) { while((dp = readdir(dir)) != NULL) { diff --git a/src/system/dlt-system-logfile.c b/src/system/dlt-system-logfile.c index 63c81b5..50964f2 100644 --- a/src/system/dlt-system-logfile.c +++ b/src/system/dlt-system-logfile.c @@ -68,7 +68,7 @@ void send_file(LogFileOptions fileopt, int n) pFile = fopen(fileopt.Filename[n],"r"); - if(pFile>0) + if(pFile != NULL) { while (!feof(pFile)) { bytes = fread(buffer,1,sizeof(buffer)-1,pFile); diff --git a/src/system/dlt-system-process-handling.c b/src/system/dlt-system-process-handling.c index 2fd07fc..3ad7908 100644 --- a/src/system/dlt-system-process-handling.c +++ b/src/system/dlt-system-process-handling.c @@ -69,7 +69,7 @@ int daemonize() exit(0); // Create a new process group - if(setsid < 0) + if(setsid() < 0) return -1; /** diff --git a/src/system/dlt-system-processes.c b/src/system/dlt-system-processes.c index fc46c97..ee37bfc 100644 --- a/src/system/dlt-system-processes.c +++ b/src/system/dlt-system-processes.c @@ -76,7 +76,7 @@ void send_process(LogProcessOptions popts, int n) /* go through all process files in directory */ DIR *dir = opendir("/proc"); - if(dir>0) + if(dir != NULL) { while ((dp=readdir(dir)) != NULL) { @@ -85,7 +85,7 @@ void send_process(LogProcessOptions popts, int n) buffer[0] = 0; sprintf(filename, "/proc/%s/cmdline",dp->d_name); pFile = fopen(filename, "r"); - if(pFile>0) + if(pFile != NULL) { bytes = fread(buffer, 1, sizeof(buffer)-1, pFile); fclose(pFile); @@ -96,7 +96,7 @@ void send_process(LogProcessOptions popts, int n) found = 1; sprintf(filename, "/proc/%s/%s", dp->d_name,popts.Filename[n]); pFile = fopen(filename, "r"); - if(pFile>0) + if(pFile != NULL) { bytes = fread(buffer, 1, sizeof(buffer)-1, pFile); fclose(pFile); |