Commit 844cc5f9 authored by Jiri Slaby's avatar Jiri Slaby Committed by Greg Kroah-Hartman
Browse files

n_hdlc: remove checking of n_hdlc 



We got rid of backup_tty recently. Also, the tty layer ensures not to
call other ldisc hooks after ldisc close. That means, all those tests
are superfluous now so remove them.

Note that we remove the magic check in write after schedule too. The tty
cannot change during schedule.

Signed-off-by: default avatarJiri Slaby <jslaby@suse.cz>
Link: https://lore.kernel.org/r/20200219084118.26491-14-jslaby@suse.cz


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent df6de639

drivers/tty/n_hdlc.c

+3 −29
Original line number Diff line number Diff line
@@ -227,9 +227,6 @@ static void n_hdlc_tty_close(struct tty_struct *tty) 
{

	struct n_hdlc *n_hdlc = tty->disc_data;



	if (!n_hdlc)

		return;



	if (n_hdlc->magic != HDLC_MAGIC) {

		printk(KERN_WARNING "n_hdlc: trying to close unopened tty!\n");

		return;
@@ -383,11 +380,7 @@ static void n_hdlc_tty_wakeup(struct tty_struct *tty) 
{

	struct n_hdlc *n_hdlc = tty->disc_data;



	if (!n_hdlc)

		return;



	n_hdlc_send_frames (n_hdlc, tty);

		

}	/* end of n_hdlc_tty_wakeup() */



/**
@@ -409,10 +402,6 @@ static void n_hdlc_tty_receive(struct tty_struct *tty, const __u8 *data, 
	pr_debug("%s(%d)%s() called count=%d\n",

			__FILE__, __LINE__, __func__, count);



	/* This can happen if stuff comes in on the backup tty */

	if (!n_hdlc)

		return;

		

	/* verify line is using HDLC discipline */

	if (n_hdlc->magic != HDLC_MAGIC) {

		printk("%s(%d) line not using HDLC discipline\n",
@@ -473,10 +462,6 @@ static ssize_t n_hdlc_tty_read(struct tty_struct *tty, struct file *file, 
	struct n_hdlc_buf *rbuf;

	DECLARE_WAITQUEUE(wait, current);



	/* Validate the pointers */

	if (!n_hdlc)

		return -EIO;



	/* verify user access to buffer */

	if (!access_ok(buf, nr)) {

		printk(KERN_WARNING "%s(%d) n_hdlc_tty_read() can't verify user "
@@ -558,10 +543,6 @@ static ssize_t n_hdlc_tty_write(struct tty_struct *tty, struct file *file, 
	pr_debug("%s(%d)%s() called count=%zd\n", __FILE__, __LINE__, __func__,

			count);



	/* Verify pointers */

	if (!n_hdlc)

		return -EIO;



	if (n_hdlc->magic != HDLC_MAGIC)

		return -EIO;
@@ -587,13 +568,6 @@ static ssize_t n_hdlc_tty_write(struct tty_struct *tty, struct file *file, 
		}

		schedule();



		n_hdlc = tty->disc_data;

		if (!n_hdlc || n_hdlc->magic != HDLC_MAGIC) {

			printk("n_hdlc_tty_write: %p invalid after wait!\n", n_hdlc);

			error = -EIO;

			break;

		}

			

		if (signal_pending(current)) {

			error = -EINTR;

			break;
@@ -638,7 +612,7 @@ static int n_hdlc_tty_ioctl(struct tty_struct *tty, struct file *file, 
	pr_debug("%s(%d)%s() called %d\n", __FILE__, __LINE__, __func__, cmd);



	/* Verify the status of the device */

	if (!n_hdlc || n_hdlc->magic != HDLC_MAGIC)

	if (n_hdlc->magic != HDLC_MAGIC)

		return -EBADF;



	switch (cmd) {
@@ -701,7 +675,7 @@ static __poll_t n_hdlc_tty_poll(struct tty_struct *tty, struct file *filp, 
	struct n_hdlc *n_hdlc = tty->disc_data;

	__poll_t mask = 0;



	if (!n_hdlc || n_hdlc->magic != HDLC_MAGIC)

	if (n_hdlc->magic != HDLC_MAGIC)

		return 0;



	/*

CRA Git | Maintained and supported by SUSTech CRA and CCSE