mirror of
https://github.com/samba-team/samba.git
synced 2024-12-27 03:21:53 +03:00
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 commit is contained in:
parent
bb6c0df4bc
commit
fe901ec194
@ -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;
|
||||
}
|
||||
}
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user