Attention is currently required from: AtomicFS, Stefan Reinauer.
Jan Samek has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/80432?usp=email )
Change subject: util/ifdtool.c: Fix long_options platform has argument ......................................................................
Patch Set 1:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/80432/comment/fc56d8a8_31ab4dd4 : PS1, Line 9: Currently there is different behavior between using short and long : option. Long option causes Segmentation fault. : I think it is safe just to say the flag `has_arg` was mistakingly set to `0` while the option apparently takes an argument (`:` in optstring and also the help text says so) and this patch fixes the undefined behaviour.
The exact behaviour could differ as the pointer to the argument might AFAIK become undefined when `has_arg` is set to `0`. Hence the segfaults.