diff options
author | Laura Abbott <labbott@fedoraproject.org> | 2015-05-14 11:42:17 -0700 |
---|---|---|
committer | Luis Henriques <luis.henriques@canonical.com> | 2015-06-11 11:24:58 +0100 |
commit | 13ede3eea086fb46dbb57e7d3fe32fff8345774f (patch) | |
tree | 2c57d55336a2adca588d6b82a456ed84a74de276 | |
parent | aa246412f102933084847d7a115dba5dc605537f (diff) | |
download | linux-stable-13ede3eea086fb46dbb57e7d3fe32fff8345774f.tar.gz linux-stable-13ede3eea086fb46dbb57e7d3fe32fff8345774f.tar.bz2 linux-stable-13ede3eea086fb46dbb57e7d3fe32fff8345774f.zip |
n_tty: Fix auditing support for cannonical mode
commit 72586c6061ab8c23ffd9f301ed19782a44ff5f04 upstream.
Commit 32f13521ca68bc624ff6effc77f308a52b038bf0
("n_tty: Line copy to user buffer in canonical mode")
changed cannonical mode copying to use copy_to_user
but missed adding the call to the audit framework.
Add in the appropriate functions to get audit support.
Fixes: 32f13521ca68 ("n_tty: Line copy to user buffer in canonical mode")
Reported-by: Miloslav Trmač <mitr@redhat.com>
Signed-off-by: Laura Abbott <labbott@fedoraproject.org>
Reviewed-by: Peter Hurley <peter@hurleysoftware.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
[ luis: backported to 3.16: adjusted context ]
Signed-off-by: Luis Henriques <luis.henriques@canonical.com>
-rw-r--r-- | drivers/tty/n_tty.c | 17 |
1 files changed, 14 insertions, 3 deletions
diff --git a/drivers/tty/n_tty.c b/drivers/tty/n_tty.c index 35de4adc3f1d..39ec99fbb135 100644 --- a/drivers/tty/n_tty.c +++ b/drivers/tty/n_tty.c @@ -161,6 +161,17 @@ static inline int tty_put_user(struct tty_struct *tty, unsigned char x, return put_user(x, ptr); } +static inline int tty_copy_to_user(struct tty_struct *tty, + void __user *to, + const void *from, + unsigned long n) +{ + struct n_tty_data *ldata = tty->disc_data; + + tty_audit_add_data(tty, to, n, ldata->icanon); + return copy_to_user(to, from, n); +} + static int receive_room(struct tty_struct *tty) { struct n_tty_data *ldata = tty->disc_data; @@ -2081,12 +2092,12 @@ static int canon_copy_from_read_buf(struct tty_struct *tty, __func__, eol, found, n, c, size, more); if (n > size) { - ret = copy_to_user(*b, read_buf_addr(ldata, tail), size); + ret = tty_copy_to_user(tty, *b, read_buf_addr(ldata, tail), size); if (ret) return -EFAULT; - ret = copy_to_user(*b + size, ldata->read_buf, n - size); + ret = tty_copy_to_user(tty, *b + size, ldata->read_buf, n - size); } else - ret = copy_to_user(*b, read_buf_addr(ldata, tail), n); + ret = tty_copy_to_user(tty, *b, read_buf_addr(ldata, tail), n); if (ret) return -EFAULT; |