ShuriZma
/
xqemu
mirror of https://github.com/xqemu/xqemu.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

net: e1000e: fix dead code in e1000e_write_packet_to_guest 

Because is_first is declared inside a loop, it is always true.  The store
is dead, and so is the "else" branch of "if (is_first)".  is_last is
okay though.

Reported by Coverity.

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Reviewed-by: Dmitry Fleytman <dmitry@daynix.com>
Signed-off-by: Jason Wang <jasowang@redhat.com>
This commit is contained in:
Paolo Bonzini 2017-01-26 11:10:10 +01:00 committed by Jason Wang
parent a2dbe1356f
commit e514fc7e12
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
hw/net/e1000e_core.c
View File

@ -1507,6 +1507,7 @@ e1000e_write_packet_to_guest(E1000ECore *core, struct NetRxPkt *pkt,
const E1000E_RingInfo *rxi; const E1000E_RingInfo *rxi;
size_t ps_hdr_len = 0; size_t ps_hdr_len = 0;
bool do_ps = e1000e_do_ps(core, pkt, &ps_hdr_len); bool do_ps = e1000e_do_ps(core, pkt, &ps_hdr_len);
bool is_first = true;
rxi = rxr->i; rxi = rxr->i;
@ -1514,7 +1515,6 @@ e1000e_write_packet_to_guest(E1000ECore *core, struct NetRxPkt *pkt,
hwaddr ba[MAX_PS_BUFFERS]; hwaddr ba[MAX_PS_BUFFERS];
e1000e_ba_state bastate = { { 0 } }; e1000e_ba_state bastate = { { 0 } };
bool is_last = false; bool is_last = false;
bool is_first = true;
desc_size = total_size - desc_offset; desc_size = total_size - desc_offset;