diff options
author | Andy Shevchenko <ext-andriy.shevchenko@nokia.com> | 2011-04-05 16:55:52 +0300 |
---|---|---|
committer | Artem Bityutskiy <Artem.Bityutskiy@nokia.com> | 2011-04-05 17:22:41 +0300 |
commit | f4e4f8b03353b049ca176642da138d22ea17f43f (patch) | |
tree | dcf65e7d6e2df6b458f862981c2828b141f10e1d /tests | |
parent | 4c2c6f48b15b69a013beb1faecb6b080d03baf16 (diff) |
tests: fs-tests: check return value of functions
chdir() returns negative value in case of error.
fscanf() returns amount of successfully parsed parameters.
Signed-off-by: Andy Shevchenko <ext-andriy.shevchenko@nokia.com>
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
Diffstat (limited to 'tests')
-rw-r--r-- | tests/fs-tests/lib/tests.c | 3 | ||||
-rw-r--r-- | tests/fs-tests/stress/atoms/pdfrun.c | 5 |
2 files changed, 6 insertions, 2 deletions
diff --git a/tests/fs-tests/lib/tests.c b/tests/fs-tests/lib/tests.c index 00b94f9..8a6a5f5 100644 --- a/tests/fs-tests/lib/tests.c +++ b/tests/fs-tests/lib/tests.c @@ -1236,7 +1236,8 @@ void tests_try_to_clear_dir(const char *dir_name) break; } } - chdir(buf); + if (chdir(buf) < 0) + perror("chdir"); closedir(dir); } diff --git a/tests/fs-tests/stress/atoms/pdfrun.c b/tests/fs-tests/stress/atoms/pdfrun.c index 3536580..46c70e1 100644 --- a/tests/fs-tests/stress/atoms/pdfrun.c +++ b/tests/fs-tests/stress/atoms/pdfrun.c @@ -40,8 +40,11 @@ void adjust_size(void) total_memory = 0; f = fopen("/proc/meminfo", "r"); - fscanf(f, "%s %lu", dummy, &total_memory); + if (fscanf(f, "%s %lu", dummy, &total_memory) != 2) + perror("fscanf error"); fclose(f); + + if (total_memory > 0 && tests_size_parameter > total_memory / 2) tests_size_parameter = total_memory / 2; } |