Bug 237973 - Wrong init script
Summary: Wrong init script
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: dhcpv6
Version: rawhide
Hardware: All
OS: Linux
low
low
Target Milestone: ---
Assignee: David Cantrell
QA Contact:
URL:
Whiteboard:
Depends On: 237970
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-04-26 12:34 UTC by Michal Marciniszyn
Modified: 2014-02-10 23:02 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2007-04-26 18:29:36 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Michal Marciniszyn 2007-04-26 12:34:38 UTC
+++ This bug was initially created as a clone of Bug #237970 +++

Description of problem:
In both init scripts (dhcp6r/dhcp6s) there are several constructions that end
the script with error code 0 when the service is not started. For example:
# Check that networking is up.
[ ${NETWORKING} = "no" ] && exit 0

# Check that files exist
[ -f /usr/sbin/dhcp6r ] || exit 0
[ -f /etc/sysconfig/dhcp6r ] || exit 0
Also there is at least one place with wrong error code (1 if script function is
not implemented (refresh/start_again :-) ) instead of 3) at the end of each script.

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

How reproducible:
always

Steps to Reproduce:
1. service network stop
2. service dhcp6r/s start/status
3. echo $?
  
Actual results:
0

Expected results:
non-zero exit code/status of the service

Additional info:
When fixing this bug, please obbey our init script guidelines and be sure that
status command is run correctly.
Our guidelines are on following two pages:
http://intranet.corp.redhat.com/ic/intranet/InitscriptsSpec.html
http://intranet.corp.redhat.com/ic/intranet/InitscriptGuidelines.html

For an example of the script that returns the error codes correctly and always
runs status see:
http://devserv.devel.redhat.com/~mmarcini/amd

This bug is tracked by 237789.

Comment 1 David Cantrell 2007-04-26 18:29:36 UTC
Fixed in rawhide in dhcpv6-0.10-42.  Thanks.


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