Changeset 78355 in vbox for trunk/src/VBox/Additions/WINNT/SharedFolders/driver/file.cpp

Timestamp:

Apr 30, 2019 3:48:46 AM (6 years ago)

Author:

vboxsync

Message:

winnt/vboxsf: Reworked the set-timestamp handling. Need testcase and some host service tinkering. Also optimized handle closing. bugref:9172

File:

• trunk/src/VBox/Additions/WINNT/SharedFolders/driver/file.cpp (modified) (4 diffs)

trunk/src/VBox/Additions/WINNT/SharedFolders/driver/file.cpp

@@ -234 +234 @@
 
     PMRX_VBOX_NETROOT_EXTENSION pNetRootExtension = VBoxMRxGetNetRootExtension(capFcb->pNetRoot);
+    PVBSFNTFCBEXT pVBoxFcbx = VBoxMRxGetFcbExtension(capFcb);
     PMRX_VBOX_FOBX pVBoxFobx = VBoxMRxGetFileObjectExtension(capFobx);
 
@@ -308 +309 @@
     Status = vbsfNtVBoxStatusToNt(vrc);
 
-    if (Status != STATUS_SUCCESS)
-    {
-        /* Nothing read. */
-        ByteCount = 0;
-    }
+    if (Status == STATUS_SUCCESS)
+    {
+        pVBoxFobx->fTimestampsImplicitlyUpdated |= VBOX_FOBX_F_INFO_LASTACCESS_TIME;
+        if (pVBoxFcbx->pFobxLastAccessTime != pVBoxFobx)
+            pVBoxFcbx->pFobxLastAccessTime = NULL;
+    }
+    else
+        ByteCount = 0; /* Nothing read. */
 
     RxContext->InformationToReturn = ByteCount;
@@ -378 +382 @@
 
     PMRX_VBOX_NETROOT_EXTENSION pNetRootExtension = VBoxMRxGetNetRootExtension(capFcb->pNetRoot);
+    PVBSFNTFCBEXT pVBoxFcbx = VBoxMRxGetFcbExtension(capFcb);
     PMRX_VBOX_FOBX pVBoxFobx = VBoxMRxGetFileObjectExtension(capFobx);
 
@@ -429 +434 @@
     Status = vbsfNtVBoxStatusToNt(vrc);
 
-    if (Status != STATUS_SUCCESS)
-    {
-        /* Nothing written. */
-        ByteCount = 0;
-    }
+    if (Status == STATUS_SUCCESS)
+    {
+        pVBoxFobx->fTimestampsImplicitlyUpdated |= VBOX_FOBX_F_INFO_LASTWRITE_TIME;
+        if (pVBoxFcbx->pFobxLastWriteTime != pVBoxFobx)
+            pVBoxFcbx->pFobxLastWriteTime = NULL;
+    }
+    else
+        ByteCount = 0; /* Nothing written. */
 
     RxContext->InformationToReturn = ByteCount;