From: Ahmad Fatoum <a.fatoum@pengutronix.de>
To: barebox@lists.infradead.org
Subject: [PATCH 00/32] treewide: fix some clang-analyze static analyzer warnings
Date: Mon, 5 Sep 2022 11:55:25 +0200 [thread overview]
Message-ID: <20220905095557.596891-1-a.fatoum@pengutronix.de> (raw)
I ran scan-build while building barebox for sandbox with clang and
patched some of the reported issues.
Ahmad Fatoum (32):
clk: define stub implementation for clk_get_parent
clk: have SCMI and SiFive clock controllers depend on COMMON_CLK
meminfo: support SANDBOX build with DEBUG log level
net: phy: micrel: drop useless assignment of dummy read
mci: core: drop useless assignment
nvmem: core: propagate read failure
fs: remove never-read initializer in mount_all()
usb: otg: always propagate failure to register parameters
usb: dwc2: gracefully handle unknown hs_phy_type
state: propagate failure to fixup enum32 into DT
of: silence warning about never-read error assignment
commands: trigger: drop unused variable
commands: tutorial: fix memory leak
bthread: fix null pointer dereference in error path
common: env: drop never-read initialization
of: refactor for of_fixup_reserved_memory() for clarity
password: avoid static analyzer false positive
regmap-mmio: regmap_mmio_get_min_stride: unify branches for
readability
crypto: caam - delete unused variable
misc: ubootvar: always initialize struct ubootvar_data::flag
nvmem: core: drop always true condition
of: fdt: gracefully handle out-of-place properties
of: overlay: drop unused variable of_overlay_apply_dir()
of: partition: drop unused variable
serial: ns16550_pci: drop useless assignment
phy: core: drop useless else clause
fs: ext4: ext_barebox: handle ext_get_inode() errors
fs: fat: propagate f_lseek failure
fs: drop duplicate follow_managed() call
lib: parse_area_spec: guard against NULL pointer dereference
net: ping: propagate failure
net: fastboot: keep error message initialized at all times
commands/trigger.c | 5 +----
commands/tutorial.c | 7 ++----
common/bthread.c | 2 ++
common/env.c | 2 +-
common/meminfo.c | 6 ++++--
common/oftree.c | 6 ++++--
common/password.c | 4 ++--
common/state/state_variables.c | 2 ++
drivers/base/regmap/regmap-mmio.c | 2 +-
drivers/clk/Kconfig | 5 ++++-
drivers/crypto/caam/jr.c | 2 --
drivers/mci/mci-core.c | 2 +-
drivers/misc/ubootvar.c | 2 +-
drivers/net/phy/micrel.c | 2 +-
drivers/nvmem/core.c | 8 ++++++-
drivers/of/fdt.c | 2 +-
drivers/of/overlay.c | 3 ---
drivers/of/partition.c | 3 ---
drivers/of/resolver.c | 1 +
drivers/phy/phy-core.c | 2 --
drivers/serial/serial_ns16550_pci.c | 2 +-
drivers/usb/dwc2/core.c | 3 +++
drivers/usb/otg/otgdev.c | 33 ++++++++++++++++-------------
fs/ext4/ext_barebox.c | 18 ++++++++++------
fs/fat/fat.c | 5 +++++
fs/fs.c | 15 ++++---------
include/linux/clk.h | 20 ++++++++++-------
lib/misc.c | 2 ++
net/fastboot.c | 2 ++
net/net.c | 2 +-
30 files changed, 95 insertions(+), 75 deletions(-)
--
2.30.2
next reply other threads:[~2022-09-05 10:58 UTC|newest]
Thread overview: 37+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-09-05 9:55 Ahmad Fatoum [this message]
2022-09-05 9:55 ` [PATCH 01/32] clk: define stub implementation for clk_get_parent Ahmad Fatoum
2022-09-12 9:23 ` Sascha Hauer
2022-09-05 9:55 ` [PATCH 02/32] clk: have SCMI and SiFive clock controllers depend on COMMON_CLK Ahmad Fatoum
2022-09-05 9:55 ` [PATCH 03/32] meminfo: support SANDBOX build with DEBUG log level Ahmad Fatoum
2022-09-05 9:55 ` [PATCH 04/32] net: phy: micrel: drop useless assignment of dummy read Ahmad Fatoum
2022-09-05 9:55 ` [PATCH 05/32] mci: core: drop useless assignment Ahmad Fatoum
2022-09-05 9:55 ` [PATCH 06/32] nvmem: core: propagate read failure Ahmad Fatoum
2022-09-05 9:55 ` [PATCH 07/32] fs: remove never-read initializer in mount_all() Ahmad Fatoum
2022-09-05 9:55 ` [PATCH 08/32] usb: otg: always propagate failure to register parameters Ahmad Fatoum
2022-09-05 9:55 ` [PATCH 09/32] usb: dwc2: gracefully handle unknown hs_phy_type Ahmad Fatoum
2022-09-05 9:55 ` [PATCH 10/32] state: propagate failure to fixup enum32 into DT Ahmad Fatoum
2022-09-05 9:55 ` [PATCH 11/32] of: silence warning about never-read error assignment Ahmad Fatoum
2022-09-12 9:41 ` Sascha Hauer
2022-09-05 9:55 ` [PATCH 12/32] commands: trigger: drop unused variable Ahmad Fatoum
2022-09-05 9:55 ` [PATCH 13/32] commands: tutorial: fix memory leak Ahmad Fatoum
2022-09-12 9:44 ` Sascha Hauer
2022-09-05 9:55 ` [PATCH 14/32] bthread: fix null pointer dereference in error path Ahmad Fatoum
2022-09-05 9:55 ` [PATCH 15/32] common: env: drop never-read initialization Ahmad Fatoum
2022-09-05 9:55 ` [PATCH 16/32] of: refactor for of_fixup_reserved_memory() for clarity Ahmad Fatoum
2022-09-05 9:55 ` [PATCH 17/32] password: avoid static analyzer false positive Ahmad Fatoum
2022-09-05 9:55 ` [PATCH 18/32] regmap-mmio: regmap_mmio_get_min_stride: unify branches for readability Ahmad Fatoum
2022-09-05 9:55 ` [PATCH 19/32] crypto: caam - delete unused variable Ahmad Fatoum
2022-09-05 9:55 ` [PATCH 20/32] misc: ubootvar: always initialize struct ubootvar_data::flag Ahmad Fatoum
2022-09-05 9:55 ` [PATCH 21/32] nvmem: core: drop always true condition Ahmad Fatoum
2022-09-05 9:55 ` [PATCH 22/32] of: fdt: gracefully handle out-of-place properties Ahmad Fatoum
2022-09-05 9:55 ` [PATCH 23/32] of: overlay: drop unused variable of_overlay_apply_dir() Ahmad Fatoum
2022-09-05 9:55 ` [PATCH 24/32] of: partition: drop unused variable Ahmad Fatoum
2022-09-05 9:55 ` [PATCH 25/32] serial: ns16550_pci: drop useless assignment Ahmad Fatoum
2022-09-05 9:55 ` [PATCH 26/32] phy: core: drop useless else clause Ahmad Fatoum
2022-09-05 9:55 ` [PATCH 27/32] fs: ext4: ext_barebox: handle ext_get_inode() errors Ahmad Fatoum
2022-09-05 9:55 ` [PATCH 28/32] fs: fat: propagate f_lseek failure Ahmad Fatoum
2022-09-05 9:55 ` [PATCH 29/32] fs: drop duplicate follow_managed() call Ahmad Fatoum
2022-09-05 9:55 ` [PATCH 30/32] lib: parse_area_spec: guard against NULL pointer dereference Ahmad Fatoum
2022-09-05 9:55 ` [PATCH 31/32] net: ping: propagate failure Ahmad Fatoum
2022-09-05 9:55 ` [PATCH 32/32] net: fastboot: keep error message initialized at all times Ahmad Fatoum
2022-09-12 10:06 ` [PATCH 00/32] treewide: fix some clang-analyze static analyzer warnings Sascha Hauer
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20220905095557.596891-1-a.fatoum@pengutronix.de \
--to=a.fatoum@pengutronix.de \
--cc=barebox@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox