mprintf: check fputc error rather than matching returned character

OS/400 ascii fputc wrapper deviates from the posix standard by the
fact that it returns the ebcdic encoding of the original ascii
character. Testing for a matching value for success will then always
fail.

This commit replaces the chariacter comparison by an explicit error
return check.

Follow-up to ef2cf58
Closes #13367

diff --git a/lib/mprintf.c b/lib/mprintf.c
index 4c60d13305b2890bbc19d9cb93be924ee1c6bee8..3ce3a7788c21e3e5c77b8ad246701c851506f38a 100644 (file)
--- a/lib/mprintf.c
+++ b/lib/mprintf.c
@@ -1160,9 +1160,7 @@ static int fputc_wrapper(unsigned char outc, void *f)
   int out = outc;
   FILE *s = f;
   int rc = fputc(out, s);
-  if(rc == out)
-    return 0;
-  return 1;
+  return rc == EOF;
 }
 
 int curl_mprintf(const char *format, ...)