sort: properly check for empty reads

src/uu/sort/src/chunks.rs

@@ -102,17 +102,17 @@ pub fn read(
         carry_over.clear();
         carry_over.extend_from_slice(&buffer[read..]);
 
-        let payload = Chunk::new(buffer, |buf| {
+        if read != 0 {
-            let mut lines = unsafe {
+            let payload = Chunk::new(buffer, |buf| {
-                // SAFETY: It is safe to transmute to a vector of lines with shorter lifetime,
+                let mut lines = unsafe {
-                // because it was only temporarily transmuted to a Vec<Line<'static>> to make recycling possible.
+                    // SAFETY: It is safe to transmute to a vector of lines with shorter lifetime,
-                std::mem::transmute::<Vec<Line<'static>>, Vec<Line<'_>>>(lines)
+                    // because it was only temporarily transmuted to a Vec<Line<'static>> to make recycling possible.
-            };
+                    std::mem::transmute::<Vec<Line<'static>>, Vec<Line<'_>>>(lines)
-            let read = crash_if_err!(1, std::str::from_utf8(&buf[..read]));
+                };
-            parse_lines(read, &mut lines, separator, &settings);
+                let read = crash_if_err!(1, std::str::from_utf8(&buf[..read]));
-            lines
+                parse_lines(read, &mut lines, separator, &settings);
-        });
+                lines
-        if !payload.borrow_lines().is_empty() {
+            });
             sender.send(payload).unwrap();
         }
         if !should_continue {

tests/by-util/test_sort.rs

@@ -800,3 +800,12 @@ fn sort_multiple() {
         .succeeds()
         .stdout_is("a\nb\nb\n");
 }
+
+#[test]
+fn sort_empty_chunk() {
+    new_ucmd!()
+        .args(&["-S", "40B"])
+        .pipe_in("a\na\n")
+        .succeeds()
+        .stdout_is("a\na\n");
+}