You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@spamassassin.apache.org by bu...@bugzilla.spamassassin.org on 2011/09/01 00:34:15 UTC

[Bug 6655] [review] sa-update might DOS mirrors if TMPDIR unwritable

https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6655

Kevin A. McGrail <km...@pccc.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |kmcgrail@pccc.com
            Summary|sa-update might DOS mirrors |[review] sa-update might
                   |if TMPDIR unwritable        |DOS mirrors if TMPDIR
                   |                            |unwritable
  Status Whiteboard|                            |Needs 2 Votes for 3.3
                   |                            |Branch

--- Comment #3 from Kevin A. McGrail <km...@pccc.com> 2011-08-31 22:34:15 UTC ---
(In reply to comment #2)
> Created attachment 4958 [details]
> replacement patch.
> 
> patches ../lib/Mail/SpamAssassin/Util.pm to check for existance of, and read
> perms on TMPDIR, dies if not.
> 
> patches ../sa-update.raw. TRY to catch errors on fsys full on TMPDIR.
> 
> (are tracking down why several sites running sa-update on 3.3.1 are pulling
> sa-updates 9000 per day.  3000 per day per mirror.)

KAM: Tested and logic seems good +1 for me for 3.3. 

Committing to trunk:

svn commit -m 'Michaels patch for more error trapping on sa-update.  See bug
6655'
Sending        lib/Mail/SpamAssassin/Util.pm
Sending        sa-update.raw
Transmitting file data ..
Committed revision 1163848.

-- 
Configure bugmail: https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.