[coreboot-gerrit] Change in coreboot[master]: util/ifdtool: add --output flag

[Marcello Sylvester Bauer (Code Review)]

Marcello Sylvester Bauer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38828 )

Change subject: util/ifdtool: add --output flag
......................................................................

Patch Set 3:

(1 comment)
> Patch Set 2:
>
> (1 comment)
>
> The new flag will only work for mode_newlayout as the other commands used to operate in place. You have to either copy the file contents first or limit -O to newlayout generation.

All operations uses the write_image() function, which creates a new image with a ".new" suffix.

https://review.coreboot.org/c/coreboot/+/38828/2/util/ifdtool/ifdtool.c 
File util/ifdtool/ifdtool.c:

https://review.coreboot.org/c/coreboot/+/38828/2/util/ifdtool/ifdtool.c@1741 
PS2, Line 1741: malloc((strlen(filename))
> Strlen(Filename) +5

Done