[Ubuntu 20.10] - When zlib acceleration is enabled, gzip fails when given multiple files larger than 5KB (gzip)

Bug #1901528 reported by bugproxy
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu on IBM z Systems
Fix Released
High
Skipper Bug Screeners
gzip (Ubuntu)
Fix Released
High
Canonical Foundations Team
Groovy
Fix Released
High
Unassigned

Bug Description

When zlib acceleration is enabled, gzip fails when given multiple files larger than 5KB.

This problem does not happen when running gzip against a single file at a time. It only happens when you provide multiple files as gzip arguments.

So this works whether zlib acceleration is enabled or not:

for file in file1 file2; do gzip $file ; done

But this fails (when zlib acceleration is enabled):

gzip file1 file2

The patch to fix this has been accepted upstream:
https://git.savannah.gnu.org/cgit/gzip.git/commit/?id=be0a534ba2b6e77da289de8da79e70843b1028cc

[Impact]

 * With zlib acceleration enabled, attempting to compress multiple files
   over 5MB would cause a segmentation fault.

 * The files could still be compressed in separate commands

[Test Case]

 * Create two files that are larger than 5MB

 * Enable zlib acceleration (z15 hardware required)

 * Run the command gzip <file1> <file2>

 * NOTE: we do not have access to z15 hardware and therefore are relying
   on IBM to verify this fix

[Where problems could occur]

 * Due to lack of testing resources, it's possible the bug has not been
   fully fixed, and the segmentation fault could still occur.

bugproxy (bugproxy)
tags: added: architecture-s39064 bugnameltc-188840 severity-high targetmilestone-inin2010
Changed in ubuntu:
assignee: nobody → Skipper Bug Screeners (skipper-screen-team)
affects: ubuntu → zlib (Ubuntu)
Frank Heimes (fheimes)
Changed in ubuntu-z-systems:
importance: Undecided → High
assignee: nobody → Canonical Foundations Team (canonical-foundations)
Steve Langasek (vorlon)
tags: added: fr-859
Frank Heimes (fheimes)
Changed in zlib (Ubuntu):
assignee: Skipper Bug Screeners (skipper-screen-team) → nobody
Frank Heimes (fheimes)
Changed in gzip (Ubuntu):
assignee: nobody → Canonical Foundations Team (canonical-foundations)
Changed in zlib (Ubuntu):
assignee: nobody → Canonical Foundations Team (canonical-foundations)
Changed in ubuntu-z-systems:
assignee: Canonical Foundations Team (canonical-foundations) → Skipper Bug Screeners (skipper-screen-team)
Changed in gzip (Ubuntu):
status: New → In Progress
Changed in zlib (Ubuntu):
status: New → Invalid
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package gzip - 1.10-2ubuntu2

---------------
gzip (1.10-2ubuntu2) hirsute; urgency=medium

  [ William 'jawn-smith' Wilson ]
  * Applying patch from upstream to fix a segfault caused by passing
    multiple files larger than 5kb to a gzip command while zlib
    acceleration is enabled (LP: #1901528)

 -- Brian Murray <email address hidden> Mon, 18 Jan 2021 10:35:03 -0800

Changed in gzip (Ubuntu):
status: In Progress → Fix Released
Revision history for this message
Matthieu Clemenceau (mclemenceau) wrote :

Frank, This is resolved in Hirsute, Do you also want this in Groovy? If yes, can you add it as a affected serie?
Thx

Revision history for this message
Dimitri John Ledkov (xnox) wrote :

Based on tags, it is requested for Groovy too.

Changed in gzip (Ubuntu Groovy):
status: New → Confirmed
Changed in zlib (Ubuntu Groovy):
status: New → Invalid
Revision history for this message
Frank Heimes (fheimes) wrote :

+1 (according to title prefix and tag 'targetmilestone-inin2010 ')

description: updated
Revision history for this message
Steve Langasek (vorlon) wrote : Please test proposed package

Hello bugproxy, or anyone else affected,

Accepted gzip into groovy-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/gzip/1.10-2ubuntu1.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-needed-groovy to verification-done-groovy. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-groovy. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Changed in gzip (Ubuntu Groovy):
status: Confirmed → Fix Committed
tags: added: verification-needed verification-needed-groovy
Frank Heimes (fheimes)
Changed in ubuntu-z-systems:
status: New → Fix Committed
Revision history for this message
bugproxy (bugproxy) wrote : Comment bridged from LTC Bugzilla

------- Comment From <email address hidden> 2021-01-20 05:23 EDT-------
s390x build of gzip 1.10-2ubuntu1.1 fixes the issue.

Revision history for this message
Frank Heimes (fheimes) wrote :

Thx Ilya for the verification on groovy - I'm adjusting the tags accordingly ...

tags: added: verification-done verification-done-groovy
removed: verification-needed verification-needed-groovy
Mathew Hodson (mhodson)
no longer affects: zlib (Ubuntu)
no longer affects: zlib (Ubuntu Groovy)
Changed in gzip (Ubuntu):
importance: Undecided → High
Changed in gzip (Ubuntu Groovy):
importance: Undecided → High
Revision history for this message
Łukasz Zemczak (sil2100) wrote : Update Released

The verification of the Stable Release Update for gzip has completed successfully and the package is now being released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package gzip - 1.10-2ubuntu1.1

---------------
gzip (1.10-2ubuntu1.1) groovy; urgency=medium

  * Applying patch from upstream to fix a segfault caused by passing
    multiple files larger than 5kb to a gzip command while zlib
    acceleration is enabled (LP: #1901528)

 -- William 'jawn-smith' Wilson <email address hidden> Tue, 19 Jan 2021 12:57:57 -0600

Changed in gzip (Ubuntu Groovy):
status: Fix Committed → Fix Released
Frank Heimes (fheimes)
Changed in ubuntu-z-systems:
status: Fix Committed → Fix Released
Revision history for this message
bugproxy (bugproxy) wrote : Comment bridged from LTC Bugzilla

------- Comment From <email address hidden> 2021-01-28 09:27 EDT-------
IBM Bugzilla status->closed, Fix Released with groovy

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.