Skip to content

Commit

Permalink
i40e, xsk: uninitialized variable in i40e_clean_rx_irq_zc()
Browse files Browse the repository at this point in the history
The "failure" variable is used without being initialized.  It should be
set to false.

Fixes: 8cbf741 ("i40e, xsk: move buffer allocation out of the Rx processing loop")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Björn Töpel <bjorn.topel@intel.com>
Tested-by: George Kuruvinakunnel <george.kuruvinakunnel@intel.com>
Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com>
  • Loading branch information
Dan Carpenter authored and Tony Nguyen committed Nov 10, 2020
1 parent 3a70017 commit 1773482
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion drivers/net/ethernet/intel/i40e/i40e_xsk.c
Original file line number Diff line number Diff line change
Expand Up @@ -281,8 +281,8 @@ int i40e_clean_rx_irq_zc(struct i40e_ring *rx_ring, int budget)
unsigned int total_rx_bytes = 0, total_rx_packets = 0;
u16 cleaned_count = I40E_DESC_UNUSED(rx_ring);
unsigned int xdp_res, xdp_xmit = 0;
bool failure = false;
struct sk_buff *skb;
bool failure;

while (likely(total_rx_packets < (unsigned int)budget)) {
union i40e_rx_desc *rx_desc;
Expand Down

0 comments on commit 1773482

Please sign in to comment.