diff options
| author | Edward O'Callaghan <quasisec@google.com> | 2023-01-15 14:50:25 +1100 |
|---|---|---|
| committer | Anastasia Klimchuk <aklm@chromium.org> | 2023-01-22 05:56:12 +0000 |
| commit | dd41980f1784c91dd8b30a9074a260621d366c2e (patch) | |
| tree | a9a006d60a296cc3a92f4826348185aeff5b30ec /tests/raiden_debug_spi.c | |
| parent | 46856234b3b6dea93e38c499eecbb432cb1e455e (diff) | |
| download | flashrom-dd41980f1784c91dd8b30a9074a260621d366c2e.tar.gz flashrom-dd41980f1784c91dd8b30a9074a260621d366c2e.tar.bz2 flashrom-dd41980f1784c91dd8b30a9074a260621d366c2e.zip | |
tests/: Assert on NULL heap allocations in tests
Change-Id: Id2adcfe859fb25d2a7f0734655c6b9a58c0890b6
Signed-off-by: Edward O'Callaghan <quasisec@google.com>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/71919
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Anastasia Klimchuk <aklm@chromium.org>
Diffstat (limited to 'tests/raiden_debug_spi.c')
| -rw-r--r-- | tests/raiden_debug_spi.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/tests/raiden_debug_spi.c b/tests/raiden_debug_spi.c index f5bf185a..5c79a909 100644 --- a/tests/raiden_debug_spi.c +++ b/tests/raiden_debug_spi.c @@ -19,6 +19,7 @@ static ssize_t raiden_debug_libusb_get_device_list(void *state, libusb_context *ctx, libusb_device ***list) { *list = calloc(1, sizeof(**list)); + assert_non_null(*list); /* * libusb_device is opaque type, it is tossed around between libusb functions but always @@ -48,10 +49,14 @@ static int raiden_debug_libusb_get_config_descriptor( void *state, libusb_device *dev, uint8_t config_index, struct libusb_config_descriptor **config) { *config = calloc(1, sizeof(**config)); + assert_non_null(*config); struct libusb_endpoint_descriptor *tmp_endpoint = calloc(2, sizeof(*tmp_endpoint)); + assert_non_null(tmp_endpoint); struct libusb_interface_descriptor *tmp_interface_desc = calloc(1, sizeof(*tmp_interface_desc)); + assert_non_null(tmp_interface_desc); struct libusb_interface *tmp_interface = calloc(1, sizeof(*tmp_interface)); + assert_non_null(tmp_interface); /* in endpoint */ tmp_endpoint[0].bEndpointAddress = 0x80; |