glusterfs icon indicating copy to clipboard operation
glusterfs copied to clipboard

dht: fix use after free issue in dht_setxattr_mds_cbk

Open mohit84 opened this issue 2 years ago • 4 comments

It is a day one bug when a feature was implemented, the issue was not caught earlier because the client crash depends on the environment.The client was crashing because during wind it was not passing cookie and in cbk it was trying to access cookie.

Solution: Pass a xlator_t as a cookie during STACK_WIND_COOKIE to avoid a crash.

Fixes: #3732 Change-Id: I5fd75e90a94d852093dfa40fb37d4211b4e9fdfb Signed-off-by: Mohit Agrawal [email protected]

mohit84 avatar Aug 25 '22 13:08 mohit84

/run regression

mohit84 avatar Aug 25 '22 13:08 mohit84

/run regression

mohit84 avatar Aug 25 '22 15:08 mohit84

1 test(s) failed ./tests/basic/distribute/spare_file_rebalance.t

0 test(s) generated core

8 test(s) needed retry ./tests/000-flaky/basic_distribute_rebal-all-nodes-migrate.t ./tests/000-flaky/basic_mount-nfs-auth.t ./tests/000-flaky/bugs_distribute_bug-1117851.t ./tests/000-flaky/bugs_glusterd_bug-857330/normal.t ./tests/000-flaky/bugs_glusterd_bug-857330/xml.t ./tests/000-flaky/bugs_nfs_bug-1116503.t ./tests/000-flaky/glusterd-restart-shd-mux.t ./tests/basic/distribute/spare_file_rebalance.t

7 flaky test(s) marked as success even though they failed ./tests/000-flaky/basic_distribute_rebal-all-nodes-migrate.t ./tests/000-flaky/basic_mount-nfs-auth.t ./tests/000-flaky/bugs_distribute_bug-1117851.t ./tests/000-flaky/bugs_glusterd_bug-857330/normal.t ./tests/000-flaky/bugs_glusterd_bug-857330/xml.t ./tests/000-flaky/bugs_nfs_bug-1116503.t ./tests/000-flaky/glusterd-restart-shd-mux.t https://build.gluster.org/job/gh_centos7-regression/2744/

gluster-ant avatar Aug 25 '22 22:08 gluster-ant

/run regression

mohit84 avatar Aug 26 '22 06:08 mohit84

Hi @mohit84 ,

I am working on CVE-2022-48340. So I just want to check if there any update on this? Or any other commit that can fix this issue.

SoumyaWind avatar Apr 05 '23 04:04 SoumyaWind

Any update on this? @mohit84

rohitkuma1313 avatar Aug 04 '23 09:08 rohitkuma1313

!!! Couldn't read commit file !!!

gluster-ant avatar Oct 20 '23 06:10 gluster-ant