Blame SOURCES/mailman-2.1.29-options_content_njection.patch

bb5438
=== modified file 'Mailman/Cgi/private.py'
bb5438
--- Mailman/Cgi/private.py	2019-03-06 17:48:32 +0000
bb5438
+++ Mailman/Cgi/private.py	2020-05-07 13:53:40 +0000
bb5438
@@ -162,13 +162,9 @@
bb5438
             if mlist.isMember(username):
bb5438
                 mlist.MailUserPassword(username)
bb5438
             elif username:
bb5438
-                # Not a member
bb5438
-                if mlist.private_roster == 0:
bb5438
-                    # Public rosters
bb5438
-                    safeuser = Utils.websafe(username)
bb5438
-                    message = Bold(FontSize('+1',
bb5438
-                                  _('No such member: %(safeuser)s.'))).Format()
bb5438
-                else:
bb5438
+                # Not a member. Don't report address in any case. It leads to
bb5438
+                # Content injection. Just log if roster is not public.
bb5438
+                if mlist.private_roster != 0:
bb5438
                     syslog('mischief',
bb5438
                        'Reminder attempt of non-member w/ private rosters: %s',
bb5438
                        username)
bb5438
bb5438