Add "return -1" after the error_function execution.
Signed-off-by: Yuan Sun <sunyu...@huawei.com>
---
.../controllers/libcontrollers/libcontrollers.c | 41 ++++++++++++++++------
.../controllers/libcontrollers/libcontrollers.h | 2 +-
2 files changed, 31 insertions(+), 12 deletions(-)
diff --git a/testcases/kernel/controllers/libcontrollers/libcontrollers.c
b/testcases/kernel/controllers/libcontrollers/libcontrollers.c
index b01e1b8..ebf9870 100644
--- a/testcases/kernel/controllers/libcontrollers/libcontrollers.c
+++ b/testcases/kernel/controllers/libcontrollers/libcontrollers.c
@@ -50,8 +50,10 @@ int scan_shares_files(unsigned int *shares_pointer)
/*
* Check if we can get stat of the file
*/
- if (lstat(fullpath, &statbuffer) < 0)
+ if (lstat(fullpath, &statbuffer) < 0) {
error_function("Can not read stat for file ", fullpath);
+ return -1;
+ }
if (S_ISDIR(statbuffer.st_mode) == 0) { /* not a directory */
/*
@@ -75,8 +77,10 @@ int scan_shares_files(unsigned int *shares_pointer)
*path_pointer++ = '/';
*path_pointer = 0;
- if ((dp = opendir(fullpath)) == NULL) /* Error in opening directory */
+ if ((dp = opendir(fullpath)) == NULL) { /* Error in opening directory */
error_function("Can't open ", fullpath);
+ return -1;
+ }
/*
* search all groups recursively and get total shares
*/
@@ -99,8 +103,10 @@ int scan_shares_files(unsigned int *shares_pointer)
path_pointer[-1] = 0;
- if (closedir(dp) < 0)
+ if (closedir(dp) < 0) {
error_function("Could not close dir ", fullpath);
+ return -1;
+ }
return 0;
}
@@ -115,27 +121,35 @@ int read_file(char *filepath, int action, unsigned int
*value)
{
int num_line = 0;
FILE *fp;
+ int tmp;
switch (action) {
case GET_SHARES:
- *value = read_shares_file(filepath);
- if (*value == -1)
+ tmp = read_shares_file(filepath);
+ if (tmp == -1)
return -1;
+ else
+ *value = (unsigned int)tmp;
break;
case GET_TASKS:
fp = fopen(filepath, "r");
- if (fp == NULL)
+ if (fp == NULL) {
error_function("Could not open file", filepath);
+ return -1;
+ }
while (fgets(target, LINE_MAX, fp) != NULL)
num_line++;
*value = (unsigned int)num_line;
- if (fclose(fp))
+ if (fclose(fp)) {
error_function("Could not close file", filepath);
+ return -1;
+ }
break;
default:
error_function("Wrong action type passed to fun read_file for ",
filepath);
+ return -1;
break;
}
return 0;
@@ -159,17 +173,20 @@ inline int error_function(char *msg1, char *msg2)
* the given pointer location. Returns 0 if success
*/
-unsigned
int read_shares_file(char *filepath)
{
FILE *fp;
unsigned int shares;
fp = fopen(filepath, "r");
- if (fp == NULL)
+ if (fp == NULL) {
error_function("Could not open file", filepath);
+ return -1;
+ }
fscanf(fp, "%u", &shares);
- if (fclose(fp))
+ if (fclose(fp)) {
error_function("Could not close file", filepath);
+ return -1;
+ }
return shares;
}
@@ -181,8 +198,10 @@ int write_to_file(char *file, const char *mode, unsigned
int value)
{
FILE *fp;
fp = fopen(file, mode);
- if (fp == NULL)
+ if (fp == NULL) {
error_function("in opening file for writing:", file);
+ return -1;
+ }
fprintf(fp, "%u\n", value);
fclose(fp);
return 0;
diff --git a/testcases/kernel/controllers/libcontrollers/libcontrollers.h
b/testcases/kernel/controllers/libcontrollers/libcontrollers.h
index 4001555..78a7fed 100644
--- a/testcases/kernel/controllers/libcontrollers/libcontrollers.h
+++ b/testcases/kernel/controllers/libcontrollers/libcontrollers.h
@@ -72,7 +72,7 @@ enum{
inline int error_function(char *msg1, char *msg2);
-unsigned int read_shares_file (char *filepath);
+int read_shares_file(char *filepath);
int read_file(char *filepath, int action, unsigned int *value);
--
1.9.1
------------------------------------------------------------------------------
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list
