Re: [PATCH] Make virConnectOpenInternal() report error in all cases

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

 




On Thu, 6 Jan 2022, Daniel P. Berrangé wrote:

> On Thu, Jan 06, 2022 at 08:34:26PM +0530, Ani Sinha wrote:
> > virConnectOpenInternal() does not report error in all failure scenarios, except
> > in some specific cases.
>
> I don't believe this is correct. My reading of the code is that it
> is already reporting errors in all exit paths that can return NULL.

Yes you are likely right. I should have gone one level deeper in the call
chain. I have not checked all the code paths exhaistively but for the one
I checked, the lowest level functions are indeed reporting the more finer
grained errors to their callers.

I have sent an updated patch that just has the last two hunks. I have kept
virConnectOpenInternal() unchanged.

[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