Results 1 to 5 of 5

Thread: Majordomo broken after perl upgrade (patch included)

  1. #1
    Join Date
    May 2013
    Posts
    2

    Majordomo broken after perl upgrade (patch included)

    Hi all,

    We've recently upgraded our server to Debian 7 (Wheezy) with DA 1.43. After the upgrade, majordomo stopped working for some of our mailing lists. This is because of the incompatibility of the new perl version (now Perl5.14)

    We were able to narrow the problem down to the use of restrict_post, which results in the mail server silently dropping the mail (BAD imho):
    2013-05-09 14:50:18 1UaQIc-0003lP-7P <= <> R=1UaQIc-0003lN-6L U=mail P=local S=1741 T="Mail failure - no recipient addresses" from <> for owner-testlist@xxxxxxx

    Reason lies in majordomo.pl, which cannot make the match between the sender and the subscribers:
    addr_match: enter
    addr_match: comparing Stijn de Bekker <stijn@xx.yy> against stijn@xx.yy
    addr_match: enter
    []
    /etc/virtual/majordomo/resend: sender check: ' Non-member submission from [Stijn de Bekker <stijn@xx.yy>] '

    Root cause was found in main'ParseAddrs, that incorrectly uses $_ [1]
    Attached is a patch that fixes this problem.

    Hope this helps you too,
    Stijn.

    [1] Bug was identified in 2010 by Todd Kover: http://mail-index.netbsd.org/pkgsrc-...msg039977.html
    Attached Files Attached Files

  2. #2
    Thanks for the report and supplied patch. Fixed in the installer for the next release of DA:
    http://www.directadmin.com/features.php?id=1447

    For existing systems, run this:
    Code:
    cd /etc/virtual/majordomo
    wget -O majordomo.patch http://files1.directadmin.com/services/all/majordomo/majordomo.patch
    patch -p0 < majordomo.patch
    John

  3. #3
    Join Date
    Jun 2003
    Location
    California
    Posts
    26,123
    Will the patch also work with older versions of Perl, or must we make sure we upgrade Perl at the same time we install this page?

    Thanks.

    Jeff
    +1 951 643-5345
    Third-Party DirectAdmin administration and support
    Dedicated Servers, Dedicated Reseller Accounts
    NoBaloney Internet Services div. Qnito Incorporated
    848 North Rainbow Blvd., Suite #3789
    Las Vegas, NV 89107-1103

  4. #4
    Join Date
    Jul 2008
    Location
    Maastricht
    Posts
    2,836
    After installing the patch, the owner of majordomo.pl changes from majordomo:daemon to root:daemon.

    I presume this has to be set back to majordomo:daemon again?
    If yes, maybe you might need to adjust the majordomo.sh to do this too?
    Greetings, Richard.

  5. #5
    Join Date
    Oct 2003
    Location
    Scottsdale, AZ
    Posts
    1,148
    I haven't applied the patch yet, but looking at the changes, there is no reason to believe it will not work on older versions of Perl. It just uses a named variable in place of the Perl's current variables (i.e. $_ and @_), I am guessing the newer version of Perl changed a behavior that majordomo took advantage of.

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •