From: "Edgar E. Iglesias" <[email protected]> Avoid use of uninitialized bufioreq_evtchn. It should only be used if buffered IOREQs are enabled.
Resolves: Coverity CID 1563383 Reported-by: Peter Maydell <[email protected]> Acked-by: Stefano Stabellini <[email protected]> Signed-off-by: Edgar E. Iglesias <[email protected]> --- hw/xen/xen-hvm-common.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/hw/xen/xen-hvm-common.c b/hw/xen/xen-hvm-common.c index 7d2b72853b..7ffbbfea23 100644 --- a/hw/xen/xen-hvm-common.c +++ b/hw/xen/xen-hvm-common.c @@ -752,9 +752,10 @@ static int xen_map_ioreq_server(XenIOState *state) return -1; } - trace_xen_map_ioreq_server_buffered_io_evtchn(bufioreq_evtchn); - - state->bufioreq_remote_port = bufioreq_evtchn; + if (state->has_bufioreq) { + trace_xen_map_ioreq_server_buffered_io_evtchn(bufioreq_evtchn); + state->bufioreq_remote_port = bufioreq_evtchn; + } return 0; } -- 2.43.0
