Re: [PATCH net v2] gve: add missing NULL check for gve_alloc_pending_packet() in TX DQO

[Simon Horman]

On Mon, Jun 02, 2025 at 03:34:29AM -0700, Alok Tiwari wrote:
> gve_alloc_pending_packet() can return NULL, but gve_tx_add_skb_dqo()
> did not check for this case before dereferencing the returned pointer.
> 
> Add a missing NULL check to prevent a potential NULL pointer
> dereference when allocation fails.
> 
> This improves robustness in low-memory scenarios.
> 
> Fixes: a57e5de476be ("gve: DQO: Add TX path")
> Signed-off-by: Alok Tiwari <alok.a.tiwari@xxxxxxxxxx>
> Reviewed-by: Mina Almasry <almasrymina@xxxxxxxxxx>
> ---
> v1->v2
> added Fixes tag and [PATCH net v2]

This patch is marked as Rejected in patchwork, but it's unclear to me why.
The patch does look good to me, there was positive review to v1 which
is addressed in v2, and i see no prior comment on v2. I'm updating it's
status accordingly. Apologies if I missed something.

Reviewed-by: Simon Horman <horms@xxxxxxxxxx>

-- 
pw-bot: under-review