Bug 760060 - update of nss breaks libcurl
Summary: update of nss breaks libcurl
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: curl
Version: rawhide
Hardware: Unspecified
OS: Linux
high
medium
Target Milestone: ---
Assignee: Kamil Dudka
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-12-05 11:33 UTC by Kamil Dudka
Modified: 2012-01-02 20:35 UTC (History)
6 users (show)

Fixed In Version: curl-7.23.0-5.fc17
Clone Of:
Environment:
Last Closed: 2012-01-02 20:35:15 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
changes made from nss-3.13.7-7 to nss-3.13.1-9 (4.00 KB, patch)
2011-12-22 18:23 UTC, Elio Maldonado Batiz
no flags Details | Diff

Description Kamil Dudka 2011-12-05 11:33:10 UTC
Description of problem:
FTPS tests are now skipped in Koji.  On s390(x), they are sometimes not skipped, which causes the build to fail:

http://s390.koji.fedoraproject.org/koji/taskinfo?taskID=511457


Version-Release number of selected component (if applicable):
curl-7.23.0-2.fc17


Actual results:
http://s390.koji.fedoraproject.org/koji/taskinfo?taskID=511457


Expected results:
FTPS regression tests passing


Additional info:
Originally reported by Daniel Horák.

Comment 1 Kamil Dudka 2011-12-19 13:13:20 UTC
Although I am able to reproduce the issue locally, I was not able to find out the root cause yet.  Here is my up to now investigation:

- on a RHEL-6 machine, I am able to run the FTPS tests for all supported RHEL-6/Fedora versions of curl

- on a rawhide machine, I am not able to run the FTPS tests on any supported RHEL-6/Fedora versions of curl

- installing RHEL-6 stunnel on the rawhide machine did not change anything

- switching SELinux to permissive mode on the rawhide machine did not change anything

I am quite sure that it was not caused by any change in curl itself, but will need some additional time to narrow it down.

Comment 2 Kamil Dudka 2011-12-20 12:44:36 UTC
After downgrading nss{,-sysinit,-devel} to 3.12.x, everything started to work again.  As there are no related changes in libcurl, I am switching the component to nss.

Version-Release number of selected component (if applicable):
nss-3.13.1-9.fc17

Comment 3 Elio Maldonado Batiz 2011-12-20 17:41:05 UTC
Following http://s390.koji.fedoraproject.org/koji/taskinfo?taskID=511457
I see "BuildError: error building package (arch s390x), mock exited with status 1; see build.log for more information"
But there is no build.log file for one to examine.

Comment 5 Dan Horák 2011-12-21 13:06:50 UTC
Kamil, should Elio see the failures in FTPS on any arch or does he need s390(x) which I can provide if needed? The original logs were deleted on the s390x koji hub.

Comment 6 Kamil Dudka 2011-12-22 14:13:02 UTC
(In reply to comment #4)
> I'm curious whether the problem shows up with nss-3.13.1-7.fc17 as well.

Good point.  nss-3.13.1-7.fc17 indeed works much better.  The FTPS tests are started and, with valgrind disabled, they pass with no problems.  With valgrind enabled, they fail because of bug 769616, but that seems to be an orthogonal issue.

Elio, are you aware of any related changes between -7 and -9?

(In reply to comment #5)
> Kamil, should Elio see the failures in FTPS on any arch or does he need s390(x)
> which I can provide if needed?

Nope, I can repeat the problem on an x86_64 rawhide machine.  It does not seem to be s390(x) specific at all.

Comment 7 Elio Maldonado Batiz 2011-12-22 18:23:59 UTC
Created attachment 549228 [details]
changes made from nss-3.13.7-7 to nss-3.13.1-9


(In reply to comment #6)
> Elio, are you aware of any related changes between -7 and -9?

Yes, I added one patch going from -7 to -8 and dropped one patch from -8 to -9 as the attached diff file shows.

The nss-3.13.1-7 build forms the basis for the merge to the new git repo for rhel. It has a patch, nosha224.path, to account for the fact that the sha224 algorithm isn't available in the old softoken from 3.12.9 that we are required to use in RHEL. It was used for the merge for the new git repo for future rhel.

Going to nss-3.13.1-8 I added your patch to fix Bug 754744
"- Resolves: Bug 754771 - [PEM] an unregistered callback causes a SIGSEGV"

3) For nss-3.13.1-9 I droppped the previously mentioned nosha224.patch as in fedora we do have, and want to, use the latest softoken from nss-3.13.1.

Comment 8 Elio Maldonado Batiz 2011-12-22 18:33:10 UTC
Reviewing attachment 549228 [details], a big change is the softoken version we are using.
From -7 and -8 where we have
Requires:         nss-softokn%{_isa} >= %{nss_softokn_fips_version}
 to -9 that has
Requires:         nss-softokn%{_isa} >= %{nss_softokn_version}

where nss_softokn_fips_version = 3.12.9 and nss_softokn_version = 3.13.3

Comment 9 Kamil Dudka 2011-12-22 21:14:56 UTC
Ooops, now I am confused as it does not work for me with nss-3.13.1-7.fc17 any more :-/  Either my previous testing was bogus, or the big update of packages meanwhile took effect.  I will need to check yum.log and figure out what happened...

Comment 10 Elio Maldonado Batiz 2011-12-22 21:25:57 UTC
Are all nss-related pacakses present in the buildroot override? I ask because Jan had problems trying some rebuilds for a firefox/thunderbid/xulrunner/nss/softokn/util bundle for bodhi because some of the lower level nss packages (nss-softokn and nss-util) buildroot overrides where expiring while nss was still there.

Comment 11 Elio Maldonado Batiz 2011-12-22 21:38:27 UTC
Ah, this refers to tests failing not the build, if I understand previous comments correctly. Pardon my ignorance, but what does FTPS mean?

Comment 12 Kamil Dudka 2011-12-24 09:36:35 UTC
Simply speaking, it is an FTP protocol that runs over TLS:

http://tools.ietf.org/html/rfc4217

Comment 13 Kamil Dudka 2011-12-27 13:16:04 UTC
This bug is caused by the following commit:

https://bugzilla.mozilla.org/show_bug.cgi?id=665814#c112

Comment 14 Kamil Dudka 2011-12-27 14:07:26 UTC
I have pushed a workaround that allows to run the FTPS tests during build:

http://pkgs.fedoraproject.org/gitweb/?p=curl.git;a=commitdiff;h=4baedf3

Comment 15 Kamil Dudka 2011-12-27 19:09:48 UTC
Switching back to curl.  I have proposed a solution upstream:

http://curl.haxx.se/mail/lib-2011-12/0291.html

Comment 16 Kamil Dudka 2012-01-02 20:35:15 UTC
fixed in curl-7.23.0-5.fc17


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