[MDEV-17957] Make Innodb_checksum_algorithm stricter for strict_* values Created: 2018-12-10 Updated: 2020-08-25 Resolved: 2018-12-13 |
|
| Status: | Closed |
| Project: | MariaDB Server |
| Component/s: | Storage Engine - InnoDB, Storage Engine - XtraDB |
| Affects Version/s: | 10.1.37, 10.2, 10.3 |
| Fix Version/s: | 10.4.1, 10.1.38, 10.0.38, 10.2.20, 10.3.12 |
| Type: | Bug | Priority: | Major |
| Reporter: | Thirunarayanan Balathandayuthapani | Assignee: | Marko Mäkelä |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Issue Links: |
|
||||||||||||||||||||||||
| Description |
|
innodb_checksum_algorithm checks for the another algorithm even though the value is |
| Comments |
| Comment by Thirunarayanan Balathandayuthapani [ 2018-12-10 ] | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Patch is present in bb-10.1-thiru https://github.com/MariaDB/server/commit/6c8538ac219cd9248ddaaaa59251de937d354b79 | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Marko Mäkelä [ 2018-12-11 ] | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Thanks! I think that we can simplify this a little more by assuming that when the crc32 checksum is being used, it has been written by MySQL 5.6.3 or later. Please submit a 10.0 version as well, and observe my review comments in GitHub. For the 10.1 version, please implement strict checksum variants also for fil_space_verify_crypt_checksum(). | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Thirunarayanan Balathandayuthapani [ 2018-12-11 ] | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Put the patch for 10.0 and addressed the review comments for 10.1 patch as well. | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Thirunarayanan Balathandayuthapani [ 2018-12-13 ] | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
In 10.2, there are few dead code:
Another issue with page_zip_verify_checksum():
| |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Thirunarayanan Balathandayuthapani [ 2018-12-13 ] | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Fixed the above issues in bb-10.2-23578 branch. | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Marko Mäkelä [ 2018-12-13 ] | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
I merged this up to 10.2 until now. |