libbb: fix CRLF handling

Ensure a trailing CR is only removed if it precedes a LF.

The two cases at issue are intended to read complete lines and remove
the line terminator.  In the normal case a trailing LF will be present
so removing the CR unconditionally worked.  However, if the last line
of a file was missing its LF or if a NUL was detected (in the case of
xmalloc_fgetline()) the CR might have been removed without justification.

1 files changed, 7 insertions, 2 deletions

diff --git a/libbb/get_line_from_file.c b/libbb/get_line_from_file.c
index ce5c7810e..9f20f68cf 100644
--- a/libbb/get_line_from_file.c
+++ b/libbb/get_line_from_file.c
@@ -66,11 +66,16 @@ char* FAST_FUNC xmalloc_fgetline(FILE *file)
         size_t i;
         char *c = bb_get_chunk_from_file(file, &i);
 
+#if !ENABLE_PLATFORM_MINGW32
         if (i && c[--i] == '\n')
                 c[i] = '\0';
-#if ENABLE_PLATFORM_MINGW32
-        if (i && c[--i] == '\r')
+#else
+        if (i && c[--i] == '\n') {
                 c[i] = '\0';
+                if (i && c[--i] == '\r') {
+                        c[i] = '\0';
+                }
+        }
 #endif
 
         return c;