diff options
| author | Alan Hourihane <alanh@fairlite.co.uk> | 2011-12-01 12:04:22 +0000 |
|---|---|---|
| committer | Alan Coopersmith <alan.coopersmith@oracle.com> | 2011-12-05 20:34:12 -0800 |
| commit | ff992be7b05483f9a0586a27ee0edb6d99580b84 (patch) | |
| tree | 2beea8fe03848e12f43066bb30a55669853a3606 /Xtransutil.c | |
| parent | 29c9e101f7ad72556aa6b3dbc17d66a1ace9b2b1 (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.c | 2 |
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) |