On 02/07/2025 08:13, Heinrich Schuchardt wrote:
On 01.07.25 15:38, Andrew Goodbody wrote:
Taking a goto to out_of_resources before receive_lengths is assigned
will result in an attempt to free an unitialised pointer. Instead
initialise receive_lengths to NULL on declaration to prevent this from
occurring.
This issue was found by Smatch.
Signed-off-by: Andrew Goodbody <andrew.goodb...@linaro.org>
---
lib/efi_loader/efi_net.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/efi_loader/efi_net.c b/lib/efi_loader/efi_net.c
index 8e708d8d350..3d092523dd5 100644
--- a/lib/efi_loader/efi_net.c
+++ b/lib/efi_loader/efi_net.c
@@ -1130,7 +1130,7 @@ efi_status_t efi_net_register(struct udevice *dev)
struct efi_net_obj *netobj;
void *transmit_buffer = NULL;
uchar **receive_buffer = NULL;
- size_t *receive_lengths;
+ size_t *receive_lengths = NULL;
CCing Adriano as author of the code.
Did scripts/get_maintainer.pl not show him?
I used 'b4 prep --auto-to-cc' which I thought was enough, but apparently
not, sorry.
Andrew
Reviewed-by: Heinrich Schuchardt <xypron.g...@gmx.de>
int i, j;
if (!dev) {
