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?