All of lore.kernel.org
 help / color / mirror / Atom feed
From: lazytyped <lazytyped@gmail.com>
To: kernel-hardening@lists.openwall.com
Subject: Re: [kernel-hardening] Re: [PATCH v7 2/2] security: tty: make TIOCSTI ioctl require CAP_SYS_ADMIN
Date: Thu, 1 Jun 2017 09:12:07 +0200	[thread overview]
Message-ID: <6e713105-02db-12e1-68a9-f167fe11d0a9@gmail.com> (raw)
In-Reply-To: <CAGXu5j+pqD1082fYDS_dvDB2QNvt9wSz+C7vAhGpMXcJWxoDkw@mail.gmail.com>



On 6/1/17 4:35 AM, Kees Cook wrote:
> I still cannot wrap my head around why providing users with a
> protection is a bad thing. Yes, the other tty games are bad, but this
> fixes a specific and especially bad case that is easy to kill. It's
> got a Kconfig and a sysctl. It's not on by default. This protects the
> common case of privileged ttys that aren't attached to consoles, etc,
> so while the framebuffer thing is an issue, it's not always an issue,
> etc.
There are a couple of reasons for that:

First of all, a protection is extra cost, in terms of maintenance, 
knowledge (a new knob) and compatibility. That extra cost may sound 
minimal, but adds up pretty quickly. If the protection is "easily" 
bypassable (that is, today we use TIOCSTI, tomorrow we use something 
else in the same path), then that extra cost/complexity stays for no 
good reason. Feature creep is a real issue, in security, IMHO - it's not 
a 'number of features' game.

Second, stuff that is delivered off by default tends to rot. I don't 
work on Linux, but generally try really hard to not add something that 
is not ON by default at least for a small number of things. Stuff 
inevitably breaks, and it's extra cost.
To me, a protection that needs to be off by default, raises a red flag. 
I know Linux has a somewhat different philosophy (centered around the 
kernel config that each distribution pieces together and ships), so 
mileage probably varies there.

I don't have enough skills to comment about all the possible TTY attacks 
and quirks, but I think I understand where Alan comes from.

Good luck.


          -  Enrico

  reply	other threads:[~2017-06-01  7:12 UTC|newest]

