summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJeremy Allison <jra@samba.org>2007-01-30 01:46:14 +0000
committerGerald (Jerry) Carter <jerry@samba.org>2007-10-10 12:17:30 -0500
commitc219ebd7e811e775bfe9e6ad6403dfce0c1b15cb (patch)
tree0b9cd7f2f48c9e116dc317caca2b31854242eba0
parent7c5221a777b12a1ab1faed3e050363969f70c880 (diff)
downloadsamba-c219ebd7e811e775bfe9e6ad6403dfce0c1b15cb.tar.gz
samba-c219ebd7e811e775bfe9e6ad6403dfce0c1b15cb.tar.bz2
samba-c219ebd7e811e775bfe9e6ad6403dfce0c1b15cb.zip
r21051: More function refactoring... Added #if 0 for
questionable practice (ignoring bad fsp on handle based call). Jeremy. (This used to be commit 5a8c9286c30319f2d2685d4ae29db37679b87e02)
-rw-r--r--source3/smbd/trans2.c89
1 files changed, 55 insertions, 34 deletions
diff --git a/source3/smbd/trans2.c b/source3/smbd/trans2.c
index 1d577f8183..90365348a0 100644
--- a/source3/smbd/trans2.c
+++ b/source3/smbd/trans2.c
@@ -3745,8 +3745,7 @@ NTSTATUS hardlink_internals(connection_struct *conn, char *oldname, char *newnam
Deal with SMB_INFO_SET_EA.
****************************************************************************/
-static int smb_info_set_ea(
- connection_struct *conn,
+static int smb_info_set_ea(connection_struct *conn,
char *outbuf,
int bufsize,
char *params,
@@ -3826,12 +3825,12 @@ static int smb_set_file_disposition_info(connection_struct *conn,
return ERROR_NT(NT_STATUS_INVALID_PARAMETER);
}
- delete_on_close = (CVAL(pdata,0) ? True : False);
-
if (fsp == NULL) {
return(UNIXERROR(ERRDOS,ERRbadfid));
}
+ delete_on_close = (CVAL(pdata,0) ? True : False);
+
status = can_set_delete_on_close(fsp, delete_on_close, dosmode);
if (!NT_STATUS_IS_OK(status)) {
@@ -3849,6 +3848,47 @@ static int smb_set_file_disposition_info(connection_struct *conn,
}
/****************************************************************************
+ Deal with SMB_FILE_POSITION_INFORMATION.
+****************************************************************************/
+
+static int smb_file_position_information(connection_struct *conn,
+ char *outbuf,
+ int bufsize,
+ char *params,
+ int total_params,
+ char *pdata,
+ int total_data,
+ unsigned int max_data_bytes,
+ files_struct *fsp)
+{
+ SMB_BIG_UINT position_information;
+
+ if (total_data < 8) {
+ return ERROR_NT(NT_STATUS_INVALID_PARAMETER);
+ }
+
+ position_information = (SMB_BIG_UINT)IVAL(pdata,0);
+#ifdef LARGE_SMB_OFF_T
+ position_information |= (((SMB_BIG_UINT)IVAL(pdata,4)) << 32);
+#else /* LARGE_SMB_OFF_T */
+ if (IVAL(pdata,4) != 0) {
+ /* more than 32 bits? */
+ return ERROR_NT(NT_STATUS_INVALID_PARAMETER);
+ }
+#endif /* LARGE_SMB_OFF_T */
+ if (fsp) {
+ DEBUG(10,("call_trans2setfilepathinfo: Set file position information for file %s to %.0f\n",
+ fsp->fsp_name, (double)position_information ));
+ fsp->fh->position_information = position_information;
+ }
+
+ /* We're done. We only set position info in this call. */
+ SSVAL(params,0,0);
+ send_trans2_replies(outbuf, bufsize, params, 2, pdata, 0, max_data_bytes);
+ return -1;
+}
+
+/****************************************************************************
Reply to a TRANS2_SETFILEINFO (set file info by fileid or pathname).
****************************************************************************/
@@ -4152,15 +4192,12 @@ static int call_trans2setfilepathinfo(connection_struct *conn, char *inbuf, char
case SMB_FILE_DISPOSITION_INFORMATION:
case SMB_SET_FILE_DISPOSITION_INFO: /* Set delete on close for open file. */
{
- if (total_data < 1) {
- return ERROR_NT(NT_STATUS_INVALID_PARAMETER);
- }
-
+#if 0 /* JRA - should we just ignore this on a path ? */
/* Just ignore this set on a path. */
if (tran_call != TRANSACT2_SETFILEINFO) {
break;
}
-
+#endif
return smb_set_file_disposition_info(conn,
outbuf,
bufsize,
@@ -4175,31 +4212,15 @@ static int call_trans2setfilepathinfo(connection_struct *conn, char *inbuf, char
case SMB_FILE_POSITION_INFORMATION:
{
- SMB_BIG_UINT position_information;
-
- if (total_data < 8) {
- return ERROR_NT(NT_STATUS_INVALID_PARAMETER);
- }
-
- position_information = (SMB_BIG_UINT)IVAL(pdata,0);
-#ifdef LARGE_SMB_OFF_T
- position_information |= (((SMB_BIG_UINT)IVAL(pdata,4)) << 32);
-#else /* LARGE_SMB_OFF_T */
- if (IVAL(pdata,4) != 0) {
- /* more than 32 bits? */
- return ERROR_NT(NT_STATUS_INVALID_PARAMETER);
- }
-#endif /* LARGE_SMB_OFF_T */
- DEBUG(10,("call_trans2setfilepathinfo: Set file position information for file %s to %.0f\n",
- fname, (double)position_information ));
- if (fsp) {
- fsp->fh->position_information = position_information;
- }
-
- /* We're done. We only get position info in this call. */
- SSVAL(params,0,0);
- send_trans2_replies(outbuf, bufsize, params, 2, *ppdata, 0, max_data_bytes);
- return(-1);
+ return smb_file_position_information(conn,
+ outbuf,
+ bufsize,
+ params,
+ total_params,
+ *ppdata,
+ total_data,
+ max_data_bytes,
+ fsp);
}
/* From tridge Samba4 :