Removed ferror(NULL) from lms_common

danielinux · danielinux · commit aaf40aa0185d · 2024-07-18T10:55:25.000+02:00
diff --git a/tools/lms/lms_common.h b/tools/lms/lms_common.h
@@ -48,8 +48,7 @@ static int lms_write_key(const byte * priv, word32 privSz, void * context)
         /* Create the file if it didn't exist. */
         file = fopen(filename, "w+");
         if (!file) {
-            fprintf(stderr, "error: fopen(%s, \"w+\") failed: %d\n", filename,
-                    ferror(file));
+            fprintf(stderr, "error: fopen(%s, \"w+\") failed.\n", filename);
             return WC_LMS_RC_WRITE_FAIL;
         }
     }
@@ -72,8 +71,7 @@ static int lms_write_key(const byte * priv, word32 privSz, void * context)
      * storage correctly. */
     file = fopen(filename, "r+");
     if (!file) {
-        fprintf(stderr, "error: fopen(%s, \"r+\") failed: %d\n", filename,
-                ferror(file));
+        fprintf(stderr, "error: fopen(%s, \"r+\") failed.\n", filename);
         return WC_LMS_RC_WRITE_FAIL;
     }
 

Original file line number	Diff line number	Diff line change
`@@ -48,8 +48,7 @@ static int lms_write_key(const byte * priv, word32 privSz, void * context)`
`48`	`48`	`/* Create the file if it didn't exist. */`
`49`	`49`	`file = fopen(filename, "w+");`
`50`	`50`	`if (!file) {`
`51`		`- fprintf(stderr, "error: fopen(%s, \"w+\") failed: %d\n", filename,`
`52`		`- ferror(file));`
	`51`	`+ fprintf(stderr, "error: fopen(%s, \"w+\") failed.\n", filename);`
`53`	`52`	`return WC_LMS_RC_WRITE_FAIL;`
`54`	`53`	`}`
`55`	`54`	`}`
`@@ -72,8 +71,7 @@ static int lms_write_key(const byte * priv, word32 privSz, void * context)`
`72`	`71`	`* storage correctly. */`
`73`	`72`	`file = fopen(filename, "r+");`
`74`	`73`	`if (!file) {`
`75`		`- fprintf(stderr, "error: fopen(%s, \"r+\") failed: %d\n", filename,`
`76`		`- ferror(file));`
	`74`	`+ fprintf(stderr, "error: fopen(%s, \"r+\") failed.\n", filename);`
`77`	`75`	`return WC_LMS_RC_WRITE_FAIL;`
`78`	`76`	`}`
`79`	`77`