]> granicus.if.org Git - sudo/commitdiff
Fix logic goof when checking open() status.
authorTodd C. Miller <Todd.Miller@courtesan.com>
Tue, 3 Sep 2013 17:24:31 +0000 (11:24 -0600)
committerTodd C. Miller <Todd.Miller@courtesan.com>
Tue, 3 Sep 2013 17:24:31 +0000 (11:24 -0600)
plugins/sudoers/iolog.c

index 57b2026b485ef4955d928273d29f040ed47bbe64..6dc8a26343aa4bd46b194df6cd2f4763b9010715 100644 (file)
@@ -487,7 +487,7 @@ write_info_log(char *pathbuf, size_t len, struct iolog_details *details,
     pathbuf[len] = '\0';
     strlcat(pathbuf, "/log", PATH_MAX);
     fd = open(pathbuf, O_CREAT|O_WRONLY, S_IRUSR|S_IWUSR);
-    if (fd != -1 || (fp = fdopen(fd, "w")) == NULL)
+    if (fd == -1 || (fp = fdopen(fd, "w")) == NULL)
        log_fatal(USE_ERRNO, N_("unable to create %s"), pathbuf);
 
     fprintf(fp, "%lld:%s:%s:%s:%s:%d:%d\n%s\n%s", (long long)now->tv_sec,