Hi hack...@freebsd.org

There is a missing double quote " in 
8.3 & 9.0 & 9.1RC2 src/etc/sendmail/freebsd.mc
8.2-RELEASE & earlier are OK.

Here's a diff -c to .mc
The diff is not to fix it, but to help generate a freebsd.cf to
understand the difference.  The patch for a fix would befar more trivial :-)

------
*** 9.1-RC3/src/etc/sendmail/freebsd.mc Mon Oct 29 21:16:44 2012
--- 9.1-RC3/src+debug/etc/sendmail/freebsdjhs.mc        Mon Dec  3 18:44:33 2012
***************
*** 66,75 ****
  dnl For that, visit
  dnl http://www.google.com/Top/Computers/Internet/E-mail/Spam/Blacklists/
  
! dnl Uncomment to activate your chosen DNS based blacklist
! dnl FEATURE(dnsbl, `dnsbl.example.com')
! dnl Alternatively, you can provide your own server and rejection message:
! dnl FEATURE(dnsbl, `dnsbl.example.com', ``"550 Mail from " $&{client_addr} " 
rejected'')
  
  dnl Dialup users should uncomment and define this appropriately
  dnl define(`SMART_HOST', `your.isp.mail.server')
--- 66,77 ----
  dnl For that, visit
  dnl http://www.google.com/Top/Computers/Internet/E-mail/Spam/Blacklists/
  
! # Uncomment to activate your chosen DNS based blacklist
! FEATURE(dnsbl, `jhs0.dnsbl.example.com')
! # Alternatively, you can provide your own server and rejection message:
! FEATURE(dnsbl, `jhs3.dnsbl.example.com', ``"550 Mail from " $&{client_addr} " 
rejected'')
! # original line above has 3 x " , line below has 4
! FEATURE(dnsbl, `jhs4.dnsbl.example.com', ``"550 Mail from " $&{client_addr} " 
rejected"'')
  
  dnl Dialup users should uncomment and define this appropriately
  dnl define(`SMART_HOST', `your.isp.mail.server')
------

The .cf output is
------
# DNS based IP address spam list jhs0.dnsbl.example.com
R$*                     $: $&{client_addr}
R$-.$-.$-.$-            $: <?> $(dnsbl $4.$3.$2.$1.jhs0.dnsbl.example.com. $: 
OK $)
R<?>OK                  $: OKSOFAR
R<?>$+<TMP>             $: TMPOK
R<?>$+          $#error $@ 5.7.1 $: "550 Rejected: " $&{client_addr} " listed 
at jhs0.dnsbl.example.com"

# DNS based IP address spam list jhs3.dnsbl.example.com
R$*                     $: $&{client_addr}
R$-.$-.$-.$-            $: <?> $(dnsbl $4.$3.$2.$1.jhs3.dnsbl.example.com. $: 
OK $)
R<?>OK                  $: OKSOFAR
R<?>$+<TMP>             $: TMPOK
R<?>$+          $#error $@ 5.7.1 $: "550 Mail from " $&{client_addr} " rejected

# DNS based IP address spam list jhs4.dnsbl.example.com
R$*                     $: $&{client_addr}
R$-.$-.$-.$-            $: <?> $(dnsbl $4.$3.$2.$1.jhs4.dnsbl.example.com. $: 
OK $)
R<?>OK                  $: OKSOFAR
R<?>$+<TMP>             $: TMPOK
R<?>$+          $#error $@ 5.7.1 $: "550 Mail from " $&{client_addr} " rejected"
------
Above, jhs3 seems bad & jhs4 seems good.

A copy of this diff + follow up is in
        
http://berklix.com/~jhs/src/bsd/fixes/FreeBSD/src/gen/etc/sendmail/freebsd.mc.REL=ALL.diff

The error appeared in 8.3 & 9.0 & 9.1RC3, it was OK earlier :

4.11-RELEASE/src/etc/sendmail/freebsd.mc:dnl FEATURE(dnsbl, 
`blackholes.mail-abuse.org', `"550 Mail from " $&{client_addr} " rejected, see 
http://mail-abuse.org/cgi-bin/lookup?"; $&{client_addr}')

6.4-RELEASE/src/etc/sendmail/freebsd.mc:dnl FEATURE(dnsbl, 
`blackholes.mail-abuse.org', ``"550 Mail from " $&{client_addr} " rejected, see 
http://mail-abuse.org/cgi-bin/lookup?"; $&{client_addr}'')

7.3-RELEASE/src/etc/sendmail/freebsd.mc:dnl FEATURE(dnsbl, 
`blackholes.mail-abuse.org', ``"550 Mail from " $&{client_addr} " rejected, see 
http://mail-abuse.org/cgi-bin/lookup?"; $&{client_addr}'')

7.4-RELEASE/src/etc/sendmail/freebsd.mc:dnl FEATURE(dnsbl, 
`blackholes.mail-abuse.org', ``"550 Mail from " $&{client_addr} " rejected, see 
http://mail-abuse.org/cgi-bin/lookup?"; $&{client_addr}'')

8.2-RELEASE/src/etc/sendmail/freebsd.mc:dnl FEATURE(dnsbl, 
`blackholes.mail-abuse.org', ``"550 Mail from " $&{client_addr} " rejected, see 
http://mail-abuse.org/cgi-bin/lookup?"; $&{client_addr}'')

8.3-RELEASE/src/etc/sendmail/freebsd.mc:dnl FEATURE(dnsbl, `dnsbl.example.com', 
``"550 Mail from " $&{client_addr} " rejected'')

9.0-RELEASE/src/etc/sendmail/freebsd.mc:dnl FEATURE(dnsbl, `dnsbl.example.com', 
``"550 Mail from " $&{client_addr} " rejected'')

9.1-RC3//src/etc/sendmail/freebsd.mc:dnl FEATURE(dnsbl, `dnsbl.example.com', 
``"550 Mail from " $&{client_addr} " rejected'')


Why others didnt notice earlier:
  It's just a comment, I assume not checked by anything automatic at
  freebsd.org.

How I caught it:
  I have my own src/etc/sendmail/ Makefile diffs & fdef'd common.cpp
  that generates various .mc files for each of my hosts, & many
  versions, which filters the .cpp to create .mcfiles, inc. a check
  file that should be an exact match with that version's freebsd.mc
  ... & cpp complained: unmatched ".

I will send-pr unless I hear otherwise.
  But as 9.1-RELEASE is imminent, & it would be a trivial anodyne
  fix to a comment for re@ , so they want to fix, so cc'd to re@.

Cheers,
Julian
-- 
Julian Stacey, BSD Unix Linux C Sys Eng Consultant, Munich http://berklix.com
 Reply below not above, like a play script.  Indent old text with "> ".
 Send plain text. Not: HTML, multipart/alternative, base64, quoted-printable.
_______________________________________________
freebsd-hackers@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-hackers
To unsubscribe, send any mail to "freebsd-hackers-unsubscr...@freebsd.org"

Reply via email to