mirror of
https://github.com/samba-team/samba.git
synced 2025-01-11 05:18:09 +03:00
Don't modify the fsp in fd_close_posix() anymore.
Now that it is inside the vfs layer, this function should not alter the fsp (i.e. set fsp->fh->fd = -1) anymore. That belongs above the vfs layer. Michael
This commit is contained in:
parent
3cf56b124a
commit
df264bf3e0
@ -621,7 +621,6 @@ NTSTATUS fd_close_posix(struct files_struct *fsp)
|
||||
* just close.
|
||||
*/
|
||||
ret = close(fsp->fh->fd);
|
||||
fsp->fh->fd = -1;
|
||||
if (ret == -1) {
|
||||
return map_nt_error_from_unix(errno);
|
||||
}
|
||||
@ -636,7 +635,6 @@ NTSTATUS fd_close_posix(struct files_struct *fsp)
|
||||
*/
|
||||
|
||||
add_fd_to_close_entry(fsp);
|
||||
fsp->fh->fd = -1;
|
||||
return NT_STATUS_OK;
|
||||
}
|
||||
|
||||
@ -680,8 +678,6 @@ NTSTATUS fd_close_posix(struct files_struct *fsp)
|
||||
ret = -1;
|
||||
}
|
||||
|
||||
fsp->fh->fd = -1;
|
||||
|
||||
if (ret == -1) {
|
||||
return map_nt_error_from_unix(errno);
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user