[LU-15901] mdc_changelog_cdev_init leaves freed items on the list

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.16.0
• Affects Version/s: Lustre 2.16.0, Lustre 2.12.9, Lustre 2.15.0
• Severity: 3
• Epic Link: FLR tech debt review
• Assignee: Oleg Drokin
• Reporter: Oleg Drokin

Description

        list_add_tail(&obd->u.cli.cl_chg_dev_linkage, &entry->ced_obds);
        list_add_tail(&entry->ced_link, &chlg_registered_devices);

        rc = chlg_minor_alloc(&minor);
        if (rc)
                GOTO(out_unlock, rc);
...
out_minor:
        chlg_minor_free(minor);
        list_del_init(&obd->u.cli.cl_chg_dev_linkage);
        list_del(&entry->ced_link);
out_unlock:
        mutex_unlock(&chlg_registered_dev_lock);
        if (entry)
                OBD_FREE_PTR(entry);
        RETURN(rc);
 

Comments

[Peter Jones]

Landed for 2.16

[Gerrit Updater]

"Oleg Drokin <[e-mail]>" merged in patch https://review.whamcloud.com/47480/
Subject: LU-15901 mdc: Remove entry from list before freeing
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 441ec2296a0938dd30cf60a70943eb0799d1d70c

[Gerrit Updater]

"Oleg Drokin <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/47480
Subject: LU-15901 mdc: Remvoe entry from list before freeing
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 322e092b8412ca787c22b9edfe8b202ab9691c4c