[LU-6872] Infinite loop in lsm_alloc_plain on error cleanup

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.8.0
• Labels: patch
• Severity: 3
• Assignee: John Hammond
• Reporter: Oleg Drokin

Description

A recent lov cleanup (patch 5ccd7a4a556b1a847eb5bff8b2395522a6f4bca8 http://review.whamcloud.com/13722 by John Hammond) has introduced an infinite loop in lsm_alloc_plain by changing type of variable 'i' from signed to unsigned:

struct lov_stripe_md *lsm_alloc_plain(u16 stripe_count)
{
...
        unsigned int i;
...
err:
        while (--i >= 0)
                OBD_SLAB_FREE(lsm->lsm_oinfo[i], lov_oinfo_slab, sizeof(*loi));

Now the while loop is infinite.