On 10/07/2015 02:50 AM, Peter Krempa wrote: > On Tue, Oct 06, 2015 at 18:34:55 -0400, John Ferlan wrote: >> Add a new boolean 'created' to virDirCreate to be set when a directory >> is created either directly or in a fork'd child. This will allow >> a caller to make "decisions" regarding whether or not to delete the >> directory since virDirCreate has many other failure scenarios and there's >> no guarantee that the mkdir was the cause for failure. > > Is there actually a need to pass the 'created' bool out? Can't the > function just delete the directory internally in case a failure happens? > > The new semantics basically would be: > > virDirCreate was successful -> we have a new dir we can use > virDirCreate failed -> nothing to do/clean up > Perhaps there's no "need" to pass a create boolean - I was merely following up on Michal's suggestion. I guess I assumed it was a suggestion discussed among those in the Brno office. Performing the cleanup in the create functions should be workable. Each of the functions would need to have code added the case where the volume/file already exists and adjust the currently common cleanup code to make decisions based on whether the create was done. I'll cobble together some patches for that. John -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list