Thread overview: 132+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-05-29 21:37 [PATCH v7 0/2] security: tty: make TIOCSTI ioctl require CAP_SYS_ADMIN Matt Brown
2017-05-29 21:37 ` [kernel-hardening] " Matt Brown
2017-05-29 21:37 ` Matt Brown
2017-05-29 21:37 ` [PATCH v7 1/2] security: tty: Add owner user namespace to tty_struct Matt Brown
2017-05-29 21:37   ` [kernel-hardening] " Matt Brown
2017-05-29 21:37   ` Matt Brown
2017-05-29 21:38 ` [PATCH v7 2/2] security: tty: make TIOCSTI ioctl require CAP_SYS_ADMIN Matt Brown
2017-05-29 21:38   ` [kernel-hardening] " Matt Brown
2017-05-29 21:38   ` Matt Brown
2017-05-29 22:26   ` Alan Cox
2017-05-29 22:26     ` [kernel-hardening] " Alan Cox
2017-05-29 22:26     ` Alan Cox
2017-05-29 23:51     ` [kernel-hardening] " Boris Lukashev
2017-05-29 23:51       ` Boris Lukashev
2017-05-30  0:27       ` Casey Schaufler
2017-05-30  0:27         ` Casey Schaufler
2017-05-30  2:00         ` Matt Brown
2017-05-30  2:00           ` Matt Brown
2017-05-30  2:46           ` Casey Schaufler
2017-05-30  2:46             ` Casey Schaufler
2017-05-30  3:18             ` Matt Brown
2017-05-30  3:18               ` Matt Brown
2017-05-30 12:24               ` Alan Cox
2017-05-30 12:24                 ` Alan Cox
2017-05-30 16:28                 ` Matt Brown
2017-05-30 16:28                   ` Matt Brown
2017-05-30 16:44                   ` Daniel Micay
2017-05-30 16:44                     ` Daniel Micay
2017-05-30 18:32                   ` Stephen Smalley
2017-05-30 18:32                     ` Stephen Smalley
2017-05-30 18:44                     ` Nick Kralevich
2017-05-30 18:44                       ` Nick Kralevich
2017-05-30 18:57                       ` Matt Brown
2017-05-30 18:57                         ` Matt Brown
2017-05-30 20:22                         ` Daniel Micay
2017-05-30 20:22                           ` Daniel Micay
2017-05-30 23:00                           ` Matt Brown
2017-05-30 23:00                             ` Matt Brown
2017-05-30 23:40                             ` Daniel Micay
2017-05-30 23:40                               ` Daniel Micay
2017-05-30 23:59                               ` Matt Brown
2017-05-30 23:59                                 ` Matt Brown
2017-05-30 22:51                   ` Alan Cox
2017-05-30 22:51                     ` Alan Cox
2017-05-30 23:19                     ` Matt Brown
2017-05-30 23:19                       ` Matt Brown
2017-05-30 23:56                       ` Alan Cox
2017-05-30 23:56                         ` Alan Cox
2017-06-01  2:35                         ` Kees Cook
2017-06-01  2:35                           ` Kees Cook
2017-06-01  2:35                           ` Kees Cook
2017-06-01  7:12                           ` lazytyped [this message]
2017-06-01 18:46                             ` Kees Cook
2017-06-01 22:56                               ` James Morris
2017-06-02 18:46                                 ` Kees Cook
2017-06-01 13:08                           ` Alan Cox
2017-06-01 13:08                             ` Alan Cox
2017-06-01 13:08                             ` Alan Cox
2017-06-01 17:18                             ` Serge E. Hallyn
2017-06-01 17:18                               ` Serge E. Hallyn
2017-06-01 17:18                               ` Serge E. Hallyn
2017-06-01 21:26                               ` Alan Cox
2017-06-01 21:26                                 ` Alan Cox
2017-06-01 21:26                                 ` Alan Cox
2017-06-01 18:58                             ` Kees Cook
2017-06-01 18:58                               ` Kees Cook
2017-06-01 18:58                               ` Kees Cook
2017-06-01 21:24                               ` Alan Cox
2017-06-01 21:24                                 ` Alan Cox
2017-06-01 21:24                                 ` Alan Cox
2017-06-02 14:46                                 ` Matt Brown
2017-06-02 14:46                                   ` Matt Brown
2017-06-02 14:46                                   ` Matt Brown
2017-06-02 15:36                                   ` Serge E. Hallyn
2017-06-02 15:36                                     ` Serge E. Hallyn
2017-06-02 15:36                                     ` Serge E. Hallyn
2017-06-02 16:02                                     ` Matt Brown
2017-06-02 16:02                                       ` Matt Brown
2017-06-02 16:02                                       ` Matt Brown
2017-06-02 16:57                                       ` Serge E. Hallyn
2017-06-02 16:57                                         ` Serge E. Hallyn
2017-06-02 16:57                                         ` Serge E. Hallyn
2017-06-02 16:57                                         ` Serge E. Hallyn
2017-06-02 17:32                                         ` Matt Brown
2017-06-02 17:32                                           ` Matt Brown
2017-06-02 17:32                                           ` Matt Brown
2017-06-02 18:18                                           ` Serge E. Hallyn
2017-06-02 18:18                                             ` Serge E. Hallyn
2017-06-02 18:18                                             ` Serge E. Hallyn
2017-06-02 18:18                                             ` Serge E. Hallyn
2017-06-02 19:22                                             ` Matt Brown
2017-06-02 19:22                                               ` Matt Brown
2017-06-02 19:22                                               ` Matt Brown
2017-06-02 19:25                                               ` Kees Cook
2017-06-02 19:25                                                 ` Kees Cook
2017-06-02 19:25                                                 ` Kees Cook
2017-06-02 19:25                                                 ` Kees Cook
2017-06-02 19:26                                                 ` Matt Brown
2017-06-02 19:26                                                   ` Matt Brown
2017-06-02 19:26                                                   ` Matt Brown
2017-06-02 19:26                                                   ` Matt Brown
2017-06-02 20:05                                       ` Alan Cox
2017-06-02 20:05                                         ` Alan Cox
2017-06-02 20:05                                         ` Alan Cox
2017-06-02 20:11                                         ` Nick Kralevich
2017-06-02 20:11                                           ` Nick Kralevich
2017-06-02 20:11                                           ` Nick Kralevich
2017-06-02 20:46                                         ` Matt Brown
2017-06-02 20:46                                           ` Matt Brown
2017-06-02 20:46                                           ` Matt Brown
2017-06-03 22:00                                           ` Alan Cox
2017-06-03 22:00                                             ` Alan Cox
2017-06-03 22:00                                             ` Alan Cox
2017-06-03 22:22                                             ` Matt Brown
2017-06-03 22:22                                               ` Matt Brown
2017-06-03 22:22                                               ` Matt Brown
2017-06-04  3:37                                               ` Peter Dolding
2017-06-04  3:37                                                 ` Peter Dolding
2017-06-04  3:37                                                 ` Peter Dolding
2017-05-30 15:20               ` Casey Schaufler
2017-05-30 15:20                 ` Casey Schaufler
2017-05-30 16:09                 ` Matt Brown
2017-05-30 16:09                   ` Matt Brown
2017-06-04  6:29         ` Boris Lukashev
2017-06-04  6:29           ` Boris Lukashev
2017-05-31  2:48       ` James Morris
2017-05-31  2:48         ` James Morris
2017-05-31  4:10         ` Matt Brown
2017-05-31  4:10           ` Matt Brown
2017-05-30  0:15     ` Matt Brown
2017-05-30  0:15       ` [kernel-hardening] " Matt Brown
2017-05-30  0:15       ` Matt Brown

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=6e713105-02db-12e1-68a9-f167fe11d0a9@gmail.com \
    --to=lazytyped@gmail.com \
    --cc=kernel-hardening@lists.openwall.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.