From owner-freebsd-current Sat Sep 16 10:51:18 1995 Return-Path: owner-current Received: (from root@localhost) by freefall.freebsd.org (8.6.12/8.6.6) id KAA15170 for current-outgoing; Sat, 16 Sep 1995 10:51:18 -0700 Received: from precipice.shockwave.com (precipice.shockwave.com [171.69.108.33]) by freefall.freebsd.org (8.6.12/8.6.6) with ESMTP id KAA15149 for ; Sat, 16 Sep 1995 10:51:15 -0700 Received: from localhost (localhost [127.0.0.1]) by precipice.shockwave.com (8.6.12/8.6.12) with SMTP id KAA02205; Sat, 16 Sep 1995 10:50:10 -0700 Message-Id: <199509161750.KAA02205@precipice.shockwave.com> To: michael butler , current@freebsd.org Subject: Re: rmail and brain-dead mail systems .. patch enclosed In-reply-to: Your message of "Sat, 16 Sep 1995 10:39:05 PDT." <199509161739.KAA00498@precipice.shockwave.com> Date: Sat, 16 Sep 1995 10:50:09 -0700 From: Paul Traina Sender: owner-current@freebsd.org Precedence: bulk In case I'm not clear, I think that this patch will cause more harm than good, as it changes the expected syntax. I doubt very much that MUAs are going to expect a null from address. Paul From: Paul Traina Subject: Re: rmail and brain-dead mail systems .. patch enclosed This seems backwards. Why would you want a null address in there, rather than just fix the one or two clients that don't do the right thing if no address is present (which has been legal for years)? Paul From: michael butler Subject: rmail and brain-dead mail systems .. patch enclosed Will someone please add this patch to bin/rmail/rmail.c. Some of us have to deal with mail from brain-dead gateways (notably MS-mail >>) that produce "From " lines without an address .. e.g. .. From Sun Sep 17 01:10:23 1995 Received: from ... .. this throws rmail into a spin :-( The patch recommended by Eric Allman is .. *** rmail.c~ Mon Mar 20 01:06:12 1995 --- rmail.c Tue May 16 03:26:57 1995 *************** *** 211,216 **** --- 211,218 ---- /* Save off from user's address; the last one wins. */ for (p = addrp; *p && !isspace(*p); ++p); *p = '\0'; + if (*addrp == '\0') + addrp = "<>"; if (from_user != NULL) free(from_user); if ((from_user = strdup(addrp)) == NULL)