Hi, Oleksandr!
All good, ok to push.
Just add a comment right after memcpy, that
/*
if acl_user->nauth was >= nauth, then we've just updated the actual
acl_user, not the copy. We rely on the fact that acl_update_user()
cannot fail anywhere below.
*/
On Oct 15, Oleksandr Byelkin wrote:
> revision-id: 6ec64caec5d (mariadb-10.4.21-64-g6ec64caec5d)
> parent(s): a736a3174a4
> author: Oleksandr Byelkin
> committer: Oleksandr Byelkin
> timestamp: 2021-10-14 16:19:09 +0200
> message:
>
> MDEV-26650: Failed ALTER USER/GRANT statement removes the password from the
> cache
>
> Starting from 10.4 AUTH is not part of ACL_USER so changes have to be done
> over a copy, and bring in the cache only in case of success.
>
Regards,
Sergei
VP of MariaDB Server Engineering
and secur...@mariadb.org
_______________________________________________
Mailing list: https://launchpad.net/~maria-developers
Post to : maria-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~maria-developers
More help : https://help.launchpad.net/ListHelp
