[LU-8014] potential memory dereference in kuc_ispayload()

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.9.0
• Labels: patch
• Severity: 3
• Assignee: Frank Zago (Inactive)
• Reporter: Frank Zago (Inactive)

Description

That function takes a pointer, decrements it and dereference the new address. However we have no idea if the page it's in is readable or even exists. Such dereference would cause an oops.

Comments

[Peter Jones]

Both patches now landed for 2.9

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch http://review.whamcloud.com/19869/
Subject: LU-8014 hsm: remove invalid kuc_free in coordinator
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 12ac7f2baa3691eaf74d468aedb7615f6ddf6d60

[Andreas Dilger]

Reopen to track http://review.whamcloud.com/19869 landing.

[Gerrit Updater]

Frank Zago ([e-mail]) uploaded a new patch: http://review.whamcloud.com/19869
Subject: LU-8014 hsm: remove invalid kuc_free in coordinator
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 1db3aa54f1f41719bf1a4bec2e6d172734959c00

[Joseph Gmitter (Inactive)]

Landed to master for 2.9.0

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch http://review.whamcloud.com/19494/
Subject: LU-8014 hsm: remove kuc_ispayload
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 9091a934fcff06538e277d87c3953775e692d9fa

[Frank Zago (Inactive)]

An optimization would be to use kuc_alloc earlier to allocate the hal, and change mdt_hsm_agent_send to take a KUC buffer instead of a hal.

[Gerrit Updater]

Frank Zago ([e-mail]) uploaded a new patch: http://review.whamcloud.com/19494
Subject: LU-8014 hsm: remove kuc_ispayload
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 70784451511e39c21863268c63c89a10e259cddd