[LU-10141] Integer overflow in llapi_changelog_start Created: 19/Oct/17 Updated: 15/Nov/17 Resolved: 24/Oct/17 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | Lustre 2.11.0, Lustre 2.10.2 |
| Type: | Improvement | Priority: | Minor |
| Reporter: | Henri Doreau (Inactive) | Assignee: | Henri Doreau (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | patch | ||
| Issue Links: |
|
||||||||
| Rank (Obsolete): | 9223372036854775807 | ||||||||
| 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: |
| Comments |
| Comment by Gerrit Updater [ 19/Oct/17 ] |
|
Henri Doreau (henri.doreau@cea.fr) uploaded a new patch: https://review.whamcloud.com/29674 |
| Comment by Peter Jones [ 19/Oct/17 ] |
|
Thanks Henri |
| Comment by Henri Doreau (Inactive) [ 19/Oct/17 ] |
|
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. |
| Comment by Peter Jones [ 19/Oct/17 ] |
|
ok Henri - I have marked it as such |
| Comment by Gerrit Updater [ 24/Oct/17 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/29674/ |
| Comment by Peter Jones [ 24/Oct/17 ] |
|
Landed for 2.11 |
| Comment by Gerrit Updater [ 24/Oct/17 ] |
|
Minh Diep (minh.diep@intel.com) uploaded a new patch: https://review.whamcloud.com/29735 |
| Comment by Gerrit Updater [ 15/Nov/17 ] |
|
John L. Hammond (john.hammond@intel.com) merged in patch https://review.whamcloud.com/29735/ |