Bug 675207 - RFE: gzdirect() function needs to be backported to fix BZ #675172
Summary: RFE: gzdirect() function needs to be backported to fix BZ #675172
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Red Hat Enterprise Linux 4
Classification: Red Hat
Component: zlib
Version: 4.8
Hardware: Unspecified
OS: Unspecified
urgent
high
Target Milestone: rc
: ---
Assignee: Peter Schiffer
QA Contact: qe-baseos-daemons
URL:
Whiteboard:
Depends On:
Blocks: 675172
TreeView+ depends on / blocked
 
Reported: 2011-02-04 16:03 UTC by Martin Osvald 🛹
Modified: 2018-11-14 15:57 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Enhancement
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-06-14 20:13:27 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
a code required to get gzdirect() into zlib (1.25 KB, patch)
2011-02-04 16:03 UTC, Martin Osvald 🛹
no flags Details | Diff

Description Martin Osvald 🛹 2011-02-04 16:03:20 UTC
Created attachment 477052 [details]
a code required to get gzdirect() into zlib

Description of problem:

There is a bug in libxml2 library (BZ #675172), which can be fixed only in case zlib would contain gzdirect() function (see BZ #675172 comment #2 for more info). gzdirect() function is included in upstream version 1.2.2.3, but RHEL4 contains 1.2.1.2. 

Version-Release number of selected component (if applicable):

zlib-1.2.1.2-1.2

How reproducible:

N/A

Steps to Reproduce:

N/A
  
Actual results:

BZ #675172 cannot be fixed due to missing gzdirect() function

Expected results:

gzdirect() in zlib

Additional info:

a patch to show the required changes attached

Comment 1 RHEL Program Management 2011-02-04 16:08:48 UTC
This request was evaluated by Red Hat Product Management for
inclusion in the current release of Red Hat Enterprise Linux.
Because the affected component is not scheduled to be updated in the
current release, Red Hat is unfortunately unable to address this
request at this time. Red Hat invites you to ask your support
representative to propose this request, if appropriate and relevant,
in the next release of Red Hat Enterprise Linux.

Comment 2 Ivana Varekova 2011-03-14 12:41:02 UTC
Only important issues like performance and security are fixed in RHEL 4.9 and
later. This bug does not meet necessary criteria and is therefore closed.

Comment 3 Marcelo Moreira de Mello 2011-05-16 14:57:52 UTC
Hello, 

I was working with Martin in case SFDC#00394024 investigating the segmentation
fault issue in up2date. 

Due the excellent Martin analysis, we could verify that the issue occurs
exactly at strncmp() function. 


2194                    if (gzread(context, buff4, 4) == 4) {
2195                        if (strncmp(buff4, cptr, 4) == 0)
2196                            ret->compressed = 0;
2197                        else
2198                            ret->compressed = 1;


Due the work made in other BZ#664060[1], we back-ported the upstream patches to
RHEL4 packages. 

We sent the back-ported packages to a customer and customer said that packages
worked as excepted. Below, the customer note: 

"Hello, 

I have a good news :)
With the update of the libxml2, libxml2-python, zlib-devel, zlib packages
Now it works. 

I registered the client to the redhat network!

You can close this call.
Thank you for your help.

Best regards

Yvan"

Those RHEL4 packages are available at
http://people.redhat.com/mdemello/fixes/SFDC00394024. 

The internal CVS branches are: 

   libxml2-2_6_16-12_8_TEST_SFDC00394024
   zlib-1_2_1_2-1_2_TEST_SFDC00394024

Related BZs: 

   * https://bugzilla.redhat.com/show_bug.cgi?id=675207
   * https://bugzilla.redhat.com/show_bug.cgi?id=664060

Since the issue is clearly, we would like to request an official back-port for
zlib and libxml2 packages for RHEL4. 

Thank you. 

Cheers, 

Marcelo Moreira de Mello


[1] - https://bugzilla.redhat.com/show_bug.cgi?id=664060

Comment 4 Marcelo Moreira de Mello 2011-05-16 14:58:46 UTC
*** BZ were re-opened ****

Comment 7 Ivana Varekova 2011-05-17 08:01:22 UTC
Patch looks ok (higher version of zlib 1.2.5 - have a feature which enable the function to return the state->direct in more situation, but the present version seems which is already tested by the customer is sufficent).


Note You need to log in before you can comment on or make changes to this bug.