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@xxxxxxxxx> Thanks, > > -- > 2.17.1 > > -- > libvir-list mailing list > libvir-list@xxxxxxxxxx > https://www.redhat.com/mailman/listinfo/libvir-list -- Thanks, Marcos -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list