Re: [libvirt] [PATCH] phyp: don't steal storage management from other drivers

> Sorry for not realizing that during my review; this is my first > time dealing with a storage driver patch. > >> It must only return VIR_DRV_OPEN_SUCCESS if conn->driver->name == VIR_DRV_PHYP > > Agreed. I think this patch does the right thing. > The intention is correct, but the implementation is not. Why do you duplicate the whole phypOpen code in phypStorageOpen? Besides of being unnecessary this also overwrites the already initialized private driver data of the virConnectPtr.

Something like I did for the ESX storage driver (and Daniel suggested) should be sufficient here too. Probably comparing the conn->driver->no would be even better. static virDrvOpenStatus phypStorageOpen(virConnectPtr conn, virConnectAuthPtr auth ATTRIBUTE_UNUSED, int flags) { virCheckFlags(0, VIR_DRV_OPEN_ERROR); if (conn->driver->no != VIR_DRV_PHYP) {

return VIR_DRV_OPEN_DECLINED; } return VIR_DRV_OPEN_SUCCESS; }