help needed to fix contrib/ee crash/exit when receiving SIGWINCH
Alexander Best
alexbestms at math.uni-muenster.de
Fri Oct 23 17:02:35 UTC 2009
Eygene Ryabinkin schrieb am 2009-10-23:
> Gentlemen, good day.
> Fri, Oct 23, 2009 at 02:02:38PM +0200, Dag-Erling Sm??rgrav wrote:
> > src/contrib/ee/ee.c in 8:
> > in = wgetch(text_win);
> > if (in == -1)
> > exit(0); /* without this exit ee will go
> > into an
> > infinite loop if the network
> > session detaches */
> > >From the wgetch() man page:
> > Programmers concerned about portability should be prepared
> > for
> > either of two cases: (a) signal receipt does not interrupt
> > getch; (b) signal receipt interrupts getch and causes it to
> > return ERR with errno set to EINTR. Under the ncurses
> > implementation, handled signals never inter??? rupt getch.
> Hmm, we can transform this code to the following one:
> -----
> errno = 0;
> do {
> in = wgetch(text_win);
> } while (errno == EINTR);
> if (in == -1)
> exit(0);
> -----
> This won't help with FreeBSD's ncurses, but may be other variants
> will feel much better with such a event loop variant.
> > The real issue, though, is that when a SIGWINCH is caught, wgetch()
> > correctly returns KEY_RESIZE, but the next call to wgetch() returns
> > -1.
> > The next call after that is fine. I suspect the error lies
> > somewhere
> > inside kgetch() in contrib/ncurses/ncurses/base/lib_getch.c.
> The problem should be healed with the attached patch. And you're
> partly right: this is kgetch() that is returning ERR for the second
> wgetch(), but kgetch() shouldn't be blamed for this -- _nc_wgetch()
> should. At least in my opinion ;)
> Any views on this?
oh...and btw. the PR (http://www.freebsd.org/cgi/query-pr.cgi?pr=bin/136223)
hasn't been assigned to anyone yet. could the person who will be committing
the patch please take care of that (maybe setting the PR to
analysed->patched->closed? ;) would be great if this fix could make it into
8.0-RELEASE in time because it's quite nasty to lose all your editor data due
to a SIGWINCH.
cheers.
alex
More information about the freebsd-hackers
mailing list