[LU-5466] suspicious else statement in lfsck_del_target()

Mon, 25 Aug 2014 23:19:02 +0000

@@ -2684,10 +2684,6 @@ void lfsck_del_target(const struct lu_en

 unlock:
        if (ltd == NULL) {
*               if (for_ost)
*                       head = &lfsck->li_ost_descs.ltd_orphan;
*               else
*                       head = &lfsck->li_ost_descs.ltd_orphan;

                list_for_each_entry(ltd, head, ltd_orphan_list) {
                        if (ltd->ltd_tgt == tgt) {

Found by the coccinelle script:

// If branch and else branch have same statement.
// Based on a coverity check mentioned in a blog post by Dave Jones.

@@
statement S;
@@

* if (...) S else S

Whamcloud Community JIRA

[LU-5466] suspicious else statement in lfsck_del_target()