Second attempt at fix for bug 6529 - Offline files conflict with Vista and Office 2003.

Confirmation from reporter that this fixes the issue in master on ext3/ext4.
Back-ports to follow.
Jeremy.

1 files changed, 47 insertions, 59 deletions

diff --git a/source3/smbd/trans2.c b/source3/smbd/trans2.c
index a1043e27ff..da23697771 100644
--- a/source3/smbd/trans2.c
+++ b/source3/smbd/trans2.c
@@ -1669,10 +1669,10 @@ static bool smbd_marshall_dir_entry(TALLOC_CTX *ctx,
 		was_8_3 = mangle_is_8_3(fname, True, conn->params);
 		p += 4;
 		SIVAL(p,0,reskey); p += 4;
-		put_long_date_timespec(p,create_date_ts); p += 8;
-		put_long_date_timespec(p,adate_ts); p += 8;
-		put_long_date_timespec(p,mdate_ts); p += 8;
-		put_long_date_timespec(p,cdate_ts); p += 8;
+		put_long_date_timespec(conn->ts_res,p,create_date_ts); p += 8;
+		put_long_date_timespec(conn->ts_res,p,adate_ts); p += 8;
+		put_long_date_timespec(conn->ts_res,p,mdate_ts); p += 8;
+		put_long_date_timespec(conn->ts_res,p,cdate_ts); p += 8;
 		SOFF_T(p,0,file_size); p += 8;
 		SOFF_T(p,0,allocation_size); p += 8;
 		SIVAL(p,0,nt_extmode); p += 4;
@@ -1735,10 +1735,10 @@ static bool smbd_marshall_dir_entry(TALLOC_CTX *ctx,
 		DEBUG(10,("get_lanman2_dir_entry: SMB_FIND_FILE_DIRECTORY_INFO\n"));
 		p += 4;
 		SIVAL(p,0,reskey); p += 4;
-		put_long_date_timespec(p,create_date_ts); p += 8;
-		put_long_date_timespec(p,adate_ts); p += 8;
-		put_long_date_timespec(p,mdate_ts); p += 8;
-		put_long_date_timespec(p,cdate_ts); p += 8;
+		put_long_date_timespec(conn->ts_res,p,create_date_ts); p += 8;
+		put_long_date_timespec(conn->ts_res,p,adate_ts); p += 8;
+		put_long_date_timespec(conn->ts_res,p,mdate_ts); p += 8;
+		put_long_date_timespec(conn->ts_res,p,cdate_ts); p += 8;
 		SOFF_T(p,0,file_size); p += 8;
 		SOFF_T(p,0,allocation_size); p += 8;
 		SIVAL(p,0,nt_extmode); p += 4;
@@ -1771,10 +1771,10 @@ static bool smbd_marshall_dir_entry(TALLOC_CTX *ctx,
 		DEBUG(10,("get_lanman2_dir_entry: SMB_FIND_FILE_FULL_DIRECTORY_INFO\n"));
 		p += 4;
 		SIVAL(p,0,reskey); p += 4;
-		put_long_date_timespec(p,create_date_ts); p += 8;
-		put_long_date_timespec(p,adate_ts); p += 8;
-		put_long_date_timespec(p,mdate_ts); p += 8;
-		put_long_date_timespec(p,cdate_ts); p += 8;
+		put_long_date_timespec(conn->ts_res,p,create_date_ts); p += 8;
+		put_long_date_timespec(conn->ts_res,p,adate_ts); p += 8;
+		put_long_date_timespec(conn->ts_res,p,mdate_ts); p += 8;
+		put_long_date_timespec(conn->ts_res,p,cdate_ts); p += 8;
 		SOFF_T(p,0,file_size); p += 8;
 		SOFF_T(p,0,allocation_size); p += 8;
 		SIVAL(p,0,nt_extmode); p += 4;
@@ -1846,10 +1846,10 @@ static bool smbd_marshall_dir_entry(TALLOC_CTX *ctx,
 		DEBUG(10,("get_lanman2_dir_entry: SMB_FIND_ID_FULL_DIRECTORY_INFO\n"));
 		p += 4;
 		SIVAL(p,0,reskey); p += 4;
-		put_long_date_timespec(p,create_date_ts); p += 8;
-		put_long_date_timespec(p,adate_ts); p += 8;
-		put_long_date_timespec(p,mdate_ts); p += 8;
-		put_long_date_timespec(p,cdate_ts); p += 8;
+		put_long_date_timespec(conn->ts_res,p,create_date_ts); p += 8;
+		put_long_date_timespec(conn->ts_res,p,adate_ts); p += 8;
+		put_long_date_timespec(conn->ts_res,p,mdate_ts); p += 8;
+		put_long_date_timespec(conn->ts_res,p,cdate_ts); p += 8;
 		SOFF_T(p,0,file_size); p += 8;
 		SOFF_T(p,0,allocation_size); p += 8;
 		SIVAL(p,0,nt_extmode); p += 4;
@@ -1893,10 +1893,10 @@ static bool smbd_marshall_dir_entry(TALLOC_CTX *ctx,
 		was_8_3 = mangle_is_8_3(fname, True, conn->params);
 		p += 4;
 		SIVAL(p,0,reskey); p += 4;
-		put_long_date_timespec(p,create_date_ts); p += 8;
-		put_long_date_timespec(p,adate_ts); p += 8;
-		put_long_date_timespec(p,mdate_ts); p += 8;
-		put_long_date_timespec(p,cdate_ts); p += 8;
+		put_long_date_timespec(conn->ts_res,p,create_date_ts); p += 8;
+		put_long_date_timespec(conn->ts_res,p,adate_ts); p += 8;
+		put_long_date_timespec(conn->ts_res,p,mdate_ts); p += 8;
+		put_long_date_timespec(conn->ts_res,p,cdate_ts); p += 8;
 		SOFF_T(p,0,file_size); p += 8;
 		SOFF_T(p,0,allocation_size); p += 8;
 		SIVAL(p,0,nt_extmode); p += 4;
@@ -3855,9 +3855,9 @@ static char *store_file_unix_basic(connection_struct *conn,
 	SOFF_T(pdata,0,SMB_VFS_GET_ALLOC_SIZE(conn,fsp,psbuf)); /* Number of bytes used on disk - 64 Bit */
 	pdata += 8;
 
-	put_long_date_timespec(pdata, psbuf->st_ex_ctime);       /* Change Time 64 Bit */
-	put_long_date_timespec(pdata+8, psbuf->st_ex_atime);     /* Last access time 64 Bit */
-	put_long_date_timespec(pdata+16, psbuf->st_ex_mtime);    /* Last modification time 64 Bit */
+	put_long_date_timespec(TIMESTAMP_SET_NT_OR_BETTER, pdata, psbuf->st_ex_ctime);       /* Change Time 64 Bit */
+	put_long_date_timespec(TIMESTAMP_SET_NT_OR_BETTER ,pdata+8, psbuf->st_ex_atime);     /* Last access time 64 Bit */
+	put_long_date_timespec(TIMESTAMP_SET_NT_OR_BETTER, pdata+16, psbuf->st_ex_mtime);    /* Last modification time 64 Bit */
 	pdata += 24;
 
 	SIVAL(pdata,0,psbuf->st_ex_uid);               /* user id for the owner */
@@ -3991,7 +3991,7 @@ static char *store_file_unix_basic_info2(connection_struct *conn,
 	pdata = store_file_unix_basic(conn, pdata, fsp, psbuf);
 
 	/* Create (birth) time 64 bit */
-	put_long_date_timespec(pdata, psbuf->st_ex_btime);
+	put_long_date_timespec(TIMESTAMP_SET_NT_OR_BETTER,pdata, psbuf->st_ex_btime);
 	pdata += 8;
 
 	map_info2_flags_from_sbuf(psbuf, &file_flags, &flags_mask);
@@ -4408,10 +4408,10 @@ NTSTATUS smbd_do_qfilepathinfo(connection_struct *conn,
 				data_size = 40;
 				SIVAL(pdata,36,0);
 			}
-			put_long_date_timespec(pdata,create_time_ts);
-			put_long_date_timespec(pdata+8,atime_ts);
-			put_long_date_timespec(pdata+16,mtime_ts); /* write time */
-			put_long_date_timespec(pdata+24,ctime_ts); /* change time */
+			put_long_date_timespec(conn->ts_res,pdata,create_time_ts);
+			put_long_date_timespec(conn->ts_res,pdata+8,atime_ts);
+			put_long_date_timespec(conn->ts_res,pdata+16,mtime_ts); /* write time */
+			put_long_date_timespec(conn->ts_res,pdata+24,ctime_ts); /* change time */
 			SIVAL(pdata,32,mode);
 
 			DEBUG(5,("SMB_QFBI - "));
@@ -4503,10 +4503,10 @@ NTSTATUS smbd_do_qfilepathinfo(connection_struct *conn,
 			unsigned int ea_size =
 			    estimate_ea_size(conn, fsp, smb_fname->base_name);
 			DEBUG(10,("smbd_do_qfilepathinfo: SMB_FILE_ALL_INFORMATION\n"));
-			put_long_date_timespec(pdata,create_time_ts);
-			put_long_date_timespec(pdata+8,atime_ts);
-			put_long_date_timespec(pdata+16,mtime_ts); /* write time */
-			put_long_date_timespec(pdata+24,ctime_ts); /* change time */
+			put_long_date_timespec(conn->ts_res,pdata,create_time_ts);
+			put_long_date_timespec(conn->ts_res,pdata+8,atime_ts);
+			put_long_date_timespec(conn->ts_res,pdata+16,mtime_ts); /* write time */
+			put_long_date_timespec(conn->ts_res,pdata+24,ctime_ts); /* change time */
 			SIVAL(pdata,32,mode);
 			SIVAL(pdata,36,0); /* padding. */
 			pdata += 40;
@@ -4535,10 +4535,10 @@ NTSTATUS smbd_do_qfilepathinfo(connection_struct *conn,
 			unsigned int ea_size =
 			    estimate_ea_size(conn, fsp, smb_fname->base_name);
 			DEBUG(10,("smbd_do_qfilepathinfo: SMB2_FILE_ALL_INFORMATION\n"));
-			put_long_date_timespec(pdata+0x00,create_time_ts);
-			put_long_date_timespec(pdata+0x08,atime_ts);
-			put_long_date_timespec(pdata+0x10,mtime_ts); /* write time */
-			put_long_date_timespec(pdata+0x18,ctime_ts); /* change time */
+			put_long_date_timespec(conn->ts_res,pdata+0x00,create_time_ts);
+			put_long_date_timespec(conn->ts_res,pdata+0x08,atime_ts);
+			put_long_date_timespec(conn->ts_res,pdata+0x10,mtime_ts); /* write time */
+			put_long_date_timespec(conn->ts_res,pdata+0x18,ctime_ts); /* change time */
 			SIVAL(pdata,	0x20, mode);
 			SIVAL(pdata,	0x24, 0); /* padding. */
 			SBVAL(pdata,	0x28, allocation_size);
@@ -4668,10 +4668,10 @@ NTSTATUS smbd_do_qfilepathinfo(connection_struct *conn,
 
 		case SMB_FILE_NETWORK_OPEN_INFORMATION:
 			DEBUG(10,("smbd_do_qfilepathinfo: SMB_FILE_NETWORK_OPEN_INFORMATION\n"));
-			put_long_date_timespec(pdata,create_time_ts);
-			put_long_date_timespec(pdata+8,atime_ts);
-			put_long_date_timespec(pdata+16,mtime_ts); /* write time */
-			put_long_date_timespec(pdata+24,ctime_ts); /* change time */
+			put_long_date_timespec(conn->ts_res,pdata,create_time_ts);
+			put_long_date_timespec(conn->ts_res,pdata+8,atime_ts);
+			put_long_date_timespec(conn->ts_res,pdata+16,mtime_ts); /* write time */
+			put_long_date_timespec(conn->ts_res,pdata+24,ctime_ts); /* change time */
 			SOFF_T(pdata,32,allocation_size);
 			SOFF_T(pdata,40,file_size);
 			SIVAL(pdata,48,mode);
@@ -5402,25 +5402,13 @@ NTSTATUS smb_set_file_time(connection_struct *conn,
 		action &= ~FILE_NOTIFY_CHANGE_LAST_WRITE;
 	}
 
-	if (!conn->hires_timestamps_avail) {
-		/* We can't store sub second timestamps
-		 * on this share. Round to seconds. */
-		round_timespec(&ft->create_time);
-		round_timespec(&ft->ctime);
-		round_timespec(&ft->atime);
-		round_timespec(&ft->mtime);
-	} else {
-		/* The highest resolution timestamp
- 		 * setting function available in POSIX
- 		 * is utimes(), which uses usec resolution,
- 		 * not nsec resolution. So we must round to
- 		 * usec, then back to nsec. JRA.
- 		 */
-		round_timespec_to_usec(&ft->create_time);
-		round_timespec_to_usec(&ft->ctime);
-		round_timespec_to_usec(&ft->atime);
-		round_timespec_to_usec(&ft->mtime);
-	}
+	/* Ensure the resolution is the correct for
+	 * what we can store on this filesystem. */
+
+	round_timespec(conn->ts_res, &ft->create_time);
+	round_timespec(conn->ts_res, &ft->ctime);
+	round_timespec(conn->ts_res, &ft->atime);
+	round_timespec(conn->ts_res, &ft->mtime);
 
 	DEBUG(5,("smb_set_filetime: actime: %s\n ",
 		time_to_asc(convert_timespec_to_time_t(ft->atime))));