[GIT PULL] fscrypt fix for v6.3-rc4

Linus Torvalds torvalds at linux-foundation.org
Mon Mar 20 22:16:48 UTC 2023


On Mon, Mar 20, 2023 at 1:56 PM Eric Biggers <ebiggers at kernel.org> wrote:
>
>       fscrypt: check for NULL keyring in fscrypt_put_master_key_activeref()

Side note: please just use WARN_ON_ONCE() for things like this, not WARN_ON.

If it's triggerable, it should be triggered only once rather than
flood the logs and possibly cause a DoS.

And if it's not triggerable, the "ONCE" doesn't matter.

I note that fscypt in general seems to be *way* too happy with
WARN_ON() as some kind of debugging aid.

It's not good in general (printf for debugging is wonderful, but
shouldn't be left in the sources to rot for all eternity), but it's
particularly not good in that form.

              Linus



More information about the Linux-security-module-archive mailing list