Quoting Adam Ford (2020-07-16 05:26:20) > When getting the names of the child nodes, kasprintf is used to > allocate memory which is used to create the string for the node > name. Unfortunately, there is no memory check to determine > if this allocation fails, it may cause an error when trying > to get child node name. > > This patch will check if the memory allocation fails, and returns > and -ENOMEM error instead of blindly moving on. > > Fixes: 260249f929e8 ("clk: vc5: Enable addition output configurations of the > Versaclock") > > Suggested-by: Dan Carpenter <dan.carpen...@oracle.com> > Signed-off-by: Adam Ford <aford...@gmail.com> > Reviewed-by: Luca Ceresoli <l...@lucaceresoli.net> > --- > V3: Fix spelling error, and use the style of checking (!variable) instead of > (variable == NULL) > > V2: Fix an issue where a goto was going to use an unitialized variable.
Is the patch from Colin also needed? https://lore.kernel.org/r/20200625132736.88832-1-colin.k...@canonical.com