From 23fe68915b55ea803eb451afe0d061dcc79607a4 Mon Sep 17 00:00:00 2001
From: Erno Kilpelainen <erno.kilpelainen@kone.com>
Date: Fri, 5 Jun 2026 11:04:13 +0000
Subject: [PATCH] Fix Medium Severity Coverity finding: CWE-252 (Ignoring
 number of bytes read ) (https://cwe.mitre.org/data/definitions/252.html)

---
 tools/fftutil.c | 27 +++++++++++++++++----------
 1 file changed, 17 insertions(+), 10 deletions(-)

diff --git a/tools/fftutil.c b/tools/fftutil.c
index 986ac42..0a432c4 100644
--- a/tools/fftutil.c
+++ b/tools/fftutil.c
@@ -88,17 +88,24 @@ void fft_filend_real(FILE * fin,FILE * fout,int *dims,int ndims,int isinverse)
 
     st = kiss_fftndr_alloc(dims, ndims, isinverse, 0, 0);
 
-    while ( fread (ibuf, sizeof(kiss_fft_scalar), insize,  fin) > 0) {
-        if (isinverse) {
-            kiss_fftndri(st,
-                    (kiss_fft_cpx*)ibuf,
-                    (kiss_fft_scalar*)obuf);
-        }else{
-            kiss_fftndr(st,
-                    (kiss_fft_scalar*)ibuf,
-                    (kiss_fft_cpx*)obuf);
+    while (1) {
+        size_t nread = fread(ibuf, sizeof(kiss_fft_scalar), insize, fin);
+        if (nread == (size_t)insize) {
+            if (isinverse) {
+                kiss_fftndri(st,
+                        (kiss_fft_cpx*)ibuf,
+                        (kiss_fft_scalar*)obuf);
+            } else {
+                kiss_fftndr(st,
+                        (kiss_fft_scalar*)ibuf,
+                        (kiss_fft_cpx*)obuf);
+            }
+            fwrite(obuf, sizeof(kiss_fft_scalar), outsize, fout);
+            continue;
         }
-        fwrite (obuf, sizeof(kiss_fft_scalar), outsize,fout);
+        if (nread != 0)
+            fprintf(stderr,"short read on nd real input\n");
+        break;
     }
     free(st);
     free(ibuf);