[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

[tor-commits] [tor/master] Fix an fd leak in check_private_dir().



commit a17537a2386a98cba39c94dfb8b09b50edf16922
Author: Nick Mathewson <nickm@xxxxxxxxxxxxxx>
Date:   Tue Mar 22 08:29:51 2016 -0400

    Fix an fd leak in check_private_dir().
    
    The fd would leak when the User wasn't recogniezed by
    getpwnam(). Since we'd then go on to exit, this wasn't a terribad
    leak, but it's still not as nice as no leak at all.
    
    CID 1355640; bugfix on no released Tor.
---
 src/common/util.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/common/util.c b/src/common/util.c
index 39d6502..056817e 100644
--- a/src/common/util.c
+++ b/src/common/util.c
@@ -2157,6 +2157,7 @@ check_private_dir(const char *dirname, cpd_check_t check,
     if (pw == NULL) {
       log_warn(LD_CONFIG, "Error setting configured user: %s not found",
                effective_user);
+      close(fd);
       return -1;
     }
     running_uid = pw->pw_uid;

_______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits