[flashrom-gerrit] Change in flashrom[master]: libflashrom: add querying functions with meson integration

[David Hendricks (Code Review)]

David Hendricks has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/34363 )

Change subject: libflashrom: add querying functions with meson integration
......................................................................

Patch Set 2: Code-Review+1

(1 comment)

Looks good to me, I'm just wondering if flashrom_data_free() is needed.

https://review.coreboot.org/c/flashrom/+/34363/2/libflashrom.c 
File libflashrom.c:

https://review.coreboot.org/c/flashrom/+/34363/2/libflashrom.c@233 
PS2, Line 233: int flashrom_data_free(void *const p)
Do you really need this function? free()'ing NULL is a valid noop (https://pubs.opengroup.org/onlinepubs/009695399/functions/free.html)