[LU-10220] Incorrect record length allocated in mdd_changelog_data_store_by_fid() Created: 09/Nov/17 Updated: 06/Dec/17 Resolved: 01/Dec/17 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | Lustre 2.10.1 |
| Fix Version/s: | Lustre 2.11.0, Lustre 2.10.2 |
| Type: | Bug | Priority: | Major |
| Reporter: | Sebastien Buisson (Inactive) | Assignee: | Sebastien Buisson (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | changelogs, patch | ||
| Issue Links: |
|
||||
| Severity: | 3 | ||||
| Rank (Obsolete): | 9223372036854775807 | ||||
| Description |
|
In mdd_changelog_data_store_by_fid(), the record length calculated to allocate mti_big_buf is not correct, because it forgets to take the header size of struct llog_changelog_rec into account. I guess it never generated memory corruptions before, because the buffer size allocated in a previous call to mdd_declare_changelog_store() was covering the need. I will submit a patch to fix this problem. Sebastien. |
| Comments |
| Comment by Gerrit Updater [ 09/Nov/17 ] |
|
Sebastien Buisson (sbuisson@ddn.com) uploaded a new patch: https://review.whamcloud.com/30014 |
| Comment by Gerrit Updater [ 01/Dec/17 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/30014/ |
| Comment by Peter Jones [ 01/Dec/17 ] |
|
Landed for 2.11 |
| Comment by Gerrit Updater [ 04/Dec/17 ] |
|
Minh Diep (minh.diep@intel.com) uploaded a new patch: https://review.whamcloud.com/30356 |
| Comment by Gerrit Updater [ 06/Dec/17 ] |
|
John L. Hammond (john.hammond@intel.com) merged in patch https://review.whamcloud.com/30356/ |