diff options
author | Maciej Suminski <maciej.suminski@cern.ch> | 2017-08-09 11:46:41 +0200 |
---|---|---|
committer | Nico Huber <nico.h@gmx.de> | 2017-08-22 10:10:34 +0000 |
commit | d5fb99eab679a5b66b64665575ff40b84d01c3b9 (patch) | |
tree | ea567e2ec7ef69cb5e585bd0f4de2320e146687c /util/msrtool/msrtool.c | |
parent | b3dfcb863cdc62cd2cb65e97e0043311b151c558 (diff) |
util/inteltool: Remove duplicated error message, switch to snprintf
Passing a string containing output from strerror() to perror() causes
double error message display. It is also causing segfaults when the
error message is longer than temp_string capacity.
To fix the problems, sterror() call has been removed so the error
message is printed only once. This could be enough to avoid segfaults,
but it is a good practice to limit output size with snprintf().
Change-Id: I5ccc37e404f278cafae0a451c5acaa27d7907cce
Signed-off-by: Maciej Suminski <maciej.suminski@cern.ch>
Reviewed-on: https://review.coreboot.org/21025
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>
Diffstat (limited to 'util/msrtool/msrtool.c')
0 files changed, 0 insertions, 0 deletions