[Devel] [PATCH RHEL9 COMMIT] fuse: pcs: dangerous typo in commit_sync_info()

Konstantin Khorenko khorenko at virtuozzo.com
Thu Nov 23 16:20:38 MSK 2023


The commit is pushed to "branch-rh9-5.14.0-284.25.1.vz9.30.x-ovz" and will appear at https://src.openvz.org/scm/ovz/vzkernel.git
after rh9-5.14.0-284.25.1.vz9.30.12
------>
commit e06e4c935e961ad96d926bbe59137c9bf0c28fa9
Author: Alexey Kuznetsov <kuznet at virtuozzo.com>
Date:   Thu Nov 23 17:54:01 2023 +0800

    fuse: pcs: dangerous typo in commit_sync_info()
    
    Unpleasant, shows the code never was in this place before.
    
    Fixes: 3202fa19f30e ("fuse: a protocol to reenable optimizations after
    replication finished")
    
    https://pmc.acronis.work/browse/VSTOR-77923
    
    Signed-off-by: Alexey Kuznetsov <kuznet at acronis.com>
---
 fs/fuse/kio/pcs/pcs_map.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/fuse/kio/pcs/pcs_map.c b/fs/fuse/kio/pcs/pcs_map.c
index ae135573d806..eabad8a9c1fe 100644
--- a/fs/fuse/kio/pcs/pcs_map.c
+++ b/fs/fuse/kio/pcs/pcs_map.c
@@ -2877,7 +2877,7 @@ static int commit_sync_info(struct pcs_int_request *req,
 			int idx;
 			clear_bit(CS_SF_REPLICATING, &csl->cs[read_idx].cslink.cs->state);
 			clear_bit(read_idx, &csl->blacklist);
-			for (idx = csl->nsrv - 1; idx >= 0; idx++) {
+			for (idx = csl->nsrv - 1; idx >= 0; idx--) {
 				if (test_bit(CS_SF_REPLICATING, &csl->cs[idx].cslink.cs->state))
 					break;
 			}


More information about the Devel mailing list