On Fri, Jul 04, 2025 at 10:53:55AM +0100, Andrew Goodbody wrote: > Hi Tom, > > Smatch reports that the callers of the functions > (dm_)pci_config_read(8|16|32) from drivers/pci/pci-uclass.c do not check the > return value for an error before using the variable that should have been > assigned with the value read. However in the error path that variable is not > assigned to and so the code can use an uninitialised variable in that case. > eg line 1396 in drivers/pci/pci-uclass.c addr may not have been assigned to > in the error case. > > This also seems to be the case with callers from other files as well. > > Would a patch that adds a default assignment before returning an error to > the affected functions be accepted?
Can you please check the size growth on say qemu_arm* and we can go from there? Thanks. -- Tom
signature.asc
Description: PGP signature
