Fix: truncated value in fgetc() usage

Reported by Coverity Scan:
CID 1327417:  API usage errors  (CHAR_IO)
Assigning the return value of "fgetc" to char "c" truncates
its value.

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

diff --git a/include/babeltrace/compat/stdio.h b/include/babeltrace/compat/stdio.h
index 0301e2835269056cb79fa2a23f852eaaff6435fc..41d1df35037052cfbe6b39c2b8f0f23ec2683a4d 100644 (file)
--- a/include/babeltrace/compat/stdio.h
+++ b/include/babeltrace/compat/stdio.h
@@ -84,9 +84,10 @@ ssize_t bt_getline(char **lineptr, size_t *n, FILE *stream)
        }
        for (;;) {
                char c;
+               int ret;
 
-               c = fgetc(stream);
-               if (c == EOF) {
+               ret = fgetc(stream);
+               if (ret == EOF) {
                        if (ferror(stream)) {
                                /* ferror() is set, errno set by fgetc(). */
                                return -1;
@@ -95,6 +96,7 @@ ssize_t bt_getline(char **lineptr, size_t *n, FILE *stream)
                        found_eof = 1;
                        break;
                }
+               c = (char) ret;
                if (linelen == SSIZE_MAX) {
                        errno = EOVERFLOW;
                        return -1;