Re: mountd: no need for critical sections

2023-03-02 Thread Alexander Bluhm
On Wed, Mar 01, 2023 at 04:31:07PM -0700, Todd C. Miller wrote: > The SIGHUP handler only sets a flag these days, there is no longer > any need to block it while using the exports list. > > OK? OK bluhm@ In the previous version of the diff you also eliminated the useless newline in new_exportlis

Re: mountd: no need for critical sections

2023-03-02 Thread Todd C . Miller
On Thu, 02 Mar 2023 07:25:17 +, Klemens Nanni wrote: > The TERM handler also just sets a flag today, but etc/rc.d/mountd still > has `rc_stop=NO' since 2013 > > Do not allow stopping/restarting mountd using the rc.d(8) framework; > if there is need to send a SIGTERM to mountd(8), it sh

Re: mountd: no need for critical sections

2023-03-01 Thread Klemens Nanni
01.03.2023 23:31, Todd C. Miller пишет: > The SIGHUP handler only sets a flag these days, there is no longer > any need to block it while using the exports list. Makes sense. The TERM handler also just sets a flag today, but etc/rc.d/mountd still has `rc_stop=NO' since 2013 Do not allow stop

mountd: no need for critical sections

2023-03-01 Thread Todd C . Miller
The SIGHUP handler only sets a flag these days, there is no longer any need to block it while using the exports list. OK? - todd Index: sbin/mountd/mountd.c === RCS file: /cvs/src/sbin/mountd/mountd.c,v retrieving revision 1.90 dif