Google Groups no longer supports new Usenet posts or subscriptions. Historical content remains viewable.
Dismiss

Non-continuation line starts with space, sendmail -bi change?

62 views
Skip to first unread message

John Dzubera

unread,
Sep 29, 2021, 5:39:29 PM9/29/21
to
In 8.16.1 and previous, long aliases in /etc/mail/aliases formatted like this:

blah:user1,user2,user3,
<tab>user4,user5,user6,
<tab>user7,user8,user9

were fine. sendmail -bi didn't complain.

8.17.1 no longer likes these lines, flags the second line with
"Non-continuation line starts with space" and doesn't seem to build
the aliases properly. Put them all on one long line and it's fine.

I'm sure I missed something. Anyone kind enough to clue me in?

Cheers.

Claus Aßmann

unread,
Sep 30, 2021, 3:47:28 AM9/30/21
to
If you don't use EAI, please try this patch
(maybe report whether it works to the bug address?)
For EAI it will be a bit more complicated to fix.

--- alias.c- Thu Sep 30 07:36:26 2021
+++ alias.c Thu Sep 30 07:37:08 2021
@@ -817,10 +817,13 @@
(void) sm_io_ungetc(af, SM_TIME_DEFAULT, c);
if (c != ' ' && c != '\t')
break;
+#if _FFR_8BITENVADDR
+ syserr("554 5.3.5 Continuation line currently not supported for USE_EAI nor _FFR_EIGHT_BIT_ADDR_OK");
+#endif

/* read continuation line */
if (sm_io_fgets(af, SM_TIME_DEFAULT, p,
- sizeof(line) - (p-line)) < 0)
+ sizeof(lbuf) - (p-line)) < 0)
break;
LineNumber++;



--
Note: please read the netiquette before posting. I will almost never
reply to top-postings which include a full copy of the previous
article(s) at the end because it's annoying, shows that the poster
is too lazy to trim his article, and it's wasting the time of all readers.

John Dzubera

unread,
Oct 1, 2021, 6:18:32 PM10/1/21
to
On Thursday, September 30, 2021 at 1:47:28 AM UTC-6, Claus Aßmann wrote:
> If you don't use EAI, please try this patch
> (maybe report whether it works to the bug address?)
> For EAI it will be a bit more complicated to fix.
>
> --- alias.c- Thu Sep 30 07:36:26 2021
> +++ alias.c Thu Sep 30 07:37:08 2021
> @@ -817,10 +817,13 @@
> (void) sm_io_ungetc(af, SM_TIME_DEFAULT, c);
> if (c != ' ' && c != '\t')
> break;
> +#if _FFR_8BITENVADDR
> + syserr("554 5.3.5 Continuation line currently not supported for USE_EAI nor _FFR_EIGHT_BIT_ADDR_OK");
> +#endif
>
> /* read continuation line */
> if (sm_io_fgets(af, SM_TIME_DEFAULT, p,
> - sizeof(line) - (p-line)) < 0)
> + sizeof(lbuf) - (p-line)) < 0)
> break;
> LineNumber++;

Thank you. This seemed to work. I sent a note to the bug address as well.

Note: I patched the file by hand. When I tried to use:

patch < claus.patch

I got:

patch: **** malformed patch at line 4: (void) sm_io_ungetc(af, SM_TIME_DEFAULT, c);

even with the -u option, so I must be missing something, but I'm grateful for the fix.

Cheers.

Claus Aßmann

unread,
Oct 2, 2021, 1:00:24 AM10/2/21
to
John Dzubera wrote:

> Thank you. This seemed to work. I sent a note to the bug address as well.

Thanks.

> patch: **** malformed patch at line 4: (void) sm_io_ungetc(af, SM_TIME_DEFAULT, c);

Seems the patch got a bit mangled "somewhere", sorry.

--
Note: please read the netiquette before posting. I will almost never
reply to top-postings which include a full copy of the previous
article(s) at the end because it's annoying, shows that the poster
is too lazy to trim his article, and it's wasting the time of all readers.
--
Please don't Cc: me, post your replies here.
0 new messages