On 02/25/2016 08:48 PM, Aaron Conole wrote:
> Silence a compiler warning that this variable may be used uninitialized.
>
> Signed-off-by: Aaron Conole <aconole at redhat.com>
> ---
> drivers/net/ixgbe/ixgbe_rxtx.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/net/ixgbe/ixgbe_rxtx.c b/drivers/net/ixgbe/ixgbe_rxtx.c
> index e95e6b7..775edc7 100644
> --- a/drivers/net/ixgbe/ixgbe_rxtx.c
> +++ b/drivers/net/ixgbe/ixgbe_rxtx.c
> @@ -1563,7 +1563,7 @@ ixgbe_recv_pkts_lro(void *rx_queue, struct rte_mbuf
> **rx_pkts, uint16_t nb_pkts,
> struct ixgbe_rx_entry *rxe;
> struct ixgbe_scattered_rx_entry *sc_entry;
> struct ixgbe_scattered_rx_entry *next_sc_entry;
> - struct ixgbe_rx_entry *next_rxe;
> + struct ixgbe_rx_entry *next_rxe = NULL;
> struct rte_mbuf *first_seg;
> struct rte_mbuf *rxm;
> struct rte_mbuf *nmb;
> @@ -1740,7 +1740,7 @@ next_desc:
> * the pointer to the first mbuf at the NEXTP entry in the
> * sw_sc_ring and continue to parse the RX ring.
> */
> - if (!eop) {
> + if (!eop && next_rxe) {
> rxm->next = next_rxe->mbuf;
> next_sc_entry->fbuf = first_seg;
> goto next_desc;
>
The patch looks ok as such, but then again warning looks like a false
positive to me: assignment and dereferencing depend on the same value of
eop, which cannot change between the two.
CC'ing the maintainers for attention...
- Panu -
