summaryrefslogtreecommitdiff
path: root/Xtransutil.c
diff options
context:
space:
mode:
authorAlan Hourihane <alanh@fairlite.co.uk>2011-12-01 12:04:22 +0000
committerAlan Coopersmith <alan.coopersmith@oracle.com>2011-12-05 20:34:12 -0800
commitff992be7b05483f9a0586a27ee0edb6d99580b84 (patch)
tree2beea8fe03848e12f43066bb30a55669853a3606 /Xtransutil.c
parent29c9e101f7ad72556aa6b3dbc17d66a1ace9b2b1 (diff)
Fix some resource & memory leaks in libxtrans.
Signed-off-by: Alan Hourihane <alanh@vmware.com> Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com> Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Diffstat (limited to 'Xtransutil.c')
-rw-r--r--Xtransutil.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/Xtransutil.c b/Xtransutil.c
index 5dd2453..dd36cae 100644
--- a/Xtransutil.c
+++ b/Xtransutil.c
@@ -586,6 +586,7 @@ trans_mkdir(const char *path, int mode)
if (fstat(fd, &fbuf) == -1) {
prmsg(1, "mkdir: ERROR: fstat failed for %s (%d)\n",
path, errno);
+ close(fd);
return -1;
}
/*
@@ -597,6 +598,7 @@ trans_mkdir(const char *path, int mode)
buf.st_ino != fbuf.st_ino) {
prmsg(1, "mkdir: ERROR: inode for %s changed\n",
path);
+ close(fd);
return -1;
}
if (updateOwner && fchown(fd, 0, 0) == 0)