Re: [PATCH] domain_conf: make virDomainDiskSetSource() void

I discovered this when I tried to call this function without wrapping it with ignore_value(). I hadn't taken a look for other functions that might be in a similar state. I also could go a bit further with this function's rewrite: the tmp variable isn't actually necessary any longer since g_strdup() will abort on failure. So, the function might as well be just two lines: g_free() followed by g_strdup(). After looking over the rest of domain_conf.c, I've decided to produce a V2 patchset that eliminates the tmp variable in virDomainDiskSetSource() and addresses several other similar issues with other functions in the file. -- Matt