> 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
