mirror of
https://github.com/Dasharo/coreboot.git
synced 2026-06-13 10:16:48 -07:00
c68645cd88
When parsing the string, if it doesn't end with 0 or \n, get_line_as_int returns -1, but wasn't freeing the buffer. Also if we got an empty string with just null termination, that byte would also cause a leak, so move the second free() to the bottom. It's always fine to free a null pointer if the allocation failed. BUG=CID 1419489 Change-Id: Ibb3a7544ec8c46820b6e47b6fd4bbe5cabafe1a8 Signed-off-by: Martin Roth <gaumless@gmail.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/88335 Reviewed-by: Elyes Haouas <ehaouas@noos.fr> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>