On Thu, Dec 28, 2023 at 08:56:38AM +0100, Markus Elfring wrote: > The kfree() function was called in up to two cases by > the cros_ec_cmd_xfer_i2c() function during error handling > even if the passed variable contained a null pointer. > This issue was detected by using the Coccinelle software. > > * Adjust jump targets. > > * Delete two initialisations which became unnecessary > with this refactoring. The patch saves few instructions but makes the code less readable a bit. I would prefer to leave the code as is or wait for other reviewers' input.