[libvirt] [PATCH 0/5] Unify check for auth and auth->cb
Michal Privoznik
mprivozn at redhat.com
Tue Aug 14 16:08:05 UTC 2018
On 08/03/2018 02:27 AM, Marcos Paulo de Souza wrote:
> Hi guys,
>
> in some drivers, auth and auth->cb are checked in connectOpen function, while in xenapi,
> only auth is checked, which that could lead to a problem if cb if invalid. In
> phyp, auth and auth->cb are checked twice, in getUser and getPassword.
>
> So, this patchset adds the check for auth and auth->cb inside
> virAuthGetUsername and virAuthGetPassword, making it safer for all drivers that
> rely in auth callbacks.
>
> Marcos Paulo de Souza (5):
> virauth.c: Check for valid auth callback
> esx: Drop check for auth and auth->cb
> hyperv: Drop check for auth and auth->cb
> phyp: Drop check for auth and auth->cb
> xenapi: Drop check for auth
>
> src/esx/esx_driver.c | 7 -------
> src/hyperv/hyperv_driver.c | 7 -------
> src/phyp/phyp_driver.c | 11 -----------
> src/util/virauth.c | 12 ++++++++++++
> src/xenapi/xenapi_driver.c | 6 ------
> 5 files changed, 12 insertions(+), 31 deletions(-)
>
ACKed and pushed.
Michal
More information about the libvir-list
mailing list