[libvirt] [PATCH REPOST 0/6] Privatize _virStoragePoolObj and _virStorageVolDefList (Batch #1)

Michal Privoznik mprivozn at redhat.com
Fri Oct 6 09:34:12 UTC 2017


On 10/05/2017 10:23 PM, John Ferlan wrote:
> Since the previous series (19 patches):
> 
> https://www.redhat.com/archives/libvir-list/2017-September/msg00594.html
> 
> didn't garner any attention, let me try with smaller patch piles with the
> hope that forward progress will be made.
> 
> The only "new" one in this series from the other is patch2 - something
> that Coverity let me know about in an error scenario.
> 
> Essentially the bulk of these patches remove direct obj->def referencing
> in favor of a virStoragePoolObjGetDef call.
> 
> John Ferlan (6):
>   conf: Fix prototype/definition for virStoragePoolObj get functions
>   tests: Fix possible NULL deref
>   storage: Use virStoragePoolObjGetDef accessor for driver
>   test: Rename @vol to @volDef in testOpenVolumesForPool
>   test: Create local virStoragePoolObjPtr VolLookup APIs
>   test: Use virStoragePoolObjGetDef accessor
> 
>  src/conf/virstorageobj.c       |   4 +-
>  src/conf/virstorageobj.h       |   4 +-
>  src/storage/storage_driver.c   | 411 +++++++++++++++++++++++------------------
>  src/test/test_driver.c         | 203 +++++++++++---------
>  tests/storagevolxml2argvtest.c |   3 +-
>  5 files changed, 352 insertions(+), 273 deletions(-)
> 

ACK series.

Michal




More information about the libvir-list mailing list