From: <fre...@va...> - 2005-07-15 09:26:19
|
> >I'm not a programmer so it wouldn't be that easy for me to spot it. >So i should just remove all of the code about, except this line: > > if (rtype =3D=3D T_TXT && queries =3D=3D 1) > >I'll try it asap. > I'm sorry to say that this didn't help all the way. Theres still something going on making sid-filter start to loop... But this time it doesn't affect resolving. It's something else since the resolver didn't consume more mem/cpu. I'll grab a trace, to see if that can be more helpful. /P |
From: <fre...@va...> - 2005-07-15 09:55:15
|
>> >>I'm not a programmer so it wouldn't be that easy for me to spot it. >>So i should just remove all of the code about, except this line: >> >> if (rtype =3D=3D T_TXT && queries =3D=3D 1) >> >>I'll try it asap. >> > >I'm sorry to say that this didn't help all the way. >Theres still something going on making sid-filter start to loop... > >But this time it doesn't affect resolving. It's something else since >the resolver didn't consume more mem/cpu. > >I'll grab a trace, to see if that can be more helpful. > No, wrong by me, it appears as last time. The resolver still get busy... (maybe the system was more busy swapping than acually run processes,=20 then i noticed it the first time today) So it is the same problem, even though i removed the 4 lines from your = patch so the if-statement always looks like this w/o NH_REDIRECTFIX compiled = in. if (rtype =3D=3D T_TXT && queries =3D=3D 1) |
From: Dick St.P. <stpeters@NetHeaven.com> - 2005-07-15 14:59:57
|
fre...@va... writes: > No, wrong by me, it appears as last time. The resolver still get busy... > (maybe the system was more busy swapping than acually run processes, > then i noticed it the first time today) I've posted a new patch with a bugfix for NH_REDIRECT. When editing changes from my working copy into my master copy, what was supposed to be a copy+paste+edit became just a copy+paste. Instead of downloading the new patch and applying it to a clean distribution, you can just change line 118 oflibmarid/sm-marid-frame.c from smf->smf_exp_tried = 1; to smf->smf_redirect_tried = 1; Having the redirect code updating the wrong "been here" flag could easily cause the kind of looping you're seeing, so I'm hopeful that this will fix the problem. The new patch (and updated README) have two minor new things in them: NH_TYPOFIX This corrects a typo in libar/ar.c, removing a misplaced "*". The typo is in error condition code that will probably never be run. NH_NONE This causes sid-filter to log {spf,senderid}=none instead of {spf,senderid}=neutral when no spf record is found. Patch and README at ftp://ftp.netheaven.com/pub/sid-milter/nh-sid-milter-0.2.8.patch ftp://ftp.netheaven.com/pub/sid-milter/README I did not change the patch to restore the "queries == 1" test. I'm not yet sure which way is right, and not having that test seems not to be causing me any problems. -- Dick St.Peters, stpeters@NetHeaven.com |