Re: [PATCH 3/3] lib: Require path in virDomainRestoreParams()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 5/12/22 19:09, Claudio Fontana wrote:
> Hello Michal,
> 
> this seems to be going backwards to special case arguments instead of putting them into typed parameters.
> I do not understand where this need comes from, but it does not seem a good direction to me.

The need stems from my testing python bindings for these new APIs that
I'm writing. When I call the restore API with no arguments, which is
equivalent to calling plain virDomainRestoreParams(conn, NULL, 0, 0);
then I see the following error:

libvirt: QEMU Driver error : Failed to open file '(null)': Bad address

Obviously, this is wrong. And indeed, when I think about the API more
then the path to restore from is critical, at least for now. But as I
suggests in one of my previous replies, I'm going to make path required
in typed params for now. If we ever come with an alternative source for
restore then that check can be relaxed.

Michal




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux