|
SA Bugzilla – Full Text Bug Listing |
Summary: | untaint_var() broken for undefined variables | ||
---|---|---|---|
Product: | Spamassassin | Reporter: | Kevin A. McGrail <kmcgrail> |
Component: | spamc/spamd | Assignee: | SpamAssassin Developer Mailing List <dev> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | kmcgrail |
Priority: | P2 | ||
Version: | 3.4.0 | ||
Target Milestone: | 3.4.1 | ||
Hardware: | PC | ||
OS: | Windows 7 | ||
Whiteboard: |
Description
Kevin A. McGrail
2014-02-20 17:10:19 UTC
svn commit -m 'disabling hash rules correctly in 20_ac_rules_test.cf until we have a plugin, bug 7013 to change BAYES_99 back and make BAYES_999 an overlap rule, bug 7015 for untaint var bug thanks to Tomasz Potega' Sending lib/Mail/SpamAssassin/Plugin/AutoLearnThreshold.pm Sending lib/Mail/SpamAssassin/Util.pm Sending rules/23_bayes.cf Sending rules/30_text_de.cf Sending rules/30_text_fr.cf Sending rules/30_text_nl.cf Sending rules/30_text_pl.cf Sending rules/30_text_pt_br.cf Sending rules/50_scores.cf Sending rules/60_shortcircuit.cf Sending rulesrc/sandbox/kmcgrail/20_ac_rules_test.cf Transmitting file data ........... Committed revision 1570286. The description and the fix do not match each other. (In reply to Tom Schulz from comment #2) > The description and the fix do not match each other. How so? The description describes a problem with --nouser-config not working and gives a patch to Util.pm. The fix is not about patching Util.pm (In reply to Tom Schulz from comment #4) > The description describes a problem with --nouser-config not working and > gives a patch to Util.pm. The fix is not about patching Util.pm According to the reporter, the fix is in Util.pm. Are you able to recreate a problem with -x using svn trunk? I am not having any problem. It was just that I read the description mentioning a proposed patch in util.pm and then read comment 1 which seemed to me to have nothing to do with that.Sorry if I am completely misunderstanding things. (In reply to Tom Schulz from comment #6) > I am not having any problem. It was just that I read the description > mentioning a proposed patch in util.pm and then read comment 1 which seemed > to me to have nothing to do with that.Sorry if I am completely > misunderstanding things. No worries. I truly appreciate the vetting. |