Details
-
Bug
-
Status: Closed (View Workflow)
-
Major
-
Resolution: Fixed
-
10.11.0
-
None
Description
MariaDB [(none)]> show grants;
|
+------------------------------------------------+ |
| Grants for developer@% | |
+------------------------------------------------+ |
| GRANT USAGE ON *.* TO `developer`@`%` | |
| GRANT ALL PRIVILEGES ON `dev_db`.* TO `PUBLIC` | |
+------------------------------------------------+ |
2 rows in set (0.000 sec) |
|
|
MariaDB [(none)]> show grants for public; |
ERROR 1044 (42000): Access denied for user 'developer'@'%' to database 'mysql' |
Any user can see public's grants by running their own SHOW GRANTS command. This means that SHOW GRANTS FOR PUBLIC should also be allowed.
Attachments
Issue Links
- is part of
-
MDEV-5215 Granted to PUBLIC
-
- Closed
-
Activity
| Assignee | Oleksandr Byelkin [ sanja ] |
| Fix Version/s | 10.11 [ 27614 ] |
| Assignee | Oleksandr Byelkin [ sanja ] | Vicențiu Ciorbaru [ cvicentiu ] |
The bug is present in 85b939ae35c74d458916ff5b3f0da8045fc16522, which is the commit used for preview-10.11-preview release.
It was later fixed in a follow-up commit in bb-10.11-MDEV-5215 tree. The relevant change lies in sql_acl.cc get_show_user() function
- do_check_access= strcmp(*rolename, sctx->priv_role);
|
+ do_check_access= !is_public(lex_user) && strcmp(*rolename, sctx->priv_role); |
The test case was not present however, so it was added in bb-10.11-MDEV-5215 tree to prevent further regressions.
| issue.field.resolutiondate | 2022-10-10 15:44:50.0 | 2022-10-10 15:44:50.513 |
| Fix Version/s | 10.11.1 [ 28454 ] | |
| Fix Version/s | 10.11 [ 27614 ] | |
| Resolution | Fixed [ 1 ] | |
| Status | Open [ 1 ] | Closed [ 6 ] |
I tried to repeat and here it is:
create database dbtest;create user `testuser`@`%`;GRANT USAGE ON *.* TO `testuser`@`%`;GRANT ALL PRIVILEGES ON `dbtest`.* TO `PUBLIC`;connect testuser,localhost,testuser,,;show grants for public;Grants for PUBLICGRANT ALL PRIVILEGES ON `dbtest`.* TO PUBLICshow grants for testuser;Grants for testuser@%GRANT USAGE ON *.* TO `testuser`@`%`connection default;disconnect testuser;REVOKE ALL PRIVILEGES ON `dbtest`.* FROM `PUBLIC`;REVOKE USAGE ON *.* FROM `testuser`@`%`;drop user `testuser`@`%`;drop database dbtest;test suite is here:
create database dbtest;create user `testuser`@`%`;GRANT USAGE ON *.* TO `testuser`@`%`;GRANT ALL PRIVILEGES ON `dbtest`.* TO `PUBLIC`;connect (testuser,localhost,testuser,,);show grants for public;show grants for testuser;connection default;disconnect testuser;REVOKE ALL PRIVILEGES ON `dbtest`.* FROM `PUBLIC`;REVOKE USAGE ON *.* FROM `testuser`@`%`;drop user `testuser`@`%`;drop database dbtest;