Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.

Bug 119987

Summary: initscripts code cleanup
Product: [Fedora] Fedora Reporter: Steve Grubb <linux_4ever>
Component: initscriptsAssignee: Bill Nottingham <notting>
Status: CLOSED RAWHIDE QA Contact: Brock Organ <borgan>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: rvokal
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: 7,.51-1 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2004-05-04 16:28:21 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:
Bug Depends On:    
Bug Blocks: 114963    
Attachments:
Description Flags
Bug fix patch none

Description Steve Grubb 2004-04-04 17:48:11 UTC 
From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux i586; en-US; rv:1.4.2)
Gecko/20040308

Description of problem:
During code review found: leaked memory, not properly checking file
descriptors before using them, a call to poll that returned -1 because
of EINTR is being processed as if it were a valid return from poll,
return code from forkCommand not being used at all, testing of device
file susceptible to races in usernetctl, and userctl tests were
unneccessarily case sensitive.

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

How reproducible:
Always

Steps to Reproduce:
Found by code inpection

Additional info:

I have a patch that I will attach. Please consider applying it before
fedora core 2 final.
Comment 1 Steve Grubb 2004-04-04 17:49:52 UTC 
Created attachment 99099 [details]
Bug fix patch

Please consider applying this.
Comment 2 Bill Nottingham 2004-05-04 16:28:21 UTC 
Someone actually looked through process.c. Yikes, my condolances (it
should really be taken out and shot.)

Some of the forkCommand stuff is only correct due to the fact that
it's only called with certain arguments; but that's ok.

Thanks for looking at this... will be in 7.51-1.