]> git.ipfire.org Git - thirdparty/samba.git/commitdiff
smbd: Remove unused _prev_offset arg from smbd_dirptr_get_entry()
authorVolker Lendecke <vl@samba.org>
Fri, 9 Jun 2023 13:05:07 +0000 (15:05 +0200)
committerJeremy Allison <jra@samba.org>
Tue, 13 Jun 2023 23:33:39 +0000 (23:33 +0000)
Neither of the two callers looked at the result.

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
source3/smbd/dir.c
source3/smbd/globals.h
source3/smbd/smb1_reply.c
source3/smbd/smb2_trans2.c

index 4635e8bbf397fda0e4df7869a3a687c4a616cfd8..02e7181c3b832d9b4ebd6e8a33fdfadcf59bbee5 100644 (file)
@@ -562,8 +562,7 @@ bool smbd_dirptr_get_entry(TALLOC_CTX *ctx,
                           void *private_data,
                           char **_fname,
                           struct smb_filename **_smb_fname,
-                          uint32_t *_mode,
-                          long *_prev_offset)
+                          uint32_t *_mode)
 {
        connection_struct *conn = dirptr->conn;
        struct smb_Dir *dir_hnd = dirptr->dir_hnd;
@@ -589,7 +588,6 @@ bool smbd_dirptr_get_entry(TALLOC_CTX *ctx,
 
        while (true) {
                long cur_offset;
-               long prev_offset;
                SMB_STRUCT_STAT sbuf = { 0 };
                char *dname = NULL;
                bool isdots;
@@ -603,7 +601,6 @@ bool smbd_dirptr_get_entry(TALLOC_CTX *ctx,
                bool ok;
 
                cur_offset = dptr_TellDir(dirptr);
-               prev_offset = cur_offset;
                dname = dptr_ReadDirName(ctx, dirptr, &cur_offset, &sbuf);
 
                DBG_DEBUG("dir [%s] dirptr [0x%lx] offset [%ld] => "
@@ -888,7 +885,6 @@ bool smbd_dirptr_get_entry(TALLOC_CTX *ctx,
                }
                *_fname = fname;
                *_mode = mode;
-               *_prev_offset = prev_offset;
 
                return true;
        }
index c58123ee501d33ea81805aa4bd938a7c370f14d4..3d9f046a15be32371dba90c7ebe5a8aa3f33eb24 100644 (file)
@@ -181,8 +181,7 @@ bool smbd_dirptr_get_entry(TALLOC_CTX *ctx,
                           void *private_data,
                           char **_fname,
                           struct smb_filename **_smb_fname,
-                          uint32_t *_mode,
-                          long *_prev_offset);
+                          uint32_t *_mode);
 void smbd_dirptr_push_overflow(struct dptr_struct *dirptr,
                               char **_fname,
                               struct smb_filename **_smb_fname,
index e96a1c8ca404108471f858dcf5bdfa4560a47c4d..7b6bd6c1f56e3066dd4d458ae724bf46550ccab2 100644 (file)
@@ -1241,7 +1241,6 @@ static bool get_dir_entry(TALLOC_CTX *ctx,
        char *fname = NULL;
        struct smb_filename *smb_fname = NULL;
        uint32_t mode = 0;
-       long prev_offset;
        bool ok;
 
        ok = smbd_dirptr_get_entry(ctx,
@@ -1256,8 +1255,7 @@ static bool get_dir_entry(TALLOC_CTX *ctx,
                                   conn,
                                   &fname,
                                   &smb_fname,
-                                  &mode,
-                                  &prev_offset);
+                                  &mode);
        if (!ok) {
                return false;
        }
index eca59125723090192529661ee460d4cee7168dd2..9b72d8637528949cff061d133eda5cd5a2b4993f 100644 (file)
@@ -1919,7 +1919,6 @@ NTSTATUS smbd_dirptr_lanman2_entry(TALLOC_CTX *ctx,
 {
        const char *p;
        const char *mask = NULL;
-       long prev_dirpos = 0;
        uint32_t mode = 0;
        char *fname = NULL;
        struct smb_filename *smb_fname = NULL;
@@ -1967,8 +1966,7 @@ NTSTATUS smbd_dirptr_lanman2_entry(TALLOC_CTX *ctx,
                                   &state,
                                   &fname,
                                   &smb_fname,
-                                  &mode,
-                                  &prev_dirpos);
+                                  &mode);
        if (!ok) {
                return NT_STATUS_END_OF_FILE;
        }