[LU-10883] llog backup files processing is not skipped because of string сomparison error

• Status: Resolved
• Project: Lustre
• Affects Version/s: Lustre 2.11.0
• Fix Version/s: Lustre 2.13.0
• Type: Bug
• Priority: Major
• Reporter: Artem Blagodarenko (Inactive)
• Assignee: Artem Blagodarenko (Inactive)
• Resolution: Fixed
• Severity: 3

Description

Buggy code was introduced in LU-7160 fix

		/* filter out ".bak" files */
		/* sizeof(".bak") - 1 == 3 */
		if (key_sz >= 3 &&
		    !memcmp(".bak", key + key_sz - 3, 3)) {
			CDEBUG(D_MGS, "Skipping backup file %.*s\n",
			       key_sz, key);
			goto next;
		}

4 last key's bytes need to be compared with ".bak"

Issue Links

• is related to: LU-7160 Saved by change_nids .bak files on MG...
• is related to: LU-11648 Create white list for CONFIGS directo...

Comments

[Gerrit Updater]

Artem Blagodarenko ([e-mail]) uploaded a new patch: https://review.whamcloud.com/31886
Subject: LU-10883 llog: fix backup file name comparison
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: e494b6f87ae2ef80c62b4fda2fd461d7214143c2

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/31886/
Subject: LU-10883 llog: fix backup file name comparison
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 5bc7ee090d6475ccd60785e120a21ecbbed3baa4

[Peter Jones]

Landed for 2.13