Revert "smbd: explain that/why we use the raw tevent_context for update_write_time_ha...

This reverts commit 7cfafaf190643eb28fc95d21a02a4e5e529e16d1.

See the discussion in

https://lists.samba.org/archive/samba-technical/2018-December/131731.html

for the reasoning behind this revert.

Signed-off-by: Ralph Boehme <slow@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>

diff --git a/source3/smbd/fileio.c b/source3/smbd/fileio.c
index 1fe806e058d28db895501bd67ba751c6902d9755..cde6a057ccdafc65f1148a4a43976330421c6a64 100644 (file)
--- a/source3/smbd/fileio.c
+++ b/source3/smbd/fileio.c
@@ -232,17 +232,9 @@ void trigger_write_time_update(struct files_struct *fsp)
        DEBUG(5, ("Update write time %d usec later on %s\n",
                  delay, fsp_str_dbg(fsp)));
 
-       /*
-        * trigger the update 2 seconds later
-        *
-        * Note that update_write_time_handler()
-        * => fsp_flush_write_time_update()
-        * won't do any SMB_VFS calls and don't
-        * need impersonation. So we use the
-        * raw event context for this.
-        */
+       /* trigger the update 2 seconds later */
        fsp->update_write_time_event =
-               tevent_add_timer(fsp->conn->sconn->raw_ev_ctx, NULL,
+               tevent_add_timer(fsp->conn->sconn->ev_ctx, NULL,
                                 timeval_current_ofs_usec(delay),
                                 update_write_time_handler, fsp);
 }