Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 88971 - new feature in gld breaks greylisting
Summary: new feature in gld breaks greylisting
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Unspecified (show other bugs)
Hardware: x86 Linux
: High critical (vote)
Assignee: Net-Mail Packages
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-04-13 08:43 UTC by Alexander Stoll
Modified: 2005-05-20 05:24 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
patch to fix mxgrey=1 crash (mxgrey-patch.diff,375 bytes, patch)
2005-04-20 09:47 UTC, Benjamin Smee (strerror) (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Stoll 2005-04-13 08:43:40 UTC
package update due to GLSA 200504-10 to version 1.5 introduces a new feature "MXGREY" which is very useful, but if activated via "MXGREY=0" in gld.conf, it breaks gld totally, postfix isn
Comment 1 Alexander Stoll 2005-04-13 08:43:40 UTC
package update due to GLSA 200504-10 to version 1.5 introduces a new feature "MXGREY" which is very useful, but if activated via "MXGREY=0" in gld.conf, it breaks gld totally, postfix isn´t able to communicate to the policy daemon process anymore:
[postfix/smtpd] warning: premature end-of-input on 127.0.0.1:2525 while reading input attribute name
[postfix/smtpd] warning: problem talking to server 127.0.0.1:2525: Success

Then it rejects mail with server configuration problem.


Reproducible: Always
Steps to Reproduce:
1. update to gld 1.5
2. activate feature
3. watch postfix log

Actual Results:  
Postfix rejects email

Expected Results:  
communicate with postfix
Comment 2 Alexander Stoll 2005-04-13 15:58:01 UTC
The feature "MXGREY" is activated via "MXGREY=1" of course... sorry for hasty cut & paste...
Further digging in logfiles shows segfaults by gld, it seems to be the cause for lost input postfix is complaining...
Comment 3 Andrej Kacian (RETIRED) gentoo-dev 2005-04-13 16:09:05 UTC
As I just tested, gld-1.5 ebuild only installs /etc/gld.conf.sample, with MXGREY set to 0.
Comment 4 Alexander Stoll 2005-04-14 01:23:59 UTC
That
Comment 5 Alexander Stoll 2005-04-14 01:23:59 UTC
That´s correct, sample conf sets "MXGREY=0", but I think a advertized and very useful feature should not cause segfaults of the daemon process and postfix rejects email because of that.
This version is heavily broken if this could happen...
If time permits, I will do a debug build and a backtrace when this error is not 	comprehensible by everyone.
Comment 6 Benjamin Smee (strerror) (RETIRED) gentoo-dev 2005-04-20 09:12:50 UTC
Confirmed I can replicate this bug. Will provide more info as I get it.
Comment 7 Benjamin Smee (strerror) (RETIRED) gentoo-dev 2005-04-20 09:30:07 UTC
Further debugging shows that it is breaking right when gld tries to connect to the database. I think this is an upstream bug.
Comment 8 Benjamin Smee (strerror) (RETIRED) gentoo-dev 2005-04-20 09:47:21 UTC
Attaching a patch which when applied stops the behaviour. I am not 100% sure that the application is functioning normally, but my tests showed it was sane. YMMV
Comment 9 Benjamin Smee (strerror) (RETIRED) gentoo-dev 2005-04-20 09:47:53 UTC
Created attachment 56769 [details, diff]
patch to fix mxgrey=1 crash
Comment 10 Benjamin Smee (strerror) (RETIRED) gentoo-dev 2005-04-21 03:19:25 UTC
The author has confirmed my patch is ok, its been incorporated and a new version has been released. This should be closed and this version masked imo.
Comment 11 Tom Martin (RETIRED) gentoo-dev 2005-04-21 10:07:05 UTC
The new version is in CVS.

Ben: I'm not going to mask the old versions because if people notice a problem they will probably go with the upgrade (or be following this bug already ;) ).

Thanks everyone, resolving.

--Tom
Comment 12 Alexander Stoll 2005-05-20 05:24:19 UTC
This obviously fixed all issues and runs stable for several weeks on different
systems. If no one complains, this really should be marked stable and maybe the
versions with known issues should be dropped...