[flashrom-gerrit] [S] Change in flashrom[master]: flashrom.c: Position heap alloc along side check in compare_range()

[Edward O'Callaghan (Code Review)]

Attention is currently required from: Felix Singer, Angel Pons.

Edward O'Callaghan has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/69473 )

Change subject: flashrom.c: Position heap alloc along side check in compare_range()
......................................................................

Patch Set 1:

(1 comment)

File flashrom.c:

https://review.coreboot.org/c/flashrom/+/69473/comment/259f53f0_5cea8f1e 
PS1, Line 451: 	uint8_t *cmpbuf = malloc(len);
> Felix, from a functional standpoint, this change only swaps the order of two variable declarations ( […]

That's right, I sometimes run djgpp on my personal machine (we don't have it in corp). However djgpp supports C99 in the GCC frontend fine, A ANSI C89 requirement would be a exceedingly old/obstructive/obsolescent environment to work with.