[et-mgmt-tools] [PATCH] Fix lookup pool by path

Cole Robinson crobinso at redhat.com
Sun Dec 7 04:19:54 UTC 2008

Andreas Schneider wrote:
> On Wednesday 03 December 2008 17:21:47 Cole Robinson wrote:
>> Hmm, yes, seems like libvirt should be sanitizing the target path when
>> the pool is defined (though virtinst could certainly do this too).
>> Looks like there could be arbitrary '/' in the middle of the path
>> though, so if you use something like os.path.abspath in your patch it
>> should sanitize everything appropriately.
> Thanks, this was the function I was looking for. I've attached an updated 
> patch. I think it should be fixed in libvirt and virt-manager. Just to be sure 
> that you can't run into the problem again.

Agreed on the above point. I've committed your fix now:


Thanks for the contribution!

- Cole

More information about the et-mgmt-tools mailing list