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
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