[LU-10141] Integer overflow in llapi_changelog_start

• Type: Improvement
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.11.0, Lustre 2.10.2
• Labels: patch
• Assignee: Henri Doreau (Inactive)
• Reporter: Henri Doreau (Inactive)

Description

When reading changelogs from a high (more than 32bits) offset, the return value from lseek() is not properly interpreted and spurious errors can occur.

ex. w/ robinhood:
ERROR -95 opening changelog for MDT 'lustre-MDT0000': Operation not supported

Issue Links

• is related to: LU-7659 Replace KUC by more standard mechanisms

Comments

[Gerrit Updater]

John L. Hammond ([e-mail]) merged in patch https://review.whamcloud.com/29735/
Subject: LU-10141 llapi: integer overflow in llapi_changelog_start
Project: fs/lustre-release
Branch: b2_10
Current Patch Set:
Commit: 7736aa81d1633ce4c710f842a88b65a8be84029e

[Gerrit Updater]

Minh Diep ([e-mail]) uploaded a new patch: https://review.whamcloud.com/29735
Subject: LU-10141 llapi: integer overflow in llapi_changelog_start
Project: fs/lustre-release
Branch: b2_10
Current Patch Set: 1
Commit: 4004ef114f634b2b6a8926e1ec86592d84acdef7

[Peter Jones]

Landed for 2.11

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/29674/
Subject: LU-10141 llapi: integer overflow in llapi_changelog_start
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: aaa808e7a40198581956a1d648f413cc30e830ed

[Peter Jones]

ok Henri - I have marked it as such

[Henri Doreau (Inactive)]

For information: the regression has been introduced by yours truly in 2.10 (https://review.whamcloud.com/#/c/18900/).

I think that once we have a fix landed, a backport to 2.10 would be needed.

[Peter Jones]

Thanks Henri

[Gerrit Updater]

Henri Doreau ([e-mail]) uploaded a new patch: https://review.whamcloud.com/29674
Subject: LU-10141 llapi: integer overflow in llapi_changelog_start
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 3b6f2b425d2443090b57d7c531f61a4f1479c84f