mbox series

[0/8] clk: check memory returned by {devm_}kasprintf()

Message ID 20230530093913.1656095-1-claudiu.beznea@microchip.com (mailing list archive)
Headers show
Series clk: check memory returned by {devm_}kasprintf() | expand

Message

Claudiu Beznea May 30, 2023, 9:39 a.m. UTC 
Hi,

While browsing some code I noticed that there are places where pointer
returned by devm_kasprintf() or kasprintf() is not checked. Thus I've
tooked the chance and fixed this (by updating kmerr.cocci script,
changes published at [1]). Along with it some other places where
resources may need to be freed on failure paths were updated.

Thank you,
Claudiu Beznea

[1] https://lore.kernel.org/all/20230530074044.1603426-1-claudiu.beznea@microchip.com/

Claudiu Beznea (8):
  clk: vc5: check memory returned by kasprintf()
  clk: cdce925: check return value of kasprintf()
  clk: si5341: return error if one synth clock registration fails
  clk: si5341: check return value of {devm_}kasprintf()
  clk: si5341: free unused memory on probe failure
  clk: keystone: sci-clk: check return value of kasprintf()
  clk: ti: clkctrl: check return value of kasprintf()
  clk: clocking-wizard: check return value of devm_kasprintf()

 drivers/clk/clk-cdce925.c                  | 12 +++++++
 drivers/clk/clk-si5341.c                   | 38 +++++++++++++---------
 drivers/clk/clk-versaclock5.c              | 29 +++++++++++++++++
 drivers/clk/keystone/sci-clk.c             |  2 ++
 drivers/clk/ti/clkctrl.c                   |  7 ++++
 drivers/clk/xilinx/clk-xlnx-clock-wizard.c |  5 +++
 6 files changed, 78 insertions(+), 15 deletions(-)