fsstress: getcwd() return value fixes
authorAri Sundholm <ari@tuxera.com>
Wed, 20 Dec 2017 17:00:11 +0000 (19:00 +0200)
committerEryu Guan <eguan@redhat.com>
Sun, 24 Dec 2017 13:30:58 +0000 (21:30 +0800)
Add a check for the return value of getcwd(). Fix another check
which mistakenly checks if the return value is less than zero
instead of checking whether the return value is NULL.

Signed-off-by: Ari Sundholm <ari@tuxera.com>
Reviewed-by: Eryu Guan <eguan@redhat.com>
Signed-off-by: Eryu Guan <eguan@redhat.com>
ltp/fsstress.c

index 13d5dd54b1e0f2a9379fe585234a4113f17f9429..028c232c11e8bb9b7e741a6f1634eea64f6f8f29 100644 (file)
@@ -495,7 +495,7 @@ int main(int argc, char **argv)
 
        (void)mkdir(dirname, 0777);
        if (logname && logname[0] != '/') {
-               if (getcwd(rpath, sizeof(rpath)) < 0){
+               if (!getcwd(rpath, sizeof(rpath))){
                        perror("getcwd failed");
                        exit(1);
                }
@@ -970,6 +970,10 @@ doproc(void)
        }
        top_ino = statbuf.st_ino;
        homedir = getcwd(NULL, 0);
+       if (!homedir) {
+               perror("getcwd failed");
+               _exit(1);
+       }
        seed += procid;
        srandom(seed);
        if (namerand)