> -----Original Message----- > From: qemu-devel-bounces+chen.zhang=intel....@nongnu.org <qemu- > devel-bounces+chen.zhang=intel....@nongnu.org> On Behalf Of Nabih > Estefan > Sent: Wednesday, June 19, 2024 1:25 AM > To: peter.mayd...@linaro.org > Cc: qemu-devel@nongnu.org; kft...@nuvoton.com; wuhao...@google.com; > jasow...@redhat.org; nabiheste...@google.com > Subject: [PATCH] hw/net: Fix Coverity Issue for npcm-gmac > > There is an extra `buf=` set that is not used by npcm-gmac. Remove it for > coverity to be happy. >
Looks good to me. By the way, I see this function comments the Steps 1/2/3... But I can't find the page 384 of datasheet, please add more details about the datasheet(versions of datasheet, URL....). Reviewed-by: Zhang Chen <chen.zh...@intel.com> Thanks Chen > Signed-off-by: Nabih Estefan <nabiheste...@google.com> > --- > hw/net/npcm_gmac.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/hw/net/npcm_gmac.c b/hw/net/npcm_gmac.c index > 1b71e2526e..b397fd5064 100644 > --- a/hw/net/npcm_gmac.c > +++ b/hw/net/npcm_gmac.c > @@ -614,7 +614,6 @@ static void > gmac_try_send_next_packet(NPCMGMACState *gmac) > net_checksum_calculate(tx_send_buffer, length, csum); > qemu_send_packet(qemu_get_queue(gmac->nic), tx_send_buffer, > length); > trace_npcm_gmac_packet_sent(DEVICE(gmac)->canonical_path, > length); > - buf = tx_send_buffer; > length = 0; > } > > -- > 2.45.2.627.g7a2c4fd464-goog >
