diff options
| author | Kirill Smelkov <kirr@mns.spb.ru> | 2009-10-21 21:18:56 +0400 |
|---|---|---|
| committer | Rusty Russell <rusty@rustcorp.com.au> | 2009-10-29 10:14:33 +1030 |
| commit | b4424f8234a78a79fb2d71d46ca208b4f12e0f9e (patch) | |
| tree | 51b84db81ee6ea5d0fd0e185efb6aa1b7d2b9330 | |
| parent | cfed5f946de0992a594c189ee3c19cf98e59d380 (diff) | |
| download | samba-b4424f8234a78a79fb2d71d46ca208b4f12e0f9e.tar.gz samba-b4424f8234a78a79fb2d71d46ca208b4f12e0f9e.tar.bz2 samba-b4424f8234a78a79fb2d71d46ca208b4f12e0f9e.zip | |
tdb: reset tdb->fd to -1 in tdb_close()
So that erroneous double tdb_close() calls do not try to close() same
fd again. This is like SAFE_FREE() but for fd.
Signed-off-by: Kirill Smelkov <kirr@mns.spb.ru>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
| -rw-r--r-- | lib/tdb/common/open.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/lib/tdb/common/open.c b/lib/tdb/common/open.c index 1ba2e7bd11..64efafeb22 100644 --- a/lib/tdb/common/open.c +++ b/lib/tdb/common/open.c @@ -403,8 +403,10 @@ int tdb_close(struct tdb_context *tdb) tdb_munmap(tdb); } SAFE_FREE(tdb->name); - if (tdb->fd != -1) + if (tdb->fd != -1) { ret = close(tdb->fd); + tdb->fd = -1; + } SAFE_FREE(tdb->lockrecs); /* Remove from contexts list */ |