Patch to fix a base64 decode error in Scrubber

Bug #558051 reported by scoggins
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
GNU Mailman
Fix Released
Undecided
Unassigned

Bug Description

The base64 decode of attachments isn't always checked
to see if an exception was raised. Call were placed
around one of the get_payload() calls, so i removed
that one and put a try/except in the _bdecode()
function. I also removed a redunent if not s: return s
in there too.

This is a fix for bug
http://sourceforge.net/tracker/index.php?func=detail&aid=691039&group_id=103&atid=100103

Revision history for this message
scoggins (scoggins-users) wrote :

The file base64.patch was added: Fix for bug #691039

Revision history for this message
twouters (twouters) wrote :

Logged In: YES
user_id=34209

This doesn't really belong in Mailman, it's more of an email
(the module) problem. I've sent in a bugreport to the
appropriate place, leaving this patch and the bug open until
it's fixed. (If it is; if it isn't, we need to wrap
get_payload in the appropriate try/except, which might be a
good idea anyway.)

Revision history for this message
bwarsaw (bwarsaw) wrote :

Logged In: YES
user_id=12800

I've fixed this in the email.Message class, so I don't think
we need to add a try/except to Scrubber.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.