jimzah/freebsd-src
1
0
Fork 0
mirror of https://github.com/freebsd/freebsd-src.git synced 2024-12-03 08:22:44 +00:00
Code Issues Actions 2 Packages Projects Releases Wiki Activity

Any call of tty_close() with a tty refcount of <= 1 is wrong and we will

Browse Source 
free the tty in this case. This is a workaround until the underlaying
devfs/tty problems are fixed.

MFC after:	1 day
This commit is contained in:
Martin Blapp 2006-09-30 08:11:51 +00:00
parent 1517bdc897
commit 35dcc318f4
Notes: svn2git 2020-12-20 02:59:44 +00:00 
svn path=/head/; revision=162833
1 changed files with 9 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
sys/kern/tty.c
View File

@ -333,7 +333,15 @@ tty_close(struct tty *tp)
tp->t_state = 0;
knlist_clear(&tp->t_rsel.si_note, 0);
knlist_clear(&tp->t_wsel.si_note, 0);
ttyrel(tp);
/*
* Any close with tp->t_refcnt == 1 is wrong and is
* an indication of a locking bug somewhere and that
* our open call has not been finished properly.
* Instead of putting an assert here we skip decrementing
* the refcount to work around any problems.
*/
if (tp->t_refcnt > 1)
ttyrel(tp);
splx(s);
return (0);
}
@ -3068,11 +3076,6 @@ ttyopen(struct cdev *dev, int flag, int mode, struct thread *td)
tp = dev->si_tty;
/* XXX It can happen that devfs_open calls us with tp->t_refcnt == 0 */
if (tp == NULL || tp->t_refcnt == 0) {
return (ENXIO);
}
s = spltty();
/*
* We jump to this label after all non-interrupted sleeps to pick