[libvirt] [PATCH 00/11] More virAuthGet* cleanups

John Ferlan posted 11 patches 5 years, 7 months ago
Only 10 patches received!
src/esx/esx_driver.c          | 27 ++++------------
src/hyperv/hyperv_driver.c    | 16 +++------
src/phyp/phyp_driver.c        | 16 +++------
src/rpc/virnetlibsshsession.c |  2 --
src/rpc/virnetsshsession.c    |  5 +--
src/test/test_driver.c        | 15 +++------
src/util/virauth.c            | 61 ++++++++++++++++++++++++-----------
src/xenapi/xenapi_driver.c    | 16 +++------
8 files changed, 67 insertions(+), 91 deletions(-)
[libvirt] [PATCH 00/11] More virAuthGet* cleanups
Posted by John Ferlan 5 years, 7 months ago
In a case of review collision, Michal pushed Marcos Paulo de Souza's
series after I had reviewed, but before seeing my review (oh well):

https://www.redhat.com/archives/libvir-list/2018-August/msg00874.html

Rather than worry about it, here's a series of changes that I
described in my review:

https://www.redhat.com/archives/libvir-list/2018-August/msg00860.html

based on top of Marcos' now pushed weries.

The only thing I didn't cover in this was my comment regarding
xenapiUtil_RequestPassword - I just left it as is.

John Ferlan (11):
  util: Alter virAuthGet*Path API to check valid parameters
  util: Alter virAuthGet*Path API to check valid callback
  util: Remove invalid parameter checks from
    virAuthGet{Username|Password}
  util: Alter virAuthGet*Path API return processing
  util: Alter virAuthGet*Path API to generate auth->cb error
  esx: Don't overwrite virAuthGet{Username|Password} errors
  hyperv: Don't overwrite virAuthGet{Username|Password} errors
  phyp: Don't overwrite virAuthGet{Username|Password} errors
  xenapi: Don't overwrite virAuthGet{Username|Password} errors
  test: Don't overwrite virAuthGet{Username|Password} errors
  rpc: Don't overwrite virAuthGet{Username|Password}Path errors

 src/esx/esx_driver.c          | 27 ++++------------
 src/hyperv/hyperv_driver.c    | 16 +++------
 src/phyp/phyp_driver.c        | 16 +++------
 src/rpc/virnetlibsshsession.c |  2 --
 src/rpc/virnetsshsession.c    |  5 +--
 src/test/test_driver.c        | 15 +++------
 src/util/virauth.c            | 61 ++++++++++++++++++++++++-----------
 src/xenapi/xenapi_driver.c    | 16 +++------
 8 files changed, 67 insertions(+), 91 deletions(-)

-- 
2.17.1

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH 00/11] More virAuthGet* cleanups
Posted by Marcos Paulo de Souza 5 years, 7 months ago
On Tue, Aug 14, 2018 at 01:07:45PM -0400, John Ferlan wrote:
> In a case of review collision, Michal pushed Marcos Paulo de Souza's
> series after I had reviewed, but before seeing my review (oh well):
> 
> https://www.redhat.com/archives/libvir-list/2018-August/msg00874.html
> 
> Rather than worry about it, here's a series of changes that I
> described in my review:
> 
> https://www.redhat.com/archives/libvir-list/2018-August/msg00860.html
> 
> based on top of Marcos' now pushed weries.
> 
> The only thing I didn't cover in this was my comment regarding
> xenapiUtil_RequestPassword - I just left it as is.
> 
> John Ferlan (11):
>   util: Alter virAuthGet*Path API to check valid parameters
>   util: Alter virAuthGet*Path API to check valid callback
>   util: Remove invalid parameter checks from
>     virAuthGet{Username|Password}
>   util: Alter virAuthGet*Path API return processing
>   util: Alter virAuthGet*Path API to generate auth->cb error
>   esx: Don't overwrite virAuthGet{Username|Password} errors
>   hyperv: Don't overwrite virAuthGet{Username|Password} errors
>   phyp: Don't overwrite virAuthGet{Username|Password} errors
>   xenapi: Don't overwrite virAuthGet{Username|Password} errors
>   test: Don't overwrite virAuthGet{Username|Password} errors
>   rpc: Don't overwrite virAuthGet{Username|Password}Path errors
> 
>  src/esx/esx_driver.c          | 27 ++++------------
>  src/hyperv/hyperv_driver.c    | 16 +++------
>  src/phyp/phyp_driver.c        | 16 +++------
>  src/rpc/virnetlibsshsession.c |  2 --
>  src/rpc/virnetsshsession.c    |  5 +--
>  src/test/test_driver.c        | 15 +++------
>  src/util/virauth.c            | 61 ++++++++++++++++++++++++-----------
>  src/xenapi/xenapi_driver.c    | 16 +++------
>  8 files changed, 67 insertions(+), 91 deletions(-)

Nice cleanup.

Reviewed-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com>

Thanks,

> 
> -- 
> 2.17.1
> 
> --
> libvir-list mailing list
> libvir-list@redhat.com
> https://www.redhat.com/mailman/listinfo/libvir-list

-- 
Thanks,
	Marcos

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list