[LU-6512] ll_dir_ioctl has some pretty strange code now.

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.9.0
• Labels: easy patch
• Severity: 3
• Assignee: WC Triage
• Reporter: Oleg Drokin

Description

smatch highlighted thid interesting piece in ll_dir_ioctl that results from a bit of code change by commit 60e07b972114df24105a3a1bfa7365892f72a4a7 LU-4690:

                rc = ll_dir_getstripe(inode, (void **)&lmm, &lmmsize, &request,
                                      valid);
                if (rc != 0)
                        GOTO(finish_req, rc);

                /* Get default LMV EA */
                if (lum.lum_magic == LMV_USER_MAGIC) {
                        if (rc != 0)
                                GOTO(finish_req, rc);

note the two identical comparisons.
Should we take the other one down?