aio: Ignore error indication on eof

Reading from a pipe can set both ferror() and feof().
In this case, feof() takes precedence.

Signed-off-by: Steve Bennett <steveb@workware.net.au>

1 files changed, 8 insertions, 5 deletions

diff --git a/jim-aio.c b/jim-aio.c
index e5e8898..8f7b12e 100644
--- a/jim-aio.c
+++ b/jim-aio.c
@@ -344,12 +344,15 @@ static int aio_cmd_read(Jim_Interp *interp, int argc, Jim_Obj *const *argv)
             break;
     }
     /* Check for error conditions */
-    if (ferror(af->fp) && errno != EAGAIN) {
-        /* I/O error */
-        Jim_FreeNewObj(interp, objPtr);
-        JimAioSetError(interp, af->filename);
+    if (ferror(af->fp)) {
         clearerr(af->fp);
-        return JIM_ERR;
+        /* eof and EAGAIN are not error conditions */
+        if (!feof(af->fp) && errno != EAGAIN) {
+            /* I/O error */
+            Jim_FreeNewObj(interp, objPtr);
+            JimAioSetError(interp, af->filename);
+            return JIM_ERR;
+        }
     }
     if (nonewline) {
         int len;