lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <20180105204009.GA72343@jaegeuk-macbookpro.roam.corp.google.com> Date: Fri, 5 Jan 2018 12:40:09 -0800 From: Jaegeuk Kim <jaegeuk@...nel.org> To: Eric Biggers <ebiggers3@...il.com> Cc: linux-fscrypt@...r.kernel.org, "Theodore Y . Ts'o" <tytso@....edu>, linux-ext4@...r.kernel.org, linux-f2fs-devel@...ts.sourceforge.net, linux-mtd@...ts.infradead.org, linux-fsdevel@...r.kernel.org, Eric Biggers <ebiggers@...gle.com> Subject: Re: [PATCH v2 05/11] fscrypt: split fscrypt_dummy_context_enabled() into supp/notsupp versions On 01/05, Eric Biggers wrote: > From: Eric Biggers <ebiggers@...gle.com> > > fscrypt_dummy_context_enabled() accesses ->s_cop, which now is only set > when the filesystem is built with encryption support. This didn't > actually matter because no filesystems called it. However, it will > start being used soon, so fix it by moving it from fscrypt.h to > fscrypt_supp.h and stubbing it out in fscrypt_notsupp.h. Ted, do we have a chance to get rid of this dummy_context? If there exists backward compatibility issue, please never mind tho. Thanks, > > Signed-off-by: Eric Biggers <ebiggers@...gle.com> > --- > include/linux/fscrypt.h | 8 -------- > include/linux/fscrypt_notsupp.h | 5 +++++ > include/linux/fscrypt_supp.h | 6 ++++++ > 3 files changed, 11 insertions(+), 8 deletions(-) > > diff --git a/include/linux/fscrypt.h b/include/linux/fscrypt.h > index 0f94d087a6d1..b671a4eef47f 100644 > --- a/include/linux/fscrypt.h > +++ b/include/linux/fscrypt.h > @@ -75,14 +75,6 @@ struct fscrypt_operations { > /* Maximum value for the third parameter of fscrypt_operations.set_context(). */ > #define FSCRYPT_SET_CONTEXT_MAX_SIZE 28 > > -static inline bool fscrypt_dummy_context_enabled(struct inode *inode) > -{ > - if (inode->i_sb->s_cop->dummy_context && > - inode->i_sb->s_cop->dummy_context(inode)) > - return true; > - return false; > -} > - > static inline bool fscrypt_valid_enc_modes(u32 contents_mode, > u32 filenames_mode) > { > diff --git a/include/linux/fscrypt_notsupp.h b/include/linux/fscrypt_notsupp.h > index 812dc701a5b3..81e02201b215 100644 > --- a/include/linux/fscrypt_notsupp.h > +++ b/include/linux/fscrypt_notsupp.h > @@ -19,6 +19,11 @@ static inline bool fscrypt_has_encryption_key(const struct inode *inode) > return false; > } > > +static inline bool fscrypt_dummy_context_enabled(struct inode *inode) > +{ > + return false; > +} > + > /* crypto.c */ > static inline struct fscrypt_ctx *fscrypt_get_ctx(const struct inode *inode, > gfp_t gfp_flags) > diff --git a/include/linux/fscrypt_supp.h b/include/linux/fscrypt_supp.h > index fd6ee089ced0..e7dfa2974906 100644 > --- a/include/linux/fscrypt_supp.h > +++ b/include/linux/fscrypt_supp.h > @@ -31,6 +31,12 @@ static inline bool fscrypt_has_encryption_key(const struct inode *inode) > return (inode->i_crypt_info != NULL); > } > > +static inline bool fscrypt_dummy_context_enabled(struct inode *inode) > +{ > + return inode->i_sb->s_cop->dummy_context && > + inode->i_sb->s_cop->dummy_context(inode); > +} > + > /* crypto.c */ > extern struct fscrypt_ctx *fscrypt_get_ctx(const struct inode *, gfp_t); > extern void fscrypt_release_ctx(struct fscrypt_ctx *); > -- > 2.16.0.rc0.223.g4a4ac83678-goog
Powered by blists - more mailing lists