From d101f917c875492051d620e1abc9580f079e2fa3 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Lo=C3=AFMinier?= Date: Mon, 14 Mar 2011 13:01:53 +0530 Subject: [PATCH] Add error checking to all file functions MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit When building with a fortify-enabled toolchain, one would get: scripts/signGP.c:305:8: warning: ignoring return value of .fread., declared with attribute warn_unused_result Signed-off-by: LoïMinier Signed-off-by: Anand Gadiyar --- scripts/signGP.c | 21 ++++++++++++++------- 1 file changed, 14 insertions(+), 7 deletions(-) diff --git a/scripts/signGP.c b/scripts/signGP.c index a57124a..ab4715c 100644 --- a/scripts/signGP.c +++ b/scripts/signGP.c @@ -290,17 +290,24 @@ int main(int argc, char *argv[]) } if (ch_add) - fwrite(&config_header, 1, 512, ofile); + if (fwrite(&config_header, 1, 512, ofile) <= 0) + pdie("fwrite"); - fwrite(&len, 1, 4, ofile); - fwrite(&loadaddr, 1, 4, ofile); + if (fwrite(&len, 1, 4, ofile) <= 0) + pdie("fwrite"); + if (fwrite(&loadaddr, 1, 4, ofile) <= 0) + pdie("fwrite"); for (i = 0; i < len; i++) { - fread(&ch, 1, 1, ifile); - fwrite(&ch, 1, 1, ofile); + if (fread(&ch, 1, 1, ifile) <= 0) + pdie("fread"); + if (fwrite(&ch, 1, 1, ofile) <= 0) + pdie("fwrite"); } - fclose(ifile); - fclose(ofile); + if (!fclose(ifile)) + perror("warning: fclose"); + if (!fclose(ofile)) + perror("warning: fclose"); return 0; } -- 2.39.5