Change in coreboot[master]: amdfwtool: Copy string in a safer way

[Felix Held (Code Review)]

Attention is currently required from: Bao Zheng, Jason Glenesk, Raul Rangel, Martin Roth, Paul Menzel, Zheng Bao.
Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/54946 )

Change subject: amdfwtool: Copy string in a safer way
......................................................................

Patch Set 13: Code-Review+1

(1 comment)

File util/amdfwtool/data_parse.c:

https://review.coreboot.org/c/coreboot/+/54946/comment/5dde0339_398c24bf 
PS13, Line 453: + 2
not 100% sure if this needs to be + 2 or if + 1 would be sufficient.
dir has a maximum length of MAX_LINE_SIZE including the final null-byte. from &(oneline[match[2].rm_so]) a maximum of MAX_LINE_SIZE bytes will be used, which don't have to include the final null-byte. with the / in between, i'd assume that we need at most MAX_LINE_SIZE * 2 + 1 bytes: MAX_LINE_SIZE - 1 for dir, 1 for / and MAX_LINE_SIZE + 1 for &(oneline[match[2].rm_so]). still better allocation one byte more than we'll need that not allocating enough bytes