https://github.com/torvalds/linux
Revision fc21f08375dbf654bd1fda748261955de580ac14 authored by Edward Cree on 19 September 2023, 18:39:49 UTC, committed by Paolo Abeni on 21 September 2023, 08:37:16 UTC
Several places in TC offload code assumed that the return from
 rhashtable_lookup_get_insert_fast() was always either NULL or a valid
 pointer to an existing entry, but in fact that function can return an
 error pointer.  In that case, perform the usual cleanup of the newly
 created entry, then pass up the error, rather than attempting to take a
 reference on the old entry.

Fixes: d902e1a737d4 ("sfc: bare bones TC offload on EF100")
Reported-by: Dan Carpenter <dan.carpenter@linaro.org>
Signed-off-by: Edward Cree <ecree.xilinx@gmail.com>
Link: https://lore.kernel.org/r/20230919183949.59392-1-edward.cree@amd.com
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
1 parent 1703b2e
History
Tip revision: fc21f08375dbf654bd1fda748261955de580ac14 authored by Edward Cree on 19 September 2023, 18:39:49 UTC
sfc: handle error pointers returned by rhashtable_lookup_get_insert_fast()
Tip revision: fc21f08

back to top