[libvirt] [PATCH 1/2] qemu: Report error on unexpected job stats type
Peter Krempa
pkrempa at redhat.com
Fri Jun 1 16:09:05 UTC 2018
On Fri, Jun 01, 2018 at 18:04:29 +0200, Jiri Denemark wrote:
> On Fri, Jun 01, 2018 at 17:55:12 +0200, Jiri Denemark wrote:
> > On Fri, Jun 01, 2018 at 17:31:02 +0200, Peter Krempa wrote:
> > > On Fri, Jun 01, 2018 at 10:46:02 +0200, Jiri Denemark wrote:
> > > > If we ever fail to properly set jobinfo->statsType,
> > > > qemuDomainJobInfoToParams would return -1 without setting an error.
> > > >
> > > > Signed-off-by: Jiri Denemark <jdenemar at redhat.com>
> > > > ---
> > > > src/qemu/qemu_domain.c | 8 +++++---
> > > > 1 file changed, 5 insertions(+), 3 deletions(-)
> > > >
> > > > diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c
> > > > index 2c51e4c0d8..360379b26c 100644
> > > > --- a/src/qemu/qemu_domain.c
> > > > +++ b/src/qemu/qemu_domain.c
> > > > @@ -717,10 +717,12 @@ qemuDomainJobInfoToParams(qemuDomainJobInfoPtr jobInfo,
> > > > return qemuDomainDumpJobInfoToParams(jobInfo, type, params, nparams);
> > > >
> > > > case QEMU_DOMAIN_JOB_STATS_TYPE_NONE:
> > > > - break;
> > > > + default:
> > > > + virReportError(VIR_ERR_INTERNAL_ERROR,
> > > > + _("unexpected type of job stats: %d"),
> > > > + jobInfo->statsType);
> > >
> > > virReportEnumRangeError?
> >
> > Right, in that case, I need to split default and NONE branch, and report
> > different errors for each of them.
>
> Well, I was wrong. Calling virReportEnumRangeError for both cases is OK.
Both are programming errors so I think it's fine.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20180601/1d71a3e9/attachment-0001.sig>
More information about the libvir-list
mailing list