tty: n_tty: use 'num' for writes' counts
authorJiri Slaby (SUSE) <jirislaby@kernel.org>
Sun, 27 Aug 2023 07:41:36 +0000 (09:41 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 27 Aug 2023 09:46:51 +0000 (11:46 +0200)
We have a separate misnomer 'c' to hold the retuned value from
tty->ops->write(). Instead, use 'num' already defined on another place
(and already properly typed).

Signed-off-by: "Jiri Slaby (SUSE)" <jirislaby@kernel.org>
Link: https://lore.kernel.org/r/20230827074147.2287-4-jirislaby@kernel.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/tty/n_tty.c

index f6fa4dbdf78f7e85d65b3520dce525bb65977722..7f9fee4cf7cff93c6f136473276894fb45d61c6d 100644 (file)
@@ -2335,8 +2335,7 @@ static ssize_t n_tty_write(struct tty_struct *tty, struct file *file,
 {
        const u8 *b = buf;
        DEFINE_WAIT_FUNC(wait, woken_wake_function);
-       int c;
-       ssize_t retval = 0;
+       ssize_t num, retval = 0;
 
        /* Job control check -- must be done at start (POSIX.1 7.1.1.4). */
        if (L_TOSTOP(tty) && file->f_op->write_iter != redirected_tty_write) {
@@ -2362,7 +2361,7 @@ static ssize_t n_tty_write(struct tty_struct *tty, struct file *file,
                }
                if (O_OPOST(tty)) {
                        while (nr > 0) {
-                               ssize_t num = process_output_block(tty, b, nr);
+                               num = process_output_block(tty, b, nr);
                                if (num < 0) {
                                        if (num == -EAGAIN)
                                                break;
@@ -2384,16 +2383,16 @@ static ssize_t n_tty_write(struct tty_struct *tty, struct file *file,
 
                        while (nr > 0) {
                                mutex_lock(&ldata->output_lock);
-                               c = tty->ops->write(tty, b, nr);
+                               num = tty->ops->write(tty, b, nr);
                                mutex_unlock(&ldata->output_lock);
-                               if (c < 0) {
-                                       retval = c;
+                               if (num < 0) {
+                                       retval = num;
                                        goto break_out;
                                }
-                               if (!c)
+                               if (!num)
                                        break;
-                               b += c;
-                               nr -= c;
+                               b += num;
+                               nr -= num;
                        }
                }
                if (!nr)