From 6134922f7b4d0141e40286c5c5acad05272fa267 Mon Sep 17 00:00:00 2001 From: Jeremy Allison Date: Mon, 8 Jul 2019 15:26:59 -0700 Subject: [PATCH] s3: smbd: Remove the 'close oldest' logic in SMB1 trans2 search processing. Will be removed in next commit. Signed-off-by: Jeremy Allison Reviewed-by: Andreas Schneider --- source3/smbd/dir.c | 27 ++++++++------------------- 1 file changed, 8 insertions(+), 19 deletions(-) diff --git a/source3/smbd/dir.c b/source3/smbd/dir.c index a2e0687da50..62331da96e3 100644 --- a/source3/smbd/dir.c +++ b/source3/smbd/dir.c @@ -288,6 +288,7 @@ void dptr_closepath(struct smbd_server_connection *sconn, } } +#if 0 /**************************************************************************** Try and close the oldest handle not marked for expect close in the hope that the client has @@ -324,6 +325,7 @@ static void dptr_close_oldest(struct smbd_server_connection *sconn, } } } +#endif /**************************************************************************** Safely do an OpenDir as root, ensuring we're in the right place. @@ -544,25 +546,12 @@ NTSTATUS dptr_create(connection_struct *conn, dptr->dnum = bitmap_find(sconn->searches.dptr_bmap, 255); if(dptr->dnum == -1 || dptr->dnum < 255) { - - /* - * Try and close the oldest handle close in the hope that - * the client has finished with that one. This will only - * happen in the case of the Win98 client bug where it leaks - * directory handles. - */ - - dptr_close_oldest(sconn, false); - - /* Now try again... */ - dptr->dnum = bitmap_find(sconn->searches.dptr_bmap, 255); - - if(dptr->dnum == -1 || dptr->dnum < 255) { - DEBUG(0,("dptr_create: returned %d: Error - all new dirptrs in use ?\n", dptr->dnum)); - TALLOC_FREE(dptr); - TALLOC_FREE(dir_hnd); - return NT_STATUS_TOO_MANY_OPENED_FILES; - } + DBG_ERR("returned %d: Error - all new " + "dirptrs in use ?\n", + dptr->dnum); + TALLOC_FREE(dptr); + TALLOC_FREE(dir_hnd); + return NT_STATUS_TOO_MANY_OPENED_FILES; } } -- 2.47.3