For some more context which you may want to add to the commit message: I noticed dash would always print “you have mail” three times, and this fixes that. On further investigation, this loop was taking the last path in MAILPATH (or the only string in MAIL) and continuing to chop off the last character, so for MAIL=/var/mail/june, it would stat: /var/mail/june /var/mail/jun /var/mail/ju /var/mail/j /var/mail/ /var/mail /var/mai /var/ma /var/m /var/ And then stop because MAXMBOXES is 10. The three “you have mail” messages were from /var/mail/june, /var/mail/ and /var/mail.
Re: [PATCH] Fix chkmail loop break condition
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- Subject: Re: [PATCH] Fix chkmail loop break condition
- From: june <june@causal.agency>
- Date: Sat, 2 Oct 2021 17:22:30 -0400
- In-reply-to: <20211002203732.6175-1-june@causal.agency>
- References: <20211002203732.6175-1-june@causal.agency>
- References:
- [PATCH] Fix chkmail loop break condition
- From: C. McEnroe
- [PATCH] Fix chkmail loop break condition
- Prev by Date: [PATCH] Fix chkmail loop break condition
- Next by Date: $ENV handling depends on defined(linux), why??
- Previous by thread: [PATCH] Fix chkmail loop break condition
- Next by thread: Re: [PATCH] Fix chkmail loop break condition
- Index(es):
 |