Bug 1921246

Summary:	split can cause data corruption with --number=K/N construct
Product:	[Fedora] Fedora	Reporter:	Pádraig Brady <p>
Component:	coreutils	Assignee:	Kamil Dudka <kdudka>
Status:	CLOSED ERRATA	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	high	Docs Contact:
Priority:	unspecified
Version:	32	CC:	admiller, jamartis, jarodwilson, kdudka, kzak, ooprala, ovasik, p, sebastian.kisela, svashisht
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	coreutils-8.32-17.fc34 coreutils-8.32-17.fc33 coreutils-8.32-4.fc32.2	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:
Clones:	1926138 (view as bug list)		Environment:
Last Closed:	2021-02-06 01:16:53 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	1926138

Description Pádraig Brady 2021-01-27 19:08:02 UTC

Description of problem:
split --number=K/N can be used when there isn't enough local storage,
and you want to split input to separate output devices in separate runs.

Ever since v8.26 (so including centos 8), there is a nasty
data corruption issue for any file over 128KiB
as part of the input file is skipped incorrectly.

The upstream fix for this is:
https://github.com/coreutils/coreutils/commit/bb21daa1.patch

Comment 1 Kamil Dudka 2021-02-02 14:51:39 UTC

Thank you for reporting the bug!

dist-git commit: https://src.fedoraproject.org/rpms/coreutils/c/7afd521b

Comment 2 Fedora Update System 2021-02-03 01:31:14 UTC

FEDORA-2021-a56b2d9e9d has been pushed to the Fedora 32 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf upgrade --enablerepo=updates-testing --advisory=FEDORA-2021-a56b2d9e9d`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2021-a56b2d9e9d

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 3 Fedora Update System 2021-02-03 02:30:05 UTC

FEDORA-2021-74e0c6f8bd has been pushed to the Fedora 33 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf upgrade --enablerepo=updates-testing --advisory=FEDORA-2021-74e0c6f8bd`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2021-74e0c6f8bd

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 4 Pádraig Brady 2021-02-05 19:00:36 UTC

Is there a bug tracking this for the centos/rhel 8 side of things.
I didn't find one when searching for "split" in the RHEL 8 component

Comment 5 Fedora Update System 2021-02-06 01:16:53 UTC

FEDORA-2021-74e0c6f8bd has been pushed to the Fedora 33 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 6 Kamil Dudka 2021-02-08 10:25:23 UTC

(In reply to Pádraig Brady from comment #4)
> Is there a bug tracking this for the centos/rhel 8 side of things.
> I didn't find one when searching for "split" in the RHEL 8 component

We do not clone Fedora bugs for RHEL proactively, only the other way around.  Given the fact that the upstream patch applies cleanly and fixes a possible data loss, I will clone this one for eventual inclusion in the next update of coreutils in RHEL-8.  Thanks for the hint!

Comment 7 Fedora Update System 2021-02-17 05:08:25 UTC

FEDORA-2021-a56b2d9e9d has been pushed to the Fedora 32 stable repository.
If problem still persists, please make note of it in this bug report.