smbd: Do not fetch the record in defer_open_done
authorVolker Lendecke <vl@samba.org>
Fri, 26 Apr 2013 13:13:14 +0000 (15:13 +0200)
committerJeremy Allison <jra@samba.org>
Sat, 27 Apr 2013 00:06:10 +0000 (02:06 +0200)
Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Sat Apr 27 02:06:10 CEST 2013 on sn-devel-104

source3/smbd/open.c

index eff36f002c4930ba70d3ae136406cc28d7cd6288..7d02e521081467bb32d5d4c7ca8561e69ac6ae77 100644 (file)
@@ -1507,11 +1507,10 @@ static void defer_open_done(struct tevent_req *req)
 {
        struct defer_open_state *state = tevent_req_callback_data(
                req, struct defer_open_state);
-       struct db_record *rec = NULL;
        NTSTATUS status;
        bool ret;
 
-       status = dbwrap_record_watch_recv(req, talloc_tos(), &rec);
+       status = dbwrap_record_watch_recv(req, talloc_tos(), NULL);
        TALLOC_FREE(req);
        if (!NT_STATUS_IS_OK(status)) {
                DEBUG(5, ("dbwrap_record_watch_recv returned %s\n",
@@ -1522,12 +1521,6 @@ static void defer_open_done(struct tevent_req *req)
                 */
        }
 
-       /*
-        * TODO: We need a version of dbwrap_record_watch_recv that does not
-        * fetch_lock the record.
-        */
-       TALLOC_FREE(rec);
-
        DEBUG(10, ("scheduling mid %llu\n", (unsigned long long)state->mid));
 
        ret = schedule_deferred_open_message_smb(state->sconn, state->mid);