[LU-5431] lmv_intent_lookup() leaks lock if lmv_revalidate_slaves() fails

Mon, 27 Apr 2015 20:41:10 +0000

In lmv_intent_lookup() if md_intent_lock() succeeds but lmv_revalidate_slaves() fails then the lock referenced by it->d.lustre.it_lock_handle is leaked.

        rc = md_intent_lock(tgt->ltd_exp, op_data, it, reqp, cb_blocking,
                            extra_lock_flags);
        if (rc < 0)
                RETURN(rc);

        if (*reqp == NULL) {
                /* If RPC happens, lsm information will be revalidated                    
                 * during update_inode process (see ll_update_lsm_md) */
                if (op_data->op_mea2 != NULL) {
                        rc = lmv_revalidate_slaves(exp, NULL, op_data->op_mea2,
                                                   cb_blocking,
                                                   extra_lock_flags);
                        if (rc != 0)
                                RETURN(rc);
                }
                RETURN(rc);
        }

This was found through manual code audit.

Whamcloud Community JIRA

[LU-5431] lmv_intent_lookup() leaks lock if lmv_revalidate_slaves() fails