Re: [2/2] ASoC: samsung: odroid: fix a double-free issue for cpu_dai
From: Markus Elfring
Date: Sun Jul 14 2019 - 08:48:09 EST
> The cpu_dai variable is still being used after the of_node_put() call,
Such an implementation detail is questionable.
https://wiki.sei.cmu.edu/confluence/display/c/MEM30-C.+Do+not+access+freed+memory
> which may result in double-free:
This consequence is also undesirable.
https://cwe.mitre.org/data/definitions/415.html
Now I wonder if two update steps are really appropriate as a fix
instead of using a single update step for the desired correction
in this software module.
Should a commit (including previous ones) usually be correct by itself?
Regards,
Markus