14 Jul
2019
14 Jul
'19
2:47 p.m.
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+m...
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