tty_audit: make icanon a bool
authorJiri Slaby <jirislaby@kernel.org>
Wed, 21 Jun 2023 10:16:09 +0000 (12:16 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 21 Jun 2023 15:57:29 +0000 (17:57 +0200)
Use bool for tty_audit_buf::icanon in favor of ugly bitfields. And get
rid of "!!" as that is completely unnecessary.

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

index c6c27ac..6cf8b63 100644 (file)
@@ -15,7 +15,7 @@
 struct tty_audit_buf {
        struct mutex mutex;     /* Protects all data below */
        dev_t dev;              /* The TTY which the data is from */
-       unsigned icanon:1;
+       bool icanon;
        size_t valid;
        unsigned char *data;    /* Allocated size N_TTY_BUF_SIZE */
 };
@@ -202,8 +202,8 @@ static struct tty_audit_buf *tty_audit_buf_get(void)
 void tty_audit_add_data(struct tty_struct *tty, const void *data, size_t size)
 {
        struct tty_audit_buf *buf;
-       unsigned int icanon = !!L_ICANON(tty);
        unsigned int audit_tty;
+       bool icanon = L_ICANON(tty);
        dev_t dev;
 
        audit_tty = READ_ONCE(current->signal->audit_tty);