Dear Fred, On Tue, Jan 10, 2023 at 9:57 PM Konrad, Frederic <frederic.kon...@amd.com> wrote:
> Hi, > > > -----Original Message----- > > From: qemu-devel-bounces+fkonrad=amd....@nongnu.org > <qemu-devel-bounces+fkonrad=amd....@nongnu.org> On Behalf Of > > Qiang Liu > > Sent: 09 January 2023 07:00 > > To: qemu-devel@nongnu.org > > Cc: Qiang Liu <cyruscy...@gmail.com>; Alistair Francis < > alist...@alistair23.me>; Edgar E. Iglesias <edgar.igles...@gmail.com>; > Peter > > Maydell <peter.mayd...@linaro.org>; open list:Xilinx ZynqMP and... < > qemu-...@nongnu.org> > > Subject: [PATCH] hw/display/xlnx_dp: fix overflow in > xlnx_dp_aux_push_tx_fifo() > > > > This patch checks if the s->tx_fifo is full. > > > > Fixes: 58ac482a66de ("introduce xlnx-dp") > > Resolves: https://gitlab.com/qemu-project/qemu/-/issues/1424 > > Reported-by: Qiang Liu <cyruscy...@gmail.com> > > Signed-off-by: Qiang Liu <cyruscy...@gmail.com> > > --- > > hw/display/xlnx_dp.c | 6 +++++- > > 1 file changed, 5 insertions(+), 1 deletion(-) > > > > diff --git a/hw/display/xlnx_dp.c b/hw/display/xlnx_dp.c > > index 972473d94f..617b394af2 100644 > > --- a/hw/display/xlnx_dp.c > > +++ b/hw/display/xlnx_dp.c > > @@ -854,7 +854,11 @@ static void xlnx_dp_write(void *opaque, hwaddr > offset, uint64_t value, > > break; > > case DP_AUX_WRITE_FIFO: { > > uint8_t c = value; > > - xlnx_dp_aux_push_tx_fifo(s, &c, 1); > > + if (fifo8_is_full(&s->tx_fifo)) { > > + qemu_log_mask(LOG_GUEST_ERROR, "xlnx_dp: TX fifo is full"); > > + } else { > > + xlnx_dp_aux_push_tx_fifo(s, &c, 1); > > + } > > I'd rather move the check in xlnx_dp_aux_push_tx_fifo, like > xlnx_dp_aux_pop_tx_fifo. > Otherwise looks good to me. > Sounds fine. Let me resend a patch. Best, Qiang