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

[John Dzubera‏]

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‏]

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‏]

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‏]

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.