[PATCH 0/4] UBIFS: add xattr support for security / SELinux
From: Ben Shelton
Date: Fri Oct 31 2014 - 15:43:13 EST
I'm reposting the patch series for security xattr / SELinux support on UBIFS
from Subodh Nijsure and Marc Kleine-Budde [1] in order to restart the process
of getting this support upstream.
Notes:
- I removed 'UBIFS: xattr: protect ui_size and data_len by ui_mutex' because
after looking through the comments before the definition of struct
ubifs_inode, I'm not sure what this was intended to fix. It looks like
i_size and data_len are not intended to be protected by ui_mutex, and I'm
unclear on why ui->ui_size needs to be protected here by host_ui's ui_mutex.
CCing Marc -- could you comment on how this is supposed to work?
- I made the suggested locking fixes in [2], with the exception of removing the
i_mutex lock/unlock around the call to security_inode_init_security(), which
caused an assert. With these fixes, I turned on lockdep and ran with SELinux
enabled on an ARM-based embedded target using UBIFS, and I saw no lockdep
warnings during filesystem labeling and normal operation.
[1] http://lists.infradead.org/pipermail/linux-mtd/2013-February/045794.html
[2] http://lists.infradead.org/pipermail/linux-mtd/2013-February/045871.html
Subodh Nijsure (4):
UBIFS: fix a couple bugs in UBIFS xattr length calculation
UBIFS: Add xattr support for symlinks
UBIFS: Add security.* XATTR support for the UBIFS
UBIFS: add ubifs_err() to print error reason
fs/ubifs/dir.c | 20 +++++++++
fs/ubifs/file.c | 4 ++
fs/ubifs/journal.c | 11 ++++-
fs/ubifs/super.c | 1 +
fs/ubifs/ubifs.h | 4 ++
fs/ubifs/xattr.c | 116 ++++++++++++++++++++++++++++++++++++++++++++++++-----
6 files changed, 145 insertions(+), 11 deletions(-)
--
2.1.1
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/