diff options
| author | Martin Pool <mbp@samba.org> | 2001-12-10 05:12:52 +0000 |
|---|---|---|
| committer | Martin Pool <mbp@samba.org> | 2001-12-10 05:12:52 +0000 |
| commit | 77d800fb3ea28586a2c1385cffbf5b4314284f14 (patch) | |
| tree | cedc7e6224163bd4114392f761fa69fd53508d5f /source3/tdb | |
| parent | 70f27f1b52b57011c053460b0084e366bfbf03f7 (diff) | |
| download | samba-77d800fb3ea28586a2c1385cffbf5b4314284f14.tar.gz samba-77d800fb3ea28586a2c1385cffbf5b4314284f14.tar.xz samba-77d800fb3ea28586a2c1385cffbf5b4314284f14.zip | |
tdb_open_ex should always "goto fail" in case of error, rather than
just returning. I don't think this would leak at the moment, but it's
an accident waiting to happen.
(This used to be commit fe901ec194d1b8ef94386b097e72508075eb8a54)
Diffstat (limited to 'source3/tdb')
| -rw-r--r-- | source3/tdb/tdb.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/source3/tdb/tdb.c b/source3/tdb/tdb.c index 955c0b322df..3b0946d5804 100644 --- a/source3/tdb/tdb.c +++ b/source3/tdb/tdb.c @@ -1463,11 +1463,12 @@ TDB_CONTEXT *tdb_open_ex(char *name, int hash_size, int tdb_flags, goto fail; /* Is it already in the open list? If so, fail. */ + if (tdb_already_open(st.st_dev, st.st_ino) for (i = tdbs; i; i = i->next) { if (i->device == st.st_dev && i->inode == st.st_ino) { errno = EBUSY; close(tdb->fd); - return NULL; + goto fail; } } |