ice: Graceful error handling in HW table calloc failure

In the ice_init_hw_tbls, if the devm_kcalloc for es->written fails, catch
that error and bail out gracefully, instead of continuing with a NULL
pointer.

Fixes: 32d63fa1e9f3 ("ice: Initialize DDP package structures")
Signed-off-by: Surabhi Boob <surabhi.boob@intel.com>
Tested-by: Andrew Bowers <andrewx.bowers@intel.com>
Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com>

diff --git a/drivers/net/ethernet/intel/ice/ice_flex_pipe.c b/drivers/net/ethernet/intel/ice/ice_flex_pipe.c
index d59869b2c65efdbdba61100c3df855a2abcdf0df..5ceba009db167b2b8bba07d8a3729ad3bfa0e5e3 100644 (file)
--- a/drivers/net/ethernet/intel/ice/ice_flex_pipe.c
+++ b/drivers/net/ethernet/intel/ice/ice_flex_pipe.c
@@ -3151,10 +3151,12 @@ enum ice_status ice_init_hw_tbls(struct ice_hw *hw)
                es->ref_count = devm_kcalloc(ice_hw_to_dev(hw), es->count,
                                             sizeof(*es->ref_count),
                                             GFP_KERNEL);
+               if (!es->ref_count)
+                       goto err;
 
                es->written = devm_kcalloc(ice_hw_to_dev(hw), es->count,
                                           sizeof(*es->written), GFP_KERNEL);
-               if (!es->ref_count)
+               if (!es->written)
                        goto err;
        }
        return 0;