Bug 2816 - Redhat init-script does not set RETVAL on status requests
Summary: Redhat init-script does not set RETVAL on status requests
Status: RESOLVED FIXED
Alias: None
Product: Spamassassin
Classification: Unclassified
Component: spamc/spamd (show other bugs)
Version: 2.60
Hardware: PC Linux
: P5 normal
Target Milestone: 3.0.0
Assignee: SpamAssassin Developer Mailing List
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-12-08 02:48 UTC by Linus Sj
Modified: 2004-02-18 10:59 UTC (History)
0 users


Attachment Type Modified Status Actions Submitter/CLA Status
Return RETVAL on queries patch None Linus Sj [NoCLA]
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Linus Sj 2003-12-08 02:48:12 UTC 
The /etc/init.d/spamassassin shipped with spamassassin does not return RETVAL on 
exit, instead it always returns zero. This causes problems with tools that 
automagically monitors running daemons.

The attached patch fixes this by setting RETVAL on status-queries and by exiting 
with RETVAL instead of 0.
Comment 1 Linus Sj 2003-12-08 02:49:07 UTC 
Created attachment 1613 [details]
Return RETVAL on queries
Comment 2 Justin Mason 2003-12-08 12:15:15 UTC 
Subject: Re: [SAdev]  Redhat init-script does not set RETVAL on status requests 

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1


>Created an attachment (id=1613)
> --> (http://bugzilla.spamassassin.org/attachment.cgi?id=1613&action=view)
>Return RETVAL on queries

Looks good.  Could you sign an Apache CLA so we can incorporate
the patch?

	http://www.apache.org/licenses#clas

- --j.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.2 (GNU/Linux)
Comment: Exmh CVS

iD8DBQE/1NvNQTcbUG5Y7woRAgcTAJ47PNyaRxgqCoEOqXIKT2GtA6OuxwCff7Pl
HfdOu/F1qbQp8fXhZMWwJJ4=
=X70b
-----END PGP SIGNATURE-----
Comment 3 Linus Sj 2003-12-08 12:29:52 UTC 
I'll fax it tomorrow (tuesday) morning.
Comment 4 Linus Sj 2004-01-15 05:34:15 UTC 
Apache CLA was submitted a month ago, any chanse of getting the patch 
incorporated?
Comment 5 Theo Van Dinter 2004-02-18 19:59:11 UTC 
took the example of the status section from sshd and put it in our rc script. :)  applied to head.