Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

offset-extract.sh failed for MySQL >= 8.0.32 #266

Open
yoku0825 opened this issue Nov 9, 2023 · 4 comments
Open

offset-extract.sh failed for MySQL >= 8.0.32 #266

yoku0825 opened this issue Nov 9, 2023 · 4 comments

Comments

@yoku0825
Copy link

yoku0825 commented Nov 9, 2023

$ bash offset-extract/offset-extract.sh /usr/mysql/8.0.35/bin/mysqld
//offsets for: /usr/mysql/8.0.35/bin/mysqld (8.0.35)
{"8.0.35","80e986a85d7d128ef07b8229193e98e6", 9496, 9536, 4952, 6436, 1288, 0, 0, 32, 64, 160, 1376, 9636, 6056, 4248, 4256, 4260, 7720, 1576, 32, 8680, 8720, 8704, 12880, 140, 664offsets.gdb:33: Error in sourced command file:
No symbol "TABLE_LIST" in current context.,```

`offset-extract.sh` expects the `TABLE_LIST` 

https://github.com/trellix-enterprise/mysql-audit/blob/466c36d6050b01ac33a7519d68d235a3dfd93cc9/offset-extract/offset-extract.sh#L134-L142

But MySQL 8.0.32 and later are refactored it to `Table_ref` ..

https://github.com/mysql/mysql-server/commit/951cb5106cb8677c08afa683fd5aa747488c1ccd
yoku0825 added a commit to yoku0825/mysql-audit that referenced this issue Nov 9, 2023
@yoku0825
Copy link
Author

yoku0825 commented Nov 9, 2023

I wrote the patch for this, but not yet verify the result is correct or not.

master...yoku0825:mysql-audit:idea_for_266

yoku0825 added a commit to yoku0825/mysql-audit that referenced this issue Nov 21, 2023
@yoku0825
Copy link
Author

yoku0825 commented Jun 5, 2024

My colleagues succeeded to build / use under our 8.0.35 environment.

@yoku0825
Copy link
Author

yoku0825 commented Jun 5, 2024

#273

@yoku0825
Copy link
Author

yoku0825 commented Jun 6, 2024

#274 instead of #273

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant