summaryrefslogtreecommitdiff
path: root/util/ectool
diff options
context:
space:
mode:
authorStefan Reinauer <stefan.reinauer@coreboot.org>2020-10-22 22:16:20 +0000
committerPatrick Georgi <pgeorgi@google.com>2020-10-26 06:56:56 +0000
commit3a7825983c506d5a521862bfa5700a87c525227d (patch)
treedf7c0bb5cc80deb33af60e342eb8e501158dd3a2 /util/ectool
parent9d50efe19dad26eba3285c3295b804490b81b0e7 (diff)
ectool: Don't ignore fgets return code
Change-Id: I12dc449e06dee31b4b0811ab23c6e8635cf31512 Signed-off-by: Stefan Reinauer <stefan.reinauer@coreboot.org> Reviewed-on: https://review.coreboot.org/c/coreboot/+/46667 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Diffstat (limited to 'util/ectool')
-rw-r--r--util/ectool/ec.c5
1 files changed, 4 insertions, 1 deletions
diff --git a/util/ectool/ec.c b/util/ectool/ec.c
index f30d12f976..a189a094b5 100644
--- a/util/ectool/ec.c
+++ b/util/ectool/ec.c
@@ -167,7 +167,10 @@ int get_ec_ports(void)
return -1;
while (!feof(fp) && (data == 0 || cmd == 0)) {
- fgets(line, sizeof(line), fp);
+ if (fgets(line, sizeof(line), fp) == NULL) {
+ fprintf(stderr, "Can not read from /proc/ioports.\n");
+ break;
+ }
if (strstr(line, "EC data") != NULL)
data = strtol(line, NULL, 16);