gluster / glusterfs

Gluster Filesystem : Build your distributed storage in minutes
https://www.gluster.org
GNU General Public License v2.0
4.51k stars 1.07k forks source link

Fixes: #3983 #4311

Closed lvtao-sec closed 2 months ago

lvtao-sec commented 2 months ago

Resubmit the patch (submitted once in #4289) by adding "Fixes: #3983" in the commit.

gluster-ant commented 2 months ago

Can one of the admins verify this patch?

gluster-ant commented 2 months ago

Can one of the admins verify this patch?

gluster-ant commented 2 months ago

Can one of the admins verify this patch?

gluster-ant commented 2 months ago

CLANG-FORMAT FAILURE: Before merging the patch, this diff needs to be considered for passing clang-format

index 911403ed3..c60cd4e6d 100644
--- a/xlators/storage/posix/src/posix-inode-fd-ops.c
+++ b/xlators/storage/posix/src/posix-inode-fd-ops.c
@@ -6036,10 +6036,7 @@ posix_readdirp(call_frame_t *frame, xlator_t *this, fd_t *fd, size_t size,
         if (op_ret >= 0) {
             op_ret = 0;

-            list_for_each_entry(entry, &entries.list, list)
-            {
-                op_ret++;
-            }
+            list_for_each_entry(entry, &entries.list, list) { op_ret++; }
         }

         STACK_UNWIND_STRICT(readdirp, frame, op_ret, op_errno, &entries, NULL);
aravindavk commented 2 months ago

/run regression