Current-Users archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: audio/moc dumps core in libcurses



On 30/01/2017 04:22, coypu%SDF.ORG@localhost wrote:
> I've applied a patch to audio/moc. It works on my end now.
> It was attempting set_escdelay (25); very early, before setting
> up a window.
> 
> On nbcurses it was:_cursesi_screen->ESCDELAY = escdelay;
> which was a null deref.
> 
> for ncurses, it was returning an error, but it was ignoring it.

We have a lot of similar functions which operate without checking the
terminal has been setup or not.
It *IS* a programming error, so it's probably best to use an assert
rather than simply returning ERR.

Anyone have any stronger opinions on this before I add asserts liberally
to our curses?

Roy


Home | Main Index | Thread Index | Old Index