diff options
| author | Michael Adam <obnox@samba.org> | 2008-03-27 11:26:33 +0100 |
|---|---|---|
| committer | Michael Adam <obnox@samba.org> | 2008-03-27 11:46:39 +0100 |
| commit | 5c30841fe7699bfa58869af110c7766a94d1f760 (patch) | |
| tree | c86d2e3883d224297ba5ff469633d5c82dc6725a /source3/lib/replace/test/os2_delete.c | |
| parent | 5b3f28f1528160ac5ddcdf140371fa750590468c (diff) | |
| download | samba-5c30841fe7699bfa58869af110c7766a94d1f760.tar.gz samba-5c30841fe7699bfa58869af110c7766a94d1f760.tar.xz samba-5c30841fe7699bfa58869af110c7766a94d1f760.zip | |
libreplace: fix coverity ID 517 - untangle close from open in test/os2_delete.c
This is not a proper bug but the code is clearer now
and we are tracking failure of open separate from that of close.
Michael
(This used to be commit 451fc9ae05f841883081a334e179cf31625a772c)
Diffstat (limited to 'source3/lib/replace/test/os2_delete.c')
| -rw-r--r-- | source3/lib/replace/test/os2_delete.c | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/source3/lib/replace/test/os2_delete.c b/source3/lib/replace/test/os2_delete.c index c6ef1800178..b45c135355a 100644 --- a/source3/lib/replace/test/os2_delete.c +++ b/source3/lib/replace/test/os2_delete.c @@ -39,8 +39,15 @@ static void create_files(void) int i; for (i=0;i<NUM_FILES;i++) { char fname[40]; + int fd; sprintf(fname, TESTDIR "/test%u.txt", i); - close(open(fname, O_CREAT|O_RDWR, 0600)) == 0 || FAILED("close"); + fd = open(fname, O_CREAT|O_RDWR, 0600); + if (fd < 0) { + FAILED("open"); + } + if (close(fd) != 0) { + FAILED("close"); + } } } |