Bug 591889

Summary: nss_pcache dies by a sighup
Product: [Fedora] Fedora Reporter: Joshua Roys <roysjosh>
Component: mod_nssAssignee: Rob Crittenden <rcritten>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: low    
Version: 12CC: josh.kayse, rcritten
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: mod_nss-1.0.8-6.fc12 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2010-06-04 18:46:08 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
signal(SIGHUP, SIG_IGN) none

Description Joshua Roys 2010-05-13 12:44:15 UTC
Created attachment 413751 [details]
signal(SIGHUP, SIG_IGN)

Description of problem:
nss_pcache dies by a sighup, causing mod_nss to behave badly

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

How reproducible:
configure mod_nss as the ssl provider for httpd

Steps to Reproduce:
1. configure mod_nss/httpd
2. start it
3. watch httpd children scream as they can't open the password-protected cert db because nss_pcache died
  
Actual results:
non-working ssl

Expected results:
working httpd w/ssl via mod_nss

Comment 1 Rob Crittenden 2010-05-13 14:59:16 UTC
What would send nss_pcache a sighup?

Comment 2 Joshua Roys 2010-05-13 15:26:43 UTC
We would notice it whenever we started httpd via the initscripts... we use a strict/mls selinux policy, so we use run_init:
run_init service httpd start

And then nss_pcache would die.  A strace would reveal it getting sent a sighup when the parent (IIRC, I can double check later) process started to close the std{in,out,err} file descriptors.  It almost seemed to be a race condition, because sometime it wouldn't die - usually under a debugger or something else that would slow various pieces down.

man 7 signal:
       SIGHUP        1       Term    Hangup detected on controlling terminal
                                     or death of controlling process

Comment 3 Rob Crittenden 2010-05-14 15:49:08 UTC
Very easy to reproduce, I don't see a problem ignoring this signal.

Committed upstream.

Checking in nss_pcache.c;
/cvs/dirsec/mod_nss/nss_pcache.c,v  <--  nss_pcache.c
new revision: 1.11; previous revision: 1.10
done

Comment 4 Fedora Update System 2010-05-17 20:23:27 UTC
mod_nss-1.0.8-6.fc12 has been submitted as an update for Fedora 12.
http://admin.fedoraproject.org/updates/mod_nss-1.0.8-6.fc12

Comment 5 Fedora Update System 2010-05-17 20:23:36 UTC
mod_nss-1.0.8-6.fc13 has been submitted as an update for Fedora 13.
http://admin.fedoraproject.org/updates/mod_nss-1.0.8-6.fc13

Comment 6 Fedora Update System 2010-05-18 21:56:03 UTC
mod_nss-1.0.8-6.fc13 has been pushed to the Fedora 13 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update mod_nss'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/mod_nss-1.0.8-6.fc13

Comment 7 Fedora Update System 2010-05-18 21:56:48 UTC
mod_nss-1.0.8-6.fc12 has been pushed to the Fedora 12 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update mod_nss'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/mod_nss-1.0.8-6.fc12

Comment 8 Fedora Update System 2010-06-04 18:45:59 UTC
mod_nss-1.0.8-6.fc13 has been pushed to the Fedora 13 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 9 Fedora Update System 2010-06-04 18:51:46 UTC
mod_nss-1.0.8-6.fc12 has been pushed to the Fedora 12 stable repository.  If problems still persist, please make note of it in this bug report.