Change in flashrom[master]: libflashrom: Allow NULL-pointer argument in flashrom_flash_release()

[Nico Huber (Code Review)]

Nico Huber has submitted this change. ( https://review.coreboot.org/c/flashrom/+/62340 )

Change subject: libflashrom: Allow NULL-pointer argument in flashrom_flash_release()
......................................................................

libflashrom: Allow NULL-pointer argument in flashrom_flash_release()

free() allows NULL and it makes error paths easier to handle when one
just needs to write `free(x);` without needing to care if `x` was
allocated already. Let's follow this rule in flashrom_flash_release().
flashrom_layout_release() already checks for NULL.

Change-Id: Id119c2e4f3aa1b11313059f11aac73c3e583185c
Signed-off-by: Nico Huber 
Reviewed-on: https://review.coreboot.org/c/flashrom/+/62340
Tested-by: build bot (Jenkins) 
Reviewed-by: Nikolai Artemiev 
Reviewed-by: Felix Singer 
---
M libflashrom.c
1 file changed, 3 insertions(+), 0 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Felix Singer: Looks good to me, approved
  Nikolai Artemiev: Looks good to me, but someone else must approve

diff --git a/libflashrom.c b/libflashrom.c
index d66c295..5f14c68 100644
--- a/libflashrom.c
+++ b/libflashrom.c
@@ -361,6 +361,9 @@
  */
 void flashrom_flash_release(struct flashrom_flashctx *const flashctx)
 {
+	if (!flashctx)
+		return;
+
    flashrom_layout_release(flashctx->default_layout);
    free(flashctx->chip);
    free(flashctx);