Re: [libvirt] [PATCH 2/3] Create virFileIsOnNetworkShare utility function

> >On Fri, Jun 25, 2010 at 07:42:14AM -0400, Laine Stump wrote: > >>This code was previously used inline during qemudDomainSaveFlag. This >>patch moves it into a utility function in preparation for it to be >>used elsewhere. >> >How about just switching to use virStorageFileIsSharedFS() ? > Ah! I either hadn't noticed that function go in, or had forgotten about it. One problem I see is that virStorageFileIsSharedFS doesn't traverse back up the path elements looking for a partial path it can stat. This means it will fail if the file in question (or any of its ancestor directories) isn't stat'able by root. Any objections to me changing virStorageFileIsSharedFS to behave like that?