1996-05-04 11:50:46 +04:00
/*
2002-01-30 09:08:46 +03:00
Unix SMB / CIFS implementation .
1996-05-04 11:50:46 +04:00
Locking functions
2000-04-28 01:12:33 +04:00
Copyright ( C ) Andrew Tridgell 1992 - 2000
Copyright ( C ) Jeremy Allison 1992 - 2000
1996-05-04 11:50:46 +04:00
This program is free software ; you can redistribute it and / or modify
it under the terms of the GNU General Public License as published by
the Free Software Foundation ; either version 2 of the License , or
( at your option ) any later version .
This program is distributed in the hope that it will be useful ,
but WITHOUT ANY WARRANTY ; without even the implied warranty of
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE . See the
GNU General Public License for more details .
You should have received a copy of the GNU General Public License
along with this program ; if not , write to the Free Software
Foundation , Inc . , 675 Mass Ave , Cambridge , MA 0213 9 , USA .
1996-08-15 19:11:34 +04:00
Revision History :
12 aug 96 : Erik . Devriendt @ te6 . siemens . be
added support for shared memory implementation of share mode locking
1997-05-20 04:32:51 +04:00
May 1997. Jeremy Allison ( jallison @ whistle . com ) . Modified share mode
locking to deal with multiple share modes per open file .
1997-10-07 22:46:19 +04:00
September 1997. Jeremy Allison ( jallison @ whistle . com ) . Added oplock
support .
1999-12-21 12:25:59 +03:00
rewrtten completely to use new tdb code . Tridge , Dec ' 99
2000-04-28 01:12:33 +04:00
Added POSIX locking support . Jeremy Allison ( jeremy @ valinux . com ) , Apr . 2000.
1996-05-04 11:50:46 +04:00
*/
# include "includes.h"
2001-07-02 06:42:41 +04:00
uint16 global_smbpid ;
1996-05-04 11:50:46 +04:00
1999-12-21 12:25:59 +03:00
/* the locking database handle */
static TDB_CONTEXT * tdb ;
1997-10-20 12:46:00 +04:00
2000-04-12 21:59:09 +04:00
/****************************************************************************
Debugging aid : - ) .
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
static const char * lock_type_name ( enum brl_type lock_type )
{
return ( lock_type = = READ_LOCK ) ? " READ " : " WRITE " ;
}
1996-05-04 11:50:46 +04:00
/****************************************************************************
1998-07-23 04:10:26 +04:00
Utility function called to see if a file region is locked .
2001-06-30 05:59:48 +04:00
If check_self is True , then checks on our own fd with the same locking context
are still made . If check_self is False , then checks are not made on our own fd
with the same locking context are not made .
1996-05-04 11:50:46 +04:00
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
2000-04-12 21:59:09 +04:00
1998-08-15 11:27:34 +04:00
BOOL is_locked ( files_struct * fsp , connection_struct * conn ,
2000-04-11 23:44:54 +04:00
SMB_BIG_UINT count , SMB_BIG_UINT offset ,
2001-06-30 05:59:48 +04:00
enum brl_type lock_type , BOOL check_self )
1996-05-04 11:50:46 +04:00
{
1998-08-14 21:38:29 +04:00
int snum = SNUM ( conn ) ;
2000-04-12 21:59:09 +04:00
BOOL ret ;
1999-12-21 12:25:59 +03:00
1998-08-14 21:38:29 +04:00
if ( count = = 0 )
return ( False ) ;
1996-05-04 11:50:46 +04:00
1998-08-14 21:38:29 +04:00
if ( ! lp_locking ( snum ) | | ! lp_strict_locking ( snum ) )
return ( False ) ;
1998-07-28 22:26:47 +04:00
2000-05-06 00:50:22 +04:00
ret = ! brl_locktest ( fsp - > dev , fsp - > inode , fsp - > fnum ,
2000-05-02 06:23:41 +04:00
global_smbpid , sys_getpid ( ) , conn - > cnum ,
2001-06-30 05:59:48 +04:00
offset , count , lock_type , check_self ) ;
DEBUG ( 10 , ( " is_locked: brl start=%.0f len=%.0f %s for file %s \n " ,
( double ) offset , ( double ) count , ret ? " locked " : " unlocked " ,
fsp - > fsp_name ) ) ;
1996-05-04 11:50:46 +04:00
2000-04-12 21:59:09 +04:00
/*
* There is no lock held by an SMB daemon , check to
* see if there is a POSIX lock from a UNIX or NFS process .
*/
2001-06-30 05:59:48 +04:00
if ( ! ret & & lp_posix_locking ( snum ) ) {
2000-04-13 03:01:11 +04:00
ret = is_posix_locked ( fsp , offset , count , lock_type ) ;
2000-04-12 21:59:09 +04:00
2001-06-30 05:59:48 +04:00
DEBUG ( 10 , ( " is_locked: posix start=%.0f len=%.0f %s for file %s \n " ,
( double ) offset , ( double ) count , ret ? " locked " : " unlocked " ,
fsp - > fsp_name ) ) ;
}
2000-04-12 21:59:09 +04:00
return ret ;
}
1996-05-04 11:50:46 +04:00
/****************************************************************************
1998-07-23 04:10:26 +04:00
Utility function called by locking requests .
1996-05-04 11:50:46 +04:00
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
2000-04-12 21:59:09 +04:00
2002-03-13 23:28:19 +03:00
static NTSTATUS do_lock ( files_struct * fsp , connection_struct * conn , uint16 lock_pid ,
2003-10-09 05:46:01 +04:00
SMB_BIG_UINT count , SMB_BIG_UINT offset , enum brl_type lock_type , BOOL * my_lock_ctx )
1996-05-04 11:50:46 +04:00
{
2003-01-03 21:50:13 +03:00
NTSTATUS status = NT_STATUS_LOCK_NOT_GRANTED ;
1999-12-21 12:25:59 +03:00
if ( ! lp_locking ( SNUM ( conn ) ) )
2001-09-04 11:13:01 +04:00
return NT_STATUS_OK ;
1999-12-21 12:25:59 +03:00
2001-08-24 08:55:28 +04:00
/* NOTE! 0 byte long ranges ARE allowed and should be stored */
2000-04-12 21:59:09 +04:00
DEBUG ( 10 , ( " do_lock: lock type %s start=%.0f len=%.0f requested for file %s \n " ,
lock_type_name ( lock_type ) , ( double ) offset , ( double ) count , fsp - > fsp_name ) ) ;
1999-12-21 12:25:59 +03:00
2000-01-13 15:09:36 +03:00
if ( OPEN_FSP ( fsp ) & & fsp - > can_lock & & ( fsp - > conn = = conn ) ) {
2001-09-06 05:19:35 +04:00
status = brl_lock ( fsp - > dev , fsp - > inode , fsp - > fnum ,
lock_pid , sys_getpid ( ) , conn - > cnum ,
offset , count ,
2003-10-09 05:46:01 +04:00
lock_type , my_lock_ctx ) ;
2000-04-12 21:59:09 +04:00
2001-09-06 05:19:35 +04:00
if ( NT_STATUS_IS_OK ( status ) & & lp_posix_locking ( SNUM ( conn ) ) ) {
2000-04-12 21:59:09 +04:00
/*
* Try and get a POSIX lock on this range .
2000-04-19 00:41:04 +04:00
* Note that this is ok if it is a read lock
* overlapping on a different fd . JRA .
2000-04-12 21:59:09 +04:00
*/
2001-09-06 05:19:35 +04:00
if ( ! set_posix_lock ( fsp , offset , count , lock_type ) ) {
2003-08-01 10:10:30 +04:00
if ( errno = = EACCES | | errno = = EAGAIN )
status = NT_STATUS_FILE_LOCK_CONFLICT ;
else
status = map_nt_error_from_unix ( errno ) ;
2000-04-12 21:59:09 +04:00
/*
* We failed to map - we must now remove the brl
* lock entry .
*/
( void ) brl_unlock ( fsp - > dev , fsp - > inode , fsp - > fnum ,
2001-07-02 06:42:41 +04:00
lock_pid , sys_getpid ( ) , conn - > cnum ,
2003-04-05 00:38:12 +04:00
offset , count , False ,
NULL , NULL ) ;
2000-04-12 21:59:09 +04:00
}
}
2000-01-13 15:09:36 +03:00
}
1999-12-21 12:25:59 +03:00
2001-09-06 05:19:35 +04:00
return status ;
1996-05-04 11:50:46 +04:00
}
2002-03-13 23:28:19 +03:00
/****************************************************************************
2002-09-25 19:19:00 +04:00
Utility function called by locking requests . This is * DISGUSTING * . It also
2002-03-13 23:28:19 +03:00
appears to be " What Windows Does " ( tm ) . Andrew , ever wonder why Windows 2000
is so slow on the locking tests . . . . . . ? This is the reason . Much though I hate
it , we need this . JRA .
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
NTSTATUS do_lock_spin ( files_struct * fsp , connection_struct * conn , uint16 lock_pid ,
2003-10-09 05:46:01 +04:00
SMB_BIG_UINT count , SMB_BIG_UINT offset , enum brl_type lock_type , BOOL * my_lock_ctx )
2002-03-13 23:28:19 +03:00
{
int j , maxj = lp_lock_spin_count ( ) ;
int sleeptime = lp_lock_sleep_time ( ) ;
2002-03-15 12:51:37 +03:00
NTSTATUS status , ret ;
2002-03-13 23:28:19 +03:00
2002-03-13 23:49:57 +03:00
if ( maxj < = 0 )
maxj = 1 ;
2002-03-15 12:51:37 +03:00
ret = NT_STATUS_OK ; /* to keep dumb compilers happy */
2002-03-13 23:28:19 +03:00
for ( j = 0 ; j < maxj ; j + + ) {
2003-10-09 05:46:01 +04:00
status = do_lock ( fsp , conn , lock_pid , count , offset , lock_type , my_lock_ctx ) ;
2002-03-13 23:28:19 +03:00
if ( ! NT_STATUS_EQUAL ( status , NT_STATUS_LOCK_NOT_GRANTED ) & &
2002-03-15 12:51:37 +03:00
! NT_STATUS_EQUAL ( status , NT_STATUS_FILE_LOCK_CONFLICT ) ) {
return status ;
}
/* if we do fail then return the first error code we got */
if ( j = = 0 ) {
ret = status ;
2003-10-09 05:46:01 +04:00
/* Don't spin if we blocked ourselves. */
if ( * my_lock_ctx )
return ret ;
2002-03-15 12:51:37 +03:00
}
2002-03-13 23:49:57 +03:00
if ( sleeptime )
2002-03-14 04:05:34 +03:00
sys_usleep ( sleeptime ) ;
2002-03-13 23:28:19 +03:00
}
2002-03-15 12:51:37 +03:00
return ret ;
2002-03-13 23:28:19 +03:00
}
2003-04-05 00:38:12 +04:00
/* Struct passed to brl_unlock. */
struct posix_unlock_data_struct {
files_struct * fsp ;
SMB_BIG_UINT offset ;
SMB_BIG_UINT count ;
} ;
/****************************************************************************
Function passed to brl_unlock to allow POSIX unlock to be done first .
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
static void posix_unlock ( void * pre_data )
{
struct posix_unlock_data_struct * pdata = ( struct posix_unlock_data_struct * ) pre_data ;
if ( lp_posix_locking ( SNUM ( pdata - > fsp - > conn ) ) )
release_posix_lock ( pdata - > fsp , pdata - > offset , pdata - > count ) ;
}
1996-05-04 11:50:46 +04:00
/****************************************************************************
1998-07-23 04:10:26 +04:00
Utility function called by unlocking requests .
1996-05-04 11:50:46 +04:00
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
2000-04-13 03:01:11 +04:00
2001-08-27 12:19:43 +04:00
NTSTATUS do_unlock ( files_struct * fsp , connection_struct * conn , uint16 lock_pid ,
SMB_BIG_UINT count , SMB_BIG_UINT offset )
1996-05-04 11:50:46 +04:00
{
1999-12-21 12:25:59 +03:00
BOOL ok = False ;
2003-04-05 00:38:12 +04:00
struct posix_unlock_data_struct posix_data ;
1999-12-21 12:25:59 +03:00
if ( ! lp_locking ( SNUM ( conn ) ) )
2001-09-04 11:13:01 +04:00
return NT_STATUS_OK ;
1999-12-21 12:25:59 +03:00
2000-04-26 00:30:58 +04:00
if ( ! OPEN_FSP ( fsp ) | | ! fsp - > can_lock | | ( fsp - > conn ! = conn ) ) {
2001-09-04 11:13:01 +04:00
return NT_STATUS_INVALID_HANDLE ;
2000-04-26 00:30:58 +04:00
}
1999-12-21 12:25:59 +03:00
DEBUG ( 10 , ( " do_unlock: unlock start=%.0f len=%.0f requested for file %s \n " ,
( double ) offset , ( double ) count , fsp - > fsp_name ) ) ;
2000-04-19 00:41:04 +04:00
2000-04-26 00:30:58 +04:00
/*
* Remove the existing lock record from the tdb lockdb
* before looking at POSIX locks . If this record doesn ' t
* match then don ' t bother looking to remove POSIX locks .
*/
2000-04-12 21:59:09 +04:00
2003-04-05 00:38:12 +04:00
posix_data . fsp = fsp ;
posix_data . offset = offset ;
posix_data . count = count ;
2000-04-26 00:30:58 +04:00
ok = brl_unlock ( fsp - > dev , fsp - > inode , fsp - > fnum ,
2003-04-05 00:38:12 +04:00
lock_pid , sys_getpid ( ) , conn - > cnum , offset , count ,
False , posix_unlock , ( void * ) & posix_data ) ;
1996-05-04 11:50:46 +04:00
1999-12-21 12:25:59 +03:00
if ( ! ok ) {
2000-04-28 02:23:04 +04:00
DEBUG ( 10 , ( " do_unlock: returning ERRlock. \n " ) ) ;
2002-03-11 02:17:07 +03:00
return NT_STATUS_RANGE_NOT_LOCKED ;
1999-12-21 12:25:59 +03:00
}
2001-09-04 11:13:01 +04:00
return NT_STATUS_OK ;
1996-05-04 11:50:46 +04:00
}
2000-04-26 00:30:58 +04:00
/****************************************************************************
Remove any locks on this fd . Called from file_close ( ) .
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
void locking_close_file ( files_struct * fsp )
{
2000-05-02 06:23:41 +04:00
pid_t pid = sys_getpid ( ) ;
2000-04-26 00:30:58 +04:00
if ( ! lp_locking ( SNUM ( fsp - > conn ) ) )
return ;
2000-04-29 00:54:23 +04:00
/*
* Just release all the brl locks , no need to release individually .
*/
2000-04-26 00:30:58 +04:00
2000-04-29 00:54:23 +04:00
brl_close ( fsp - > dev , fsp - > inode , pid , fsp - > conn - > cnum , fsp - > fnum ) ;
2000-04-26 00:30:58 +04:00
2000-04-29 00:54:23 +04:00
if ( lp_posix_locking ( SNUM ( fsp - > conn ) ) ) {
2000-04-26 00:30:58 +04:00
2000-04-29 00:54:23 +04:00
/*
* Release all the POSIX locks .
2000-04-26 00:30:58 +04:00
*/
2000-04-29 00:54:23 +04:00
posix_locking_close_file ( fsp ) ;
2000-04-26 00:30:58 +04:00
}
}
1997-10-20 12:46:00 +04:00
/****************************************************************************
1998-07-23 04:10:26 +04:00
Initialise the locking functions .
1997-10-20 12:46:00 +04:00
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
2001-05-23 00:35:48 +04:00
static int open_read_only ;
1997-10-27 17:27:17 +03:00
BOOL locking_init ( int read_only )
1996-08-15 19:11:34 +04:00
{
2000-01-16 14:14:44 +03:00
brl_init ( read_only ) ;
2000-01-13 15:09:36 +03:00
2001-05-23 00:35:48 +04:00
if ( tdb )
return True ;
1997-10-29 05:18:08 +03:00
2004-02-12 08:24:02 +03:00
tdb = tdb_open_ex ( lock_path ( " locking.tdb " ) ,
2001-09-07 02:08:19 +04:00
0 , TDB_DEFAULT | ( read_only ? 0x0 : TDB_CLEAR_IF_FIRST ) ,
1999-12-21 12:25:59 +03:00
read_only ? O_RDONLY : O_RDWR | O_CREAT ,
2004-02-12 08:24:02 +03:00
0644 , smbd_tdb_log ) ;
1998-07-29 07:08:05 +04:00
1999-12-21 12:25:59 +03:00
if ( ! tdb ) {
2000-09-11 11:02:43 +04:00
DEBUG ( 0 , ( " ERROR: Failed to initialise locking database \n " ) ) ;
1999-12-13 16:27:58 +03:00
return False ;
}
1997-10-20 12:46:00 +04:00
2000-06-13 19:46:46 +04:00
if ( ! posix_locking_init ( read_only ) )
2000-04-29 00:54:23 +04:00
return False ;
2001-05-23 00:35:48 +04:00
open_read_only = read_only ;
1999-12-13 16:27:58 +03:00
return True ;
1996-05-04 11:50:46 +04:00
}
/*******************************************************************
1998-07-23 04:10:26 +04:00
Deinitialize the share_mode management .
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
2001-05-23 00:35:48 +04:00
1997-10-20 12:46:00 +04:00
BOOL locking_end ( void )
1996-05-04 11:50:46 +04:00
{
2001-05-23 00:35:48 +04:00
brl_shutdown ( open_read_only ) ;
if ( tdb ) {
if ( tdb_close ( tdb ) ! = 0 )
return False ;
}
1997-10-29 05:18:08 +03:00
return True ;
1996-05-04 11:50:46 +04:00
}
1999-12-21 12:25:59 +03:00
/*******************************************************************
2001-08-21 05:25:45 +04:00
Form a static locking key for a dev / inode pair .
1999-12-21 12:25:59 +03:00
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
2001-08-21 05:25:45 +04:00
1999-12-21 12:25:59 +03:00
static TDB_DATA locking_key ( SMB_DEV_T dev , SMB_INO_T inode )
{
static struct locking_key key ;
TDB_DATA kbuf ;
2000-05-10 04:05:27 +04:00
memset ( & key , ' \0 ' , sizeof ( key ) ) ;
1999-12-21 12:25:59 +03:00
key . dev = dev ;
key . inode = inode ;
kbuf . dptr = ( char * ) & key ;
kbuf . dsize = sizeof ( key ) ;
return kbuf ;
}
2001-08-21 05:25:45 +04:00
1999-12-21 12:25:59 +03:00
static TDB_DATA locking_key_fsp ( files_struct * fsp )
{
2000-04-10 16:58:14 +04:00
return locking_key ( fsp - > dev , fsp - > inode ) ;
1999-12-21 12:25:59 +03:00
}
1996-05-04 11:50:46 +04:00
1997-05-20 04:32:51 +04:00
/*******************************************************************
1998-07-23 04:10:26 +04:00
Lock a hash bucket entry .
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
2001-08-22 05:01:37 +04:00
1998-08-14 21:38:29 +04:00
BOOL lock_share_entry ( connection_struct * conn ,
1999-12-21 12:25:59 +03:00
SMB_DEV_T dev , SMB_INO_T inode )
1997-10-18 03:08:07 +04:00
{
2000-12-06 03:05:15 +03:00
return tdb_chainlock ( tdb , locking_key ( dev , inode ) ) = = 0 ;
1997-10-18 03:08:07 +04:00
}
1997-05-20 04:32:51 +04:00
/*******************************************************************
1998-07-23 04:10:26 +04:00
Unlock a hash bucket entry .
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
2001-08-22 05:01:37 +04:00
2000-12-06 05:51:13 +03:00
void unlock_share_entry ( connection_struct * conn ,
1999-12-21 12:25:59 +03:00
SMB_DEV_T dev , SMB_INO_T inode )
1997-05-20 04:32:51 +04:00
{
2000-12-06 05:51:13 +03:00
tdb_chainunlock ( tdb , locking_key ( dev , inode ) ) ;
1996-05-04 11:50:46 +04:00
}
2000-04-10 16:58:14 +04:00
/*******************************************************************
Lock a hash bucket entry . use a fsp for convenience
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
2001-08-21 05:25:45 +04:00
2000-04-10 16:58:14 +04:00
BOOL lock_share_entry_fsp ( files_struct * fsp )
{
2000-12-06 03:05:15 +03:00
return tdb_chainlock ( tdb , locking_key ( fsp - > dev , fsp - > inode ) ) = = 0 ;
2000-04-10 16:58:14 +04:00
}
/*******************************************************************
Unlock a hash bucket entry .
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
2001-08-21 05:25:45 +04:00
2000-12-06 05:51:13 +03:00
void unlock_share_entry_fsp ( files_struct * fsp )
2000-04-10 16:58:14 +04:00
{
2000-12-06 05:51:13 +03:00
tdb_chainunlock ( tdb , locking_key ( fsp - > dev , fsp - > inode ) ) ;
2000-04-10 16:58:14 +04:00
}
2002-01-08 03:52:36 +03:00
/*******************************************************************
Print out a share mode .
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
2003-05-22 22:56:51 +04:00
char * share_mode_str ( int num , share_mode_entry * e )
2002-01-08 03:52:36 +03:00
{
static pstring share_str ;
slprintf ( share_str , sizeof ( share_str ) - 1 , " share_mode_entry[%d]: \
2003-07-22 08:31:20 +04:00
pid = % lu , share_mode = 0 x % x , desired_access = 0 x % x , port = 0 x % x , type = 0 x % x , file_id = % lu , dev = 0 x % x , inode = % .0f " ,
num , ( unsigned long ) e - > pid , e - > share_mode , ( unsigned int ) e - > desired_access , e - > op_port , e - > op_type , e - > share_file_id ,
2002-01-08 03:52:36 +03:00
( unsigned int ) e - > dev , ( double ) e - > inode ) ;
return share_str ;
}
/*******************************************************************
Print out a share mode table .
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
static void print_share_mode_table ( struct locking_data * data )
{
int num_share_modes = data - > u . num_share_mode_entries ;
share_mode_entry * shares = ( share_mode_entry * ) ( data + 1 ) ;
int i ;
for ( i = 0 ; i < num_share_modes ; i + + ) {
share_mode_entry * entry_p = & shares [ i ] ;
DEBUG ( 10 , ( " print_share_mode_table: %s \n " , share_mode_str ( i , entry_p ) ) ) ;
}
}
1996-05-04 11:50:46 +04:00
/*******************************************************************
1998-07-23 04:10:26 +04:00
Get all share mode entries for a dev / inode pair .
1997-10-20 12:46:00 +04:00
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
2001-08-21 05:25:45 +04:00
1998-08-14 21:38:29 +04:00
int get_share_modes ( connection_struct * conn ,
1999-12-21 12:25:59 +03:00
SMB_DEV_T dev , SMB_INO_T inode ,
2001-11-30 04:59:03 +03:00
share_mode_entry * * pp_shares )
1997-05-20 04:32:51 +04:00
{
1999-12-21 12:25:59 +03:00
TDB_DATA dbuf ;
struct locking_data * data ;
2001-11-30 04:59:03 +03:00
int num_share_modes ;
share_mode_entry * shares = NULL ;
2003-09-09 00:42:33 +04:00
TDB_DATA key = locking_key ( dev , inode ) ;
2001-11-30 04:59:03 +03:00
* pp_shares = NULL ;
1999-12-22 03:08:25 +03:00
2003-09-09 00:42:33 +04:00
dbuf = tdb_fetch ( tdb , key ) ;
2001-08-21 05:25:45 +04:00
if ( ! dbuf . dptr )
return 0 ;
1999-12-21 12:25:59 +03:00
data = ( struct locking_data * ) dbuf . dptr ;
2001-11-30 04:59:03 +03:00
num_share_modes = data - > u . num_share_mode_entries ;
if ( num_share_modes ) {
int i ;
int del_count = 0 ;
1999-12-21 12:25:59 +03:00
2001-11-30 04:59:03 +03:00
shares = ( share_mode_entry * ) memdup ( dbuf . dptr + sizeof ( * data ) ,
2002-01-08 03:52:36 +03:00
num_share_modes * sizeof ( share_mode_entry ) ) ;
1999-12-21 12:25:59 +03:00
2001-11-30 04:59:03 +03:00
if ( ! shares ) {
SAFE_FREE ( dbuf . dptr ) ;
return 0 ;
}
/*
* Ensure that each entry has a real process attached .
*/
for ( i = 0 ; i < num_share_modes ; ) {
share_mode_entry * entry_p = & shares [ i ] ;
2002-01-08 03:52:36 +03:00
if ( process_exists ( entry_p - > pid ) ) {
DEBUG ( 10 , ( " get_share_modes: %s \n " , share_mode_str ( i , entry_p ) ) ) ;
2001-11-30 04:59:03 +03:00
i + + ;
2002-01-08 03:52:36 +03:00
} else {
DEBUG ( 10 , ( " get_share_modes: deleted %s \n " , share_mode_str ( i , entry_p ) ) ) ;
2004-06-03 03:19:36 +04:00
if ( num_share_modes - i - 1 > 0 ) {
memcpy ( & shares [ i ] , & shares [ i + 1 ] ,
sizeof ( share_mode_entry ) * ( num_share_modes - i - 1 ) ) ;
}
2001-11-30 04:59:03 +03:00
num_share_modes - - ;
del_count + + ;
}
}
/* Did we delete any ? If so, re-store in tdb. */
if ( del_count ) {
data - > u . num_share_mode_entries = num_share_modes ;
if ( num_share_modes )
memcpy ( dbuf . dptr + sizeof ( * data ) , shares ,
num_share_modes * sizeof ( share_mode_entry ) ) ;
/* The record has shrunk a bit */
dbuf . dsize - = del_count * sizeof ( share_mode_entry ) ;
2003-09-09 00:42:33 +04:00
if ( tdb_store ( tdb , key , dbuf , TDB_REPLACE ) = = - 1 ) {
2001-11-30 04:59:03 +03:00
SAFE_FREE ( shares ) ;
SAFE_FREE ( dbuf . dptr ) ;
return 0 ;
}
}
}
SAFE_FREE ( dbuf . dptr ) ;
* pp_shares = shares ;
return num_share_modes ;
1999-12-21 12:25:59 +03:00
}
/*******************************************************************
2001-08-21 05:25:45 +04:00
Fill a share mode entry .
1999-12-21 12:25:59 +03:00
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
This is a big, rather ugly patch. Whilst investigating the files not truncated
when copying to a full disk problem, I discovered that we were not allowing
the delete on close flag to be set properly, this led to other things, and
after investigation of the proper delete on close semantics and their relationship
to the file_share_delete flag I discovered there were some cases where we
weren't doing the deny modes properly. And this after only 5 years working
on them..... :-) :-).
So here's the latest attempt. I realised the delete on close flag needs to
be set across all smbds with a dev/ino pair open - in addition, the delete
on close flag, allow share delete and delete access requested all need to
be stored in the share mode tdb.
The "delete_on_close" entry in the fsp struct is now redundant and should
really be removed. This may also mean we can get rid of the "iterate_fsp"
calls that I didn't like adding in the first place. Whilst doing this patch,
I also discovered we needed to do the se_map_generic() call for file opens
and POSIX ACL mapping, so I added that also.
This code, although ugly, now passes the deny mode torture tests plus the
delete on close tests I added. I do need to add one more multiple connection
delete on close test to make sure I got the semantics exactly right, plus we
should also (as Andrew suggested) move to random testing here.
The good news is that NT should now correctly delete the file on disk
full error when copying to a disk :-).
Jeremy.
(This used to be commit 51987684bd231c744da2e5f3705fd236d5616173)
2001-03-30 12:57:24 +04:00
2001-08-21 05:25:45 +04:00
static void fill_share_mode ( char * p , files_struct * fsp , uint16 port , uint16 op_type )
{
share_mode_entry * e = ( share_mode_entry * ) p ;
2001-09-18 10:41:29 +04:00
void * x = & e - > time ; /* Needed to force alignment. p may not be aligned.... */
2001-08-21 05:25:45 +04:00
memset ( e , ' \0 ' , sizeof ( share_mode_entry ) ) ;
e - > pid = sys_getpid ( ) ;
e - > share_mode = fsp - > share_mode ;
2002-03-23 05:57:44 +03:00
e - > desired_access = fsp - > desired_access ;
2001-08-21 05:25:45 +04:00
e - > op_port = port ;
e - > op_type = op_type ;
2001-09-18 10:41:29 +04:00
memcpy ( x , & fsp - > open_time , sizeof ( struct timeval ) ) ;
2001-10-21 01:59:34 +04:00
e - > share_file_id = fsp - > file_id ;
e - > dev = fsp - > dev ;
e - > inode = fsp - > inode ;
2001-08-21 05:25:45 +04:00
}
/*******************************************************************
Check if two share mode entries are identical , ignoring oplock
2002-03-23 05:57:44 +03:00
and port info and desired_access .
2001-08-21 05:25:45 +04:00
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
2001-08-22 04:29:40 +04:00
BOOL share_modes_identical ( share_mode_entry * e1 , share_mode_entry * e2 )
2001-08-21 05:25:45 +04:00
{
2001-10-21 01:59:34 +04:00
# if 1 /* JRA PARANOIA TEST - REMOVE LATER */
if ( e1 - > pid = = e2 - > pid & &
e1 - > share_file_id = = e2 - > share_file_id & &
e1 - > dev = = e2 - > dev & &
e1 - > inode = = e2 - > inode & &
( e1 - > share_mode & ~ DELETE_ON_CLOSE_FLAG ) ! = ( e2 - > share_mode & ~ DELETE_ON_CLOSE_FLAG ) ) {
DEBUG ( 0 , ( " PANIC: share_modes_identical: share_mode missmatch (e1 = %u, e2 = %u). Logic error. \n " ,
( unsigned int ) ( e1 - > share_mode & ~ DELETE_ON_CLOSE_FLAG ) ,
( unsigned int ) ( e2 - > share_mode & ~ DELETE_ON_CLOSE_FLAG ) ) ) ;
smb_panic ( " PANIC: share_modes_identical logic error. \n " ) ;
}
# endif
2001-08-21 05:25:45 +04:00
return ( e1 - > pid = = e2 - > pid & &
2001-09-06 01:39:43 +04:00
( e1 - > share_mode & ~ DELETE_ON_CLOSE_FLAG ) = = ( e2 - > share_mode & ~ DELETE_ON_CLOSE_FLAG ) & &
2001-10-21 01:59:34 +04:00
e1 - > dev = = e2 - > dev & &
e1 - > inode = = e2 - > inode & &
e1 - > share_file_id = = e2 - > share_file_id ) ;
2001-08-21 05:25:45 +04:00
}
/*******************************************************************
Delete a specific share mode . Return the number
of entries left , and a memdup ' ed copy of the entry deleted ( if required ) .
Ignore if no entry deleted .
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
2001-08-22 05:01:37 +04:00
ssize_t del_share_entry ( SMB_DEV_T dev , SMB_INO_T inode ,
2001-08-21 05:25:45 +04:00
share_mode_entry * entry , share_mode_entry * * ppse )
1999-12-21 12:25:59 +03:00
{
TDB_DATA dbuf ;
struct locking_data * data ;
int i , del_count = 0 ;
share_mode_entry * shares ;
2001-08-21 22:33:58 +04:00
ssize_t count = 0 ;
2003-09-09 00:42:33 +04:00
TDB_DATA key = locking_key ( dev , inode ) ;
This is a big, rather ugly patch. Whilst investigating the files not truncated
when copying to a full disk problem, I discovered that we were not allowing
the delete on close flag to be set properly, this led to other things, and
after investigation of the proper delete on close semantics and their relationship
to the file_share_delete flag I discovered there were some cases where we
weren't doing the deny modes properly. And this after only 5 years working
on them..... :-) :-).
So here's the latest attempt. I realised the delete on close flag needs to
be set across all smbds with a dev/ino pair open - in addition, the delete
on close flag, allow share delete and delete access requested all need to
be stored in the share mode tdb.
The "delete_on_close" entry in the fsp struct is now redundant and should
really be removed. This may also mean we can get rid of the "iterate_fsp"
calls that I didn't like adding in the first place. Whilst doing this patch,
I also discovered we needed to do the se_map_generic() call for file opens
and POSIX ACL mapping, so I added that also.
This code, although ugly, now passes the deny mode torture tests plus the
delete on close tests I added. I do need to add one more multiple connection
delete on close test to make sure I got the semantics exactly right, plus we
should also (as Andrew suggested) move to random testing here.
The good news is that NT should now correctly delete the file on disk
full error when copying to a disk :-).
Jeremy.
(This used to be commit 51987684bd231c744da2e5f3705fd236d5616173)
2001-03-30 12:57:24 +04:00
2001-08-21 05:25:45 +04:00
if ( ppse )
* ppse = NULL ;
1999-12-21 12:25:59 +03:00
/* read in the existing share modes */
2003-09-09 00:42:33 +04:00
dbuf = tdb_fetch ( tdb , key ) ;
2001-08-21 05:25:45 +04:00
if ( ! dbuf . dptr )
return - 1 ;
1999-12-21 12:25:59 +03:00
data = ( struct locking_data * ) dbuf . dptr ;
shares = ( share_mode_entry * ) ( dbuf . dptr + sizeof ( * data ) ) ;
2001-08-21 05:25:45 +04:00
/*
* Find any with this pid and delete it
* by overwriting with the rest of the data
* from the record .
*/
2001-10-20 03:56:21 +04:00
DEBUG ( 10 , ( " del_share_entry: num_share_modes = %d \n " , data - > u . num_share_mode_entries ) ) ;
2001-08-22 04:29:40 +04:00
2001-09-18 10:41:29 +04:00
for ( i = 0 ; i < data - > u . num_share_mode_entries ; ) {
2001-08-21 05:25:45 +04:00
if ( share_modes_identical ( & shares [ i ] , entry ) ) {
2002-01-08 03:52:36 +03:00
DEBUG ( 10 , ( " del_share_entry: deleted %s \n " ,
share_mode_str ( i , & shares [ i ] ) ) ) ;
2001-08-21 05:25:45 +04:00
if ( ppse )
* ppse = memdup ( & shares [ i ] , sizeof ( * shares ) ) ;
2001-09-18 10:41:29 +04:00
data - > u . num_share_mode_entries - - ;
2004-06-03 03:19:36 +04:00
if ( ( dbuf . dsize - ( sizeof ( * data ) + ( i + 1 ) * sizeof ( * shares ) ) ) > 0 ) {
memmove ( & shares [ i ] , & shares [ i + 1 ] ,
dbuf . dsize - ( sizeof ( * data ) + ( i + 1 ) * sizeof ( * shares ) ) ) ;
}
1999-12-21 12:25:59 +03:00
del_count + + ;
2001-08-22 04:29:40 +04:00
2001-10-20 03:56:21 +04:00
DEBUG ( 10 , ( " del_share_entry: deleting entry %d \n " , i ) ) ;
2001-08-22 04:29:40 +04:00
1999-12-21 12:25:59 +03:00
} else {
i + + ;
}
}
2001-08-21 05:25:45 +04:00
if ( del_count ) {
/* the record may have shrunk a bit */
dbuf . dsize - = del_count * sizeof ( * shares ) ;
1999-12-21 12:25:59 +03:00
2001-09-18 10:41:29 +04:00
count = ( ssize_t ) data - > u . num_share_mode_entries ;
This is a big, rather ugly patch. Whilst investigating the files not truncated
when copying to a full disk problem, I discovered that we were not allowing
the delete on close flag to be set properly, this led to other things, and
after investigation of the proper delete on close semantics and their relationship
to the file_share_delete flag I discovered there were some cases where we
weren't doing the deny modes properly. And this after only 5 years working
on them..... :-) :-).
So here's the latest attempt. I realised the delete on close flag needs to
be set across all smbds with a dev/ino pair open - in addition, the delete
on close flag, allow share delete and delete access requested all need to
be stored in the share mode tdb.
The "delete_on_close" entry in the fsp struct is now redundant and should
really be removed. This may also mean we can get rid of the "iterate_fsp"
calls that I didn't like adding in the first place. Whilst doing this patch,
I also discovered we needed to do the se_map_generic() call for file opens
and POSIX ACL mapping, so I added that also.
This code, although ugly, now passes the deny mode torture tests plus the
delete on close tests I added. I do need to add one more multiple connection
delete on close test to make sure I got the semantics exactly right, plus we
should also (as Andrew suggested) move to random testing here.
The good news is that NT should now correctly delete the file on disk
full error when copying to a disk :-).
Jeremy.
(This used to be commit 51987684bd231c744da2e5f3705fd236d5616173)
2001-03-30 12:57:24 +04:00
2001-08-21 05:25:45 +04:00
/* store it back in the database */
2001-09-18 10:41:29 +04:00
if ( data - > u . num_share_mode_entries = = 0 ) {
2003-09-09 00:42:33 +04:00
if ( tdb_delete ( tdb , key ) = = - 1 )
2001-08-21 05:25:45 +04:00
count = - 1 ;
} else {
2003-09-09 00:42:33 +04:00
if ( tdb_store ( tdb , key , dbuf , TDB_REPLACE ) = = - 1 )
2001-08-21 05:25:45 +04:00
count = - 1 ;
}
2000-04-23 18:25:04 +04:00
}
2002-01-08 03:52:36 +03:00
DEBUG ( 10 , ( " del_share_entry: Remaining table. \n " ) ) ;
print_share_mode_table ( ( struct locking_data * ) dbuf . dptr ) ;
2001-09-17 08:16:35 +04:00
SAFE_FREE ( dbuf . dptr ) ;
This is a big, rather ugly patch. Whilst investigating the files not truncated
when copying to a full disk problem, I discovered that we were not allowing
the delete on close flag to be set properly, this led to other things, and
after investigation of the proper delete on close semantics and their relationship
to the file_share_delete flag I discovered there were some cases where we
weren't doing the deny modes properly. And this after only 5 years working
on them..... :-) :-).
So here's the latest attempt. I realised the delete on close flag needs to
be set across all smbds with a dev/ino pair open - in addition, the delete
on close flag, allow share delete and delete access requested all need to
be stored in the share mode tdb.
The "delete_on_close" entry in the fsp struct is now redundant and should
really be removed. This may also mean we can get rid of the "iterate_fsp"
calls that I didn't like adding in the first place. Whilst doing this patch,
I also discovered we needed to do the se_map_generic() call for file opens
and POSIX ACL mapping, so I added that also.
This code, although ugly, now passes the deny mode torture tests plus the
delete on close tests I added. I do need to add one more multiple connection
delete on close test to make sure I got the semantics exactly right, plus we
should also (as Andrew suggested) move to random testing here.
The good news is that NT should now correctly delete the file on disk
full error when copying to a disk :-).
Jeremy.
(This used to be commit 51987684bd231c744da2e5f3705fd236d5616173)
2001-03-30 12:57:24 +04:00
return count ;
1997-05-20 04:32:51 +04:00
}
/*******************************************************************
2001-08-21 05:25:45 +04:00
Del the share mode of a file for this process . Return the number
of entries left , and a memdup ' ed copy of the entry deleted .
1997-05-20 04:32:51 +04:00
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
2001-08-21 05:25:45 +04:00
ssize_t del_share_mode ( files_struct * fsp , share_mode_entry * * ppse )
1997-05-20 04:32:51 +04:00
{
2001-08-21 05:25:45 +04:00
share_mode_entry entry ;
/*
* Fake up a share_mode_entry for comparisons .
*/
fill_share_mode ( ( char * ) & entry , fsp , 0 , 0 ) ;
return del_share_entry ( fsp - > dev , fsp - > inode , & entry , ppse ) ;
1997-05-20 04:32:51 +04:00
}
/*******************************************************************
1998-07-23 04:10:26 +04:00
Set the share mode of a file . Return False on fail , True on success .
1997-05-20 04:32:51 +04:00
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
2001-08-21 05:25:45 +04:00
1999-12-21 12:25:59 +03:00
BOOL set_share_mode ( files_struct * fsp , uint16 port , uint16 op_type )
1997-05-20 04:32:51 +04:00
{
1999-12-21 12:25:59 +03:00
TDB_DATA dbuf ;
struct locking_data * data ;
char * p = NULL ;
int size ;
2003-09-09 00:42:33 +04:00
TDB_DATA key = locking_key_fsp ( fsp ) ;
2001-08-21 05:25:45 +04:00
BOOL ret = True ;
1999-12-21 12:25:59 +03:00
/* read in the existing share modes if any */
2003-09-09 00:42:33 +04:00
dbuf = tdb_fetch ( tdb , key ) ;
1999-12-21 12:25:59 +03:00
if ( ! dbuf . dptr ) {
2003-02-10 14:47:21 +03:00
size_t offset ;
1999-12-21 12:25:59 +03:00
/* we'll need to create a new record */
pstring fname ;
pstrcpy ( fname , fsp - > conn - > connectpath ) ;
pstrcat ( fname , " / " ) ;
pstrcat ( fname , fsp - > fsp_name ) ;
2001-03-11 03:32:10 +03:00
size = sizeof ( * data ) + sizeof ( share_mode_entry ) + strlen ( fname ) + 1 ;
1999-12-21 12:25:59 +03:00
p = ( char * ) malloc ( size ) ;
2001-08-21 05:25:45 +04:00
if ( ! p )
return False ;
1999-12-21 12:25:59 +03:00
data = ( struct locking_data * ) p ;
2001-09-18 10:41:29 +04:00
data - > u . num_share_mode_entries = 1 ;
2002-01-05 05:37:23 +03:00
DEBUG ( 10 , ( " set_share_mode: creating entry for file %s. num_share_modes = 1 \n " ,
fsp - > fsp_name ) ) ;
2003-02-10 14:47:21 +03:00
offset = sizeof ( * data ) + sizeof ( share_mode_entry ) ;
2003-02-24 06:03:15 +03:00
safe_strcpy ( p + offset , fname , size - offset - 1 ) ;
1999-12-21 12:25:59 +03:00
fill_share_mode ( p + sizeof ( * data ) , fsp , port , op_type ) ;
dbuf . dptr = p ;
dbuf . dsize = size ;
2003-09-09 00:42:33 +04:00
if ( tdb_store ( tdb , key , dbuf , TDB_REPLACE ) = = - 1 )
2001-08-21 05:25:45 +04:00
ret = False ;
2002-01-08 03:52:36 +03:00
print_share_mode_table ( ( struct locking_data * ) p ) ;
2001-09-17 08:16:35 +04:00
SAFE_FREE ( p ) ;
2001-08-21 05:25:45 +04:00
return ret ;
1999-12-21 12:25:59 +03:00
}
/* we're adding to an existing entry - this is a bit fiddly */
data = ( struct locking_data * ) dbuf . dptr ;
2001-09-18 10:41:29 +04:00
data - > u . num_share_mode_entries + + ;
2002-01-05 05:37:23 +03:00
DEBUG ( 10 , ( " set_share_mode: adding entry for file %s. new num_share_modes = %d \n " ,
fsp - > fsp_name , data - > u . num_share_mode_entries ) ) ;
2001-03-11 03:32:10 +03:00
size = dbuf . dsize + sizeof ( share_mode_entry ) ;
1999-12-21 12:25:59 +03:00
p = malloc ( size ) ;
2002-11-23 05:52:36 +03:00
if ( ! p ) {
SAFE_FREE ( dbuf . dptr ) ;
2001-08-21 05:25:45 +04:00
return False ;
2002-11-23 05:52:36 +03:00
}
1999-12-21 12:25:59 +03:00
memcpy ( p , dbuf . dptr , sizeof ( * data ) ) ;
fill_share_mode ( p + sizeof ( * data ) , fsp , port , op_type ) ;
2001-03-11 03:32:10 +03:00
memcpy ( p + sizeof ( * data ) + sizeof ( share_mode_entry ) , dbuf . dptr + sizeof ( * data ) ,
1999-12-21 12:25:59 +03:00
dbuf . dsize - sizeof ( * data ) ) ;
2001-09-17 08:16:35 +04:00
SAFE_FREE ( dbuf . dptr ) ;
1999-12-21 12:25:59 +03:00
dbuf . dptr = p ;
dbuf . dsize = size ;
2003-09-09 00:42:33 +04:00
if ( tdb_store ( tdb , key , dbuf , TDB_REPLACE ) = = - 1 )
2001-08-21 05:25:45 +04:00
ret = False ;
2002-01-08 03:52:36 +03:00
print_share_mode_table ( ( struct locking_data * ) p ) ;
2001-09-17 08:16:35 +04:00
SAFE_FREE ( p ) ;
2001-08-21 05:25:45 +04:00
return ret ;
1996-05-04 11:50:46 +04:00
}
1999-12-21 12:25:59 +03:00
/*******************************************************************
2001-08-21 05:25:45 +04:00
A generic in - place modification call for share mode entries .
1999-12-21 12:25:59 +03:00
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
2001-08-21 05:25:45 +04:00
2001-08-22 04:29:40 +04:00
static BOOL mod_share_mode ( SMB_DEV_T dev , SMB_INO_T inode , share_mode_entry * entry ,
1999-12-21 12:25:59 +03:00
void ( * mod_fn ) ( share_mode_entry * , SMB_DEV_T , SMB_INO_T , void * ) ,
void * param )
{
TDB_DATA dbuf ;
struct locking_data * data ;
int i ;
2001-08-22 04:29:40 +04:00
share_mode_entry * shares ;
2001-08-21 05:25:45 +04:00
BOOL need_store = False ;
2002-07-15 14:35:28 +04:00
BOOL ret = True ;
2003-09-09 00:42:33 +04:00
TDB_DATA key = locking_key ( dev , inode ) ;
1999-12-21 12:25:59 +03:00
/* read in the existing share modes */
2003-09-09 00:42:33 +04:00
dbuf = tdb_fetch ( tdb , key ) ;
2001-08-21 05:25:45 +04:00
if ( ! dbuf . dptr )
return False ;
1999-12-21 12:25:59 +03:00
data = ( struct locking_data * ) dbuf . dptr ;
shares = ( share_mode_entry * ) ( dbuf . dptr + sizeof ( * data ) ) ;
/* find any with our pid and call the supplied function */
2001-09-18 10:41:29 +04:00
for ( i = 0 ; i < data - > u . num_share_mode_entries ; i + + ) {
2001-08-22 04:29:40 +04:00
if ( share_modes_identical ( entry , & shares [ i ] ) ) {
mod_fn ( & shares [ i ] , dev , inode , param ) ;
2001-08-21 05:25:45 +04:00
need_store = True ;
1999-12-21 12:25:59 +03:00
}
}
/* if the mod fn was called then store it back */
if ( need_store ) {
2001-09-18 10:41:29 +04:00
if ( data - > u . num_share_mode_entries = = 0 ) {
2003-09-09 00:42:33 +04:00
if ( tdb_delete ( tdb , key ) = = - 1 )
2002-07-15 14:35:28 +04:00
ret = False ;
2000-04-23 18:25:04 +04:00
} else {
2003-09-09 00:42:33 +04:00
if ( tdb_store ( tdb , key , dbuf , TDB_REPLACE ) = = - 1 )
2002-07-15 14:35:28 +04:00
ret = False ;
2000-04-23 18:25:04 +04:00
}
1999-12-21 12:25:59 +03:00
}
2001-09-17 08:16:35 +04:00
SAFE_FREE ( dbuf . dptr ) ;
2002-07-15 14:35:28 +04:00
return ret ;
1999-12-21 12:25:59 +03:00
}
1998-10-23 07:34:50 +04:00
/*******************************************************************
Static function that actually does the work for the generic function
below .
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
2001-08-21 05:25:45 +04:00
1998-10-23 07:34:50 +04:00
static void remove_share_oplock_fn ( share_mode_entry * entry , SMB_DEV_T dev , SMB_INO_T inode ,
void * param )
{
1999-12-21 12:25:59 +03:00
DEBUG ( 10 , ( " remove_share_oplock_fn: removing oplock info for entry dev=%x ino=%.0f \n " ,
( unsigned int ) dev , ( double ) inode ) ) ;
/* Delete the oplock info. */
entry - > op_port = 0 ;
entry - > op_type = NO_OPLOCK ;
1998-10-23 07:34:50 +04:00
}
1996-05-04 11:50:46 +04:00
/*******************************************************************
1998-07-23 04:10:26 +04:00
Remove an oplock port and mode entry from a share mode .
1996-05-04 11:50:46 +04:00
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
2001-08-21 05:25:45 +04:00
1999-12-21 12:25:59 +03:00
BOOL remove_share_oplock ( files_struct * fsp )
1998-10-23 07:34:50 +04:00
{
2001-08-22 04:29:40 +04:00
share_mode_entry entry ;
/*
* Fake up an entry for comparisons . . .
*/
fill_share_mode ( ( char * ) & entry , fsp , 0 , 0 ) ;
return mod_share_mode ( fsp - > dev , fsp - > inode , & entry , remove_share_oplock_fn , NULL ) ;
1998-10-23 07:34:50 +04:00
}
/*******************************************************************
Static function that actually does the work for the generic function
below .
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
2001-08-21 05:25:45 +04:00
1999-12-13 16:27:58 +03:00
static void downgrade_share_oplock_fn ( share_mode_entry * entry , SMB_DEV_T dev , SMB_INO_T inode ,
void * param )
{
1999-12-21 12:25:59 +03:00
DEBUG ( 10 , ( " downgrade_share_oplock_fn: downgrading oplock info for entry dev=%x ino=%.0f \n " ,
( unsigned int ) dev , ( double ) inode ) ) ;
entry - > op_type = LEVEL_II_OPLOCK ;
1999-12-13 16:27:58 +03:00
}
/*******************************************************************
Downgrade a oplock type from exclusive to level II .
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
2001-08-21 05:25:45 +04:00
1999-12-21 12:25:59 +03:00
BOOL downgrade_share_oplock ( files_struct * fsp )
1999-12-13 16:27:58 +03:00
{
2001-08-22 04:29:40 +04:00
share_mode_entry entry ;
/*
* Fake up an entry for comparisons . . .
*/
fill_share_mode ( ( char * ) & entry , fsp , 0 , 0 ) ;
return mod_share_mode ( fsp - > dev , fsp - > inode , & entry , downgrade_share_oplock_fn , NULL ) ;
}
1999-12-13 16:27:58 +03:00
/*******************************************************************
This is a big, rather ugly patch. Whilst investigating the files not truncated
when copying to a full disk problem, I discovered that we were not allowing
the delete on close flag to be set properly, this led to other things, and
after investigation of the proper delete on close semantics and their relationship
to the file_share_delete flag I discovered there were some cases where we
weren't doing the deny modes properly. And this after only 5 years working
on them..... :-) :-).
So here's the latest attempt. I realised the delete on close flag needs to
be set across all smbds with a dev/ino pair open - in addition, the delete
on close flag, allow share delete and delete access requested all need to
be stored in the share mode tdb.
The "delete_on_close" entry in the fsp struct is now redundant and should
really be removed. This may also mean we can get rid of the "iterate_fsp"
calls that I didn't like adding in the first place. Whilst doing this patch,
I also discovered we needed to do the se_map_generic() call for file opens
and POSIX ACL mapping, so I added that also.
This code, although ugly, now passes the deny mode torture tests plus the
delete on close tests I added. I do need to add one more multiple connection
delete on close test to make sure I got the semantics exactly right, plus we
should also (as Andrew suggested) move to random testing here.
The good news is that NT should now correctly delete the file on disk
full error when copying to a disk :-).
Jeremy.
(This used to be commit 51987684bd231c744da2e5f3705fd236d5616173)
2001-03-30 12:57:24 +04:00
Get / Set the delete on close flag in a set of share modes .
Return False on fail , True on success .
1999-12-13 16:27:58 +03:00
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
This is a big, rather ugly patch. Whilst investigating the files not truncated
when copying to a full disk problem, I discovered that we were not allowing
the delete on close flag to be set properly, this led to other things, and
after investigation of the proper delete on close semantics and their relationship
to the file_share_delete flag I discovered there were some cases where we
weren't doing the deny modes properly. And this after only 5 years working
on them..... :-) :-).
So here's the latest attempt. I realised the delete on close flag needs to
be set across all smbds with a dev/ino pair open - in addition, the delete
on close flag, allow share delete and delete access requested all need to
be stored in the share mode tdb.
The "delete_on_close" entry in the fsp struct is now redundant and should
really be removed. This may also mean we can get rid of the "iterate_fsp"
calls that I didn't like adding in the first place. Whilst doing this patch,
I also discovered we needed to do the se_map_generic() call for file opens
and POSIX ACL mapping, so I added that also.
This code, although ugly, now passes the deny mode torture tests plus the
delete on close tests I added. I do need to add one more multiple connection
delete on close test to make sure I got the semantics exactly right, plus we
should also (as Andrew suggested) move to random testing here.
The good news is that NT should now correctly delete the file on disk
full error when copying to a disk :-).
Jeremy.
(This used to be commit 51987684bd231c744da2e5f3705fd236d5616173)
2001-03-30 12:57:24 +04:00
BOOL modify_delete_flag ( SMB_DEV_T dev , SMB_INO_T inode , BOOL delete_on_close )
1998-10-23 07:34:50 +04:00
{
This is a big, rather ugly patch. Whilst investigating the files not truncated
when copying to a full disk problem, I discovered that we were not allowing
the delete on close flag to be set properly, this led to other things, and
after investigation of the proper delete on close semantics and their relationship
to the file_share_delete flag I discovered there were some cases where we
weren't doing the deny modes properly. And this after only 5 years working
on them..... :-) :-).
So here's the latest attempt. I realised the delete on close flag needs to
be set across all smbds with a dev/ino pair open - in addition, the delete
on close flag, allow share delete and delete access requested all need to
be stored in the share mode tdb.
The "delete_on_close" entry in the fsp struct is now redundant and should
really be removed. This may also mean we can get rid of the "iterate_fsp"
calls that I didn't like adding in the first place. Whilst doing this patch,
I also discovered we needed to do the se_map_generic() call for file opens
and POSIX ACL mapping, so I added that also.
This code, although ugly, now passes the deny mode torture tests plus the
delete on close tests I added. I do need to add one more multiple connection
delete on close test to make sure I got the semantics exactly right, plus we
should also (as Andrew suggested) move to random testing here.
The good news is that NT should now correctly delete the file on disk
full error when copying to a disk :-).
Jeremy.
(This used to be commit 51987684bd231c744da2e5f3705fd236d5616173)
2001-03-30 12:57:24 +04:00
TDB_DATA dbuf ;
struct locking_data * data ;
int i ;
share_mode_entry * shares ;
2003-09-09 00:42:33 +04:00
TDB_DATA key = locking_key ( dev , inode ) ;
1998-10-23 07:34:50 +04:00
This is a big, rather ugly patch. Whilst investigating the files not truncated
when copying to a full disk problem, I discovered that we were not allowing
the delete on close flag to be set properly, this led to other things, and
after investigation of the proper delete on close semantics and their relationship
to the file_share_delete flag I discovered there were some cases where we
weren't doing the deny modes properly. And this after only 5 years working
on them..... :-) :-).
So here's the latest attempt. I realised the delete on close flag needs to
be set across all smbds with a dev/ino pair open - in addition, the delete
on close flag, allow share delete and delete access requested all need to
be stored in the share mode tdb.
The "delete_on_close" entry in the fsp struct is now redundant and should
really be removed. This may also mean we can get rid of the "iterate_fsp"
calls that I didn't like adding in the first place. Whilst doing this patch,
I also discovered we needed to do the se_map_generic() call for file opens
and POSIX ACL mapping, so I added that also.
This code, although ugly, now passes the deny mode torture tests plus the
delete on close tests I added. I do need to add one more multiple connection
delete on close test to make sure I got the semantics exactly right, plus we
should also (as Andrew suggested) move to random testing here.
The good news is that NT should now correctly delete the file on disk
full error when copying to a disk :-).
Jeremy.
(This used to be commit 51987684bd231c744da2e5f3705fd236d5616173)
2001-03-30 12:57:24 +04:00
/* read in the existing share modes */
2003-09-09 00:42:33 +04:00
dbuf = tdb_fetch ( tdb , key ) ;
2001-08-21 05:25:45 +04:00
if ( ! dbuf . dptr )
return False ;
This is a big, rather ugly patch. Whilst investigating the files not truncated
when copying to a full disk problem, I discovered that we were not allowing
the delete on close flag to be set properly, this led to other things, and
after investigation of the proper delete on close semantics and their relationship
to the file_share_delete flag I discovered there were some cases where we
weren't doing the deny modes properly. And this after only 5 years working
on them..... :-) :-).
So here's the latest attempt. I realised the delete on close flag needs to
be set across all smbds with a dev/ino pair open - in addition, the delete
on close flag, allow share delete and delete access requested all need to
be stored in the share mode tdb.
The "delete_on_close" entry in the fsp struct is now redundant and should
really be removed. This may also mean we can get rid of the "iterate_fsp"
calls that I didn't like adding in the first place. Whilst doing this patch,
I also discovered we needed to do the se_map_generic() call for file opens
and POSIX ACL mapping, so I added that also.
This code, although ugly, now passes the deny mode torture tests plus the
delete on close tests I added. I do need to add one more multiple connection
delete on close test to make sure I got the semantics exactly right, plus we
should also (as Andrew suggested) move to random testing here.
The good news is that NT should now correctly delete the file on disk
full error when copying to a disk :-).
Jeremy.
(This used to be commit 51987684bd231c744da2e5f3705fd236d5616173)
2001-03-30 12:57:24 +04:00
data = ( struct locking_data * ) dbuf . dptr ;
shares = ( share_mode_entry * ) ( dbuf . dptr + sizeof ( * data ) ) ;
1999-12-13 16:27:58 +03:00
This is a big, rather ugly patch. Whilst investigating the files not truncated
when copying to a full disk problem, I discovered that we were not allowing
the delete on close flag to be set properly, this led to other things, and
after investigation of the proper delete on close semantics and their relationship
to the file_share_delete flag I discovered there were some cases where we
weren't doing the deny modes properly. And this after only 5 years working
on them..... :-) :-).
So here's the latest attempt. I realised the delete on close flag needs to
be set across all smbds with a dev/ino pair open - in addition, the delete
on close flag, allow share delete and delete access requested all need to
be stored in the share mode tdb.
The "delete_on_close" entry in the fsp struct is now redundant and should
really be removed. This may also mean we can get rid of the "iterate_fsp"
calls that I didn't like adding in the first place. Whilst doing this patch,
I also discovered we needed to do the se_map_generic() call for file opens
and POSIX ACL mapping, so I added that also.
This code, although ugly, now passes the deny mode torture tests plus the
delete on close tests I added. I do need to add one more multiple connection
delete on close test to make sure I got the semantics exactly right, plus we
should also (as Andrew suggested) move to random testing here.
The good news is that NT should now correctly delete the file on disk
full error when copying to a disk :-).
Jeremy.
(This used to be commit 51987684bd231c744da2e5f3705fd236d5616173)
2001-03-30 12:57:24 +04:00
/* Set/Unset the delete on close element. */
2001-09-18 10:41:29 +04:00
for ( i = 0 ; i < data - > u . num_share_mode_entries ; i + + , shares + + ) {
This is a big, rather ugly patch. Whilst investigating the files not truncated
when copying to a full disk problem, I discovered that we were not allowing
the delete on close flag to be set properly, this led to other things, and
after investigation of the proper delete on close semantics and their relationship
to the file_share_delete flag I discovered there were some cases where we
weren't doing the deny modes properly. And this after only 5 years working
on them..... :-) :-).
So here's the latest attempt. I realised the delete on close flag needs to
be set across all smbds with a dev/ino pair open - in addition, the delete
on close flag, allow share delete and delete access requested all need to
be stored in the share mode tdb.
The "delete_on_close" entry in the fsp struct is now redundant and should
really be removed. This may also mean we can get rid of the "iterate_fsp"
calls that I didn't like adding in the first place. Whilst doing this patch,
I also discovered we needed to do the se_map_generic() call for file opens
and POSIX ACL mapping, so I added that also.
This code, although ugly, now passes the deny mode torture tests plus the
delete on close tests I added. I do need to add one more multiple connection
delete on close test to make sure I got the semantics exactly right, plus we
should also (as Andrew suggested) move to random testing here.
The good news is that NT should now correctly delete the file on disk
full error when copying to a disk :-).
Jeremy.
(This used to be commit 51987684bd231c744da2e5f3705fd236d5616173)
2001-03-30 12:57:24 +04:00
shares - > share_mode = ( delete_on_close ?
( shares - > share_mode | DELETE_ON_CLOSE_FLAG ) :
( shares - > share_mode & ~ DELETE_ON_CLOSE_FLAG ) ) ;
}
1999-12-13 16:27:58 +03:00
This is a big, rather ugly patch. Whilst investigating the files not truncated
when copying to a full disk problem, I discovered that we were not allowing
the delete on close flag to be set properly, this led to other things, and
after investigation of the proper delete on close semantics and their relationship
to the file_share_delete flag I discovered there were some cases where we
weren't doing the deny modes properly. And this after only 5 years working
on them..... :-) :-).
So here's the latest attempt. I realised the delete on close flag needs to
be set across all smbds with a dev/ino pair open - in addition, the delete
on close flag, allow share delete and delete access requested all need to
be stored in the share mode tdb.
The "delete_on_close" entry in the fsp struct is now redundant and should
really be removed. This may also mean we can get rid of the "iterate_fsp"
calls that I didn't like adding in the first place. Whilst doing this patch,
I also discovered we needed to do the se_map_generic() call for file opens
and POSIX ACL mapping, so I added that also.
This code, although ugly, now passes the deny mode torture tests plus the
delete on close tests I added. I do need to add one more multiple connection
delete on close test to make sure I got the semantics exactly right, plus we
should also (as Andrew suggested) move to random testing here.
The good news is that NT should now correctly delete the file on disk
full error when copying to a disk :-).
Jeremy.
(This used to be commit 51987684bd231c744da2e5f3705fd236d5616173)
2001-03-30 12:57:24 +04:00
/* store it back */
2001-09-18 10:41:29 +04:00
if ( data - > u . num_share_mode_entries ) {
2003-09-09 00:42:33 +04:00
if ( tdb_store ( tdb , key , dbuf , TDB_REPLACE ) = = - 1 ) {
2001-09-17 08:16:35 +04:00
SAFE_FREE ( dbuf . dptr ) ;
This is a big, rather ugly patch. Whilst investigating the files not truncated
when copying to a full disk problem, I discovered that we were not allowing
the delete on close flag to be set properly, this led to other things, and
after investigation of the proper delete on close semantics and their relationship
to the file_share_delete flag I discovered there were some cases where we
weren't doing the deny modes properly. And this after only 5 years working
on them..... :-) :-).
So here's the latest attempt. I realised the delete on close flag needs to
be set across all smbds with a dev/ino pair open - in addition, the delete
on close flag, allow share delete and delete access requested all need to
be stored in the share mode tdb.
The "delete_on_close" entry in the fsp struct is now redundant and should
really be removed. This may also mean we can get rid of the "iterate_fsp"
calls that I didn't like adding in the first place. Whilst doing this patch,
I also discovered we needed to do the se_map_generic() call for file opens
and POSIX ACL mapping, so I added that also.
This code, although ugly, now passes the deny mode torture tests plus the
delete on close tests I added. I do need to add one more multiple connection
delete on close test to make sure I got the semantics exactly right, plus we
should also (as Andrew suggested) move to random testing here.
The good news is that NT should now correctly delete the file on disk
full error when copying to a disk :-).
Jeremy.
(This used to be commit 51987684bd231c744da2e5f3705fd236d5616173)
2001-03-30 12:57:24 +04:00
return False ;
2001-08-21 05:25:45 +04:00
}
This is a big, rather ugly patch. Whilst investigating the files not truncated
when copying to a full disk problem, I discovered that we were not allowing
the delete on close flag to be set properly, this led to other things, and
after investigation of the proper delete on close semantics and their relationship
to the file_share_delete flag I discovered there were some cases where we
weren't doing the deny modes properly. And this after only 5 years working
on them..... :-) :-).
So here's the latest attempt. I realised the delete on close flag needs to
be set across all smbds with a dev/ino pair open - in addition, the delete
on close flag, allow share delete and delete access requested all need to
be stored in the share mode tdb.
The "delete_on_close" entry in the fsp struct is now redundant and should
really be removed. This may also mean we can get rid of the "iterate_fsp"
calls that I didn't like adding in the first place. Whilst doing this patch,
I also discovered we needed to do the se_map_generic() call for file opens
and POSIX ACL mapping, so I added that also.
This code, although ugly, now passes the deny mode torture tests plus the
delete on close tests I added. I do need to add one more multiple connection
delete on close test to make sure I got the semantics exactly right, plus we
should also (as Andrew suggested) move to random testing here.
The good news is that NT should now correctly delete the file on disk
full error when copying to a disk :-).
Jeremy.
(This used to be commit 51987684bd231c744da2e5f3705fd236d5616173)
2001-03-30 12:57:24 +04:00
}
2001-09-17 08:16:35 +04:00
SAFE_FREE ( dbuf . dptr ) ;
This is a big, rather ugly patch. Whilst investigating the files not truncated
when copying to a full disk problem, I discovered that we were not allowing
the delete on close flag to be set properly, this led to other things, and
after investigation of the proper delete on close semantics and their relationship
to the file_share_delete flag I discovered there were some cases where we
weren't doing the deny modes properly. And this after only 5 years working
on them..... :-) :-).
So here's the latest attempt. I realised the delete on close flag needs to
be set across all smbds with a dev/ino pair open - in addition, the delete
on close flag, allow share delete and delete access requested all need to
be stored in the share mode tdb.
The "delete_on_close" entry in the fsp struct is now redundant and should
really be removed. This may also mean we can get rid of the "iterate_fsp"
calls that I didn't like adding in the first place. Whilst doing this patch,
I also discovered we needed to do the se_map_generic() call for file opens
and POSIX ACL mapping, so I added that also.
This code, although ugly, now passes the deny mode torture tests plus the
delete on close tests I added. I do need to add one more multiple connection
delete on close test to make sure I got the semantics exactly right, plus we
should also (as Andrew suggested) move to random testing here.
The good news is that NT should now correctly delete the file on disk
full error when copying to a disk :-).
Jeremy.
(This used to be commit 51987684bd231c744da2e5f3705fd236d5616173)
2001-03-30 12:57:24 +04:00
return True ;
1997-10-20 12:46:00 +04:00
}
1997-05-20 04:32:51 +04:00
1999-12-21 12:25:59 +03:00
/****************************************************************************
2001-08-21 05:25:45 +04:00
Traverse the whole database with this function , calling traverse_callback
on each share mode
1999-12-21 12:25:59 +03:00
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
2001-08-21 05:25:45 +04:00
2000-02-07 03:24:04 +03:00
static int traverse_fn ( TDB_CONTEXT * the_tdb , TDB_DATA kbuf , TDB_DATA dbuf ,
void * state )
1997-05-20 04:32:51 +04:00
{
1999-12-21 12:25:59 +03:00
struct locking_data * data ;
share_mode_entry * shares ;
char * name ;
int i ;
1) added void* state argument to tdb_traverse. guess what! there were
two places i found where it was appropriate to _use_ that third argument,
in locking.c and brlock.c! there was a static traverse_function and
i removed the static variable, typecast it to a void*, passed it to
tdb_traverse and re-cast it back to the traverse_function inside the
tdb_traverse function. this makes the use of tdb_traverse() reentrant,
which is never going to happen, i know, i just don't like to see
statics lying about when there's no need for them.
as i had to do in samba-tng, all uses of tdb_traverse modified to take
the new void* state argument.
2) disabled rpcclient: referring people to use SAMBA_TNG rpcclient.
i don't know how the other samba team members would react if i deleted
rpcclient from cvs main. damn, that code's so old, it's unreal.
20 rpcclient commands, instead of about 70 in SAMBA_TNG.
(This used to be commit 49d7f0afbc1c5425d53019e234d54ddf205c8e9a)
2000-02-04 07:59:31 +03:00
SHAREMODE_FN ( traverse_callback ) = ( SHAREMODE_FN_CAST ( ) ) state ;
1999-12-21 12:25:59 +03:00
data = ( struct locking_data * ) dbuf . dptr ;
shares = ( share_mode_entry * ) ( dbuf . dptr + sizeof ( * data ) ) ;
2001-09-18 10:41:29 +04:00
name = dbuf . dptr + sizeof ( * data ) + data - > u . num_share_mode_entries * sizeof ( * shares ) ;
1999-12-21 12:25:59 +03:00
2001-09-18 10:41:29 +04:00
for ( i = 0 ; i < data - > u . num_share_mode_entries ; i + + ) {
1999-12-21 12:25:59 +03:00
traverse_callback ( & shares [ i ] , name ) ;
}
return 0 ;
1996-05-04 11:50:46 +04:00
}
1997-10-02 03:32:22 +04:00
/*******************************************************************
1999-12-21 12:25:59 +03:00
Call the specified function on each entry under management by the
share mode system .
1997-10-02 03:32:22 +04:00
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
2001-08-21 05:25:45 +04:00
1) added void* state argument to tdb_traverse. guess what! there were
two places i found where it was appropriate to _use_ that third argument,
in locking.c and brlock.c! there was a static traverse_function and
i removed the static variable, typecast it to a void*, passed it to
tdb_traverse and re-cast it back to the traverse_function inside the
tdb_traverse function. this makes the use of tdb_traverse() reentrant,
which is never going to happen, i know, i just don't like to see
statics lying about when there's no need for them.
as i had to do in samba-tng, all uses of tdb_traverse modified to take
the new void* state argument.
2) disabled rpcclient: referring people to use SAMBA_TNG rpcclient.
i don't know how the other samba team members would react if i deleted
rpcclient from cvs main. damn, that code's so old, it's unreal.
20 rpcclient commands, instead of about 70 in SAMBA_TNG.
(This used to be commit 49d7f0afbc1c5425d53019e234d54ddf205c8e9a)
2000-02-04 07:59:31 +03:00
int share_mode_forall ( SHAREMODE_FN ( fn ) )
1997-10-02 03:32:22 +04:00
{
2001-08-21 05:25:45 +04:00
if ( ! tdb )
return 0 ;
1) added void* state argument to tdb_traverse. guess what! there were
two places i found where it was appropriate to _use_ that third argument,
in locking.c and brlock.c! there was a static traverse_function and
i removed the static variable, typecast it to a void*, passed it to
tdb_traverse and re-cast it back to the traverse_function inside the
tdb_traverse function. this makes the use of tdb_traverse() reentrant,
which is never going to happen, i know, i just don't like to see
statics lying about when there's no need for them.
as i had to do in samba-tng, all uses of tdb_traverse modified to take
the new void* state argument.
2) disabled rpcclient: referring people to use SAMBA_TNG rpcclient.
i don't know how the other samba team members would react if i deleted
rpcclient from cvs main. damn, that code's so old, it's unreal.
20 rpcclient commands, instead of about 70 in SAMBA_TNG.
(This used to be commit 49d7f0afbc1c5425d53019e234d54ddf205c8e9a)
2000-02-04 07:59:31 +03:00
return tdb_traverse ( tdb , traverse_fn , ( void * ) fn ) ;
1997-10-02 03:32:22 +04:00
}