Chao Yu
f52c88e8e1
ecryptfs: avoid to access NULL pointer when write metadata in xattr
commit 35425ea249 upstream.
Christopher Head 2014-06-28 05:26:20 UTC described:
"I tried to reproduce this on 3.12.21. Instead, when I do "echo hello > foo"
in an ecryptfs mount with ecryptfs_xattr specified, I get a kernel crash:
BUG: unable to handle kernel NULL pointer dereference at (null)
IP: [<ffffffff8110eb39>] fsstack_copy_attr_all+0x2/0x61
PGD d7840067 PUD b2c3c067 PMD 0
Oops: 0002 [#1] SMP
Modules linked in: nvidia(PO)
CPU: 3 PID: 3566 Comm: bash Tainted: P O 3.12.21-gentoo-r1 #2
Hardware name: ASUSTek Computer Inc. G60JX/G60JX, BIOS 206 03/15/2010
task: ffff8801948944c0 ti: ffff8800bad70000 task.ti: ffff8800bad70000
RIP: 0010:[<ffffffff8110eb39>] [<ffffffff8110eb39>] fsstack_copy_attr_all+0x2/0x61
RSP: 0018:ffff8800bad71c10 EFLAGS: 00010246
RAX: 00000000000181a4 RBX: ffff880198648480 RCX: 0000000000000000
RDX: 0000000000000004 RSI: ffff880172010450 RDI: 0000000000000000
RBP: ffff880198490e40 R08: 0000000000000000 R09: 0000000000000000
R10: ffff880172010450 R11: ffffea0002c51e80 R12: 0000000000002000
R13: 000000000000001a R14: 0000000000000000 R15: ffff880198490e40
FS: 00007ff224caa700(0000) GS:ffff88019fcc0000(0000) knlGS:0000000000000000
CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
CR2: 0000000000000000 CR3: 00000000bb07f000 CR4: 00000000000007e0
Stack:
ffffffff811826e8 ffff8800a39d8000 0000000000000000 000000000000001a
ffff8800a01d0000 ffff8800a39d8000 ffffffff81185fd5 ffffffff81082c2c
00000001a39d8000 53d0abbc98490e40 0000000000000037 ffff8800a39d8220
Call Trace:
[<ffffffff811826e8>] ? ecryptfs_setxattr+0x40/0x52
[<ffffffff81185fd5>] ? ecryptfs_write_metadata+0x1b3/0x223
[<ffffffff81082c2c>] ? should_resched+0x5/0x23
[<ffffffff8118322b>] ? ecryptfs_initialize_file+0xaf/0xd4
[<ffffffff81183344>] ? ecryptfs_create+0xf4/0x142
[<ffffffff810f8c0d>] ? vfs_create+0x48/0x71
[<ffffffff810f9c86>] ? do_last.isra.68+0x559/0x952
[<ffffffff810f7ce7>] ? link_path_walk+0xbd/0x458
[<ffffffff810fa2a3>] ? path_openat+0x224/0x472
[<ffffffff810fa7bd>] ? do_filp_open+0x2b/0x6f
[<ffffffff81103606>] ? __alloc_fd+0xd6/0xe7
[<ffffffff810ee6ab>] ? do_sys_open+0x65/0xe9
[<ffffffff8157d022>] ? system_call_fastpath+0x16/0x1b
RIP [<ffffffff8110eb39>] fsstack_copy_attr_all+0x2/0x61
RSP <ffff8800bad71c10>
CR2: 0000000000000000
---[ end trace df9dba5f1ddb8565 ]---"
If we create a file when we mount with ecryptfs_xattr_metadata option, we will
encounter a crash in this path:
->ecryptfs_create
->ecryptfs_initialize_file
->ecryptfs_write_metadata
->ecryptfs_write_metadata_to_xattr
->ecryptfs_setxattr
->fsstack_copy_attr_all
It's because our dentry->d_inode used in fsstack_copy_attr_all is NULL, and it
will be initialized when ecryptfs_initialize_file finish.
So we should skip copying attr from lower inode when the value of ->d_inode is
invalid.
Signed-off-by: Chao Yu <chao2.yu@samsung.com>
Signed-off-by: Tyler Hicks <tyhicks@canonical.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-10-30 09:35:13 -07:00
..
2013-05-07 20:16:25 -07:00
2013-03-03 19:36:31 -08:00
2013-03-03 19:36:31 -08:00
2013-05-07 20:16:25 -07:00
2013-05-06 13:06:59 -07:00
2013-05-31 15:17:56 -04:00
2013-03-03 19:36:31 -08:00
2014-10-30 09:35:09 -07:00
2013-04-09 14:12:56 -04:00
2014-01-09 12:24:25 -08:00
2014-10-05 14:54:11 -07:00
2013-04-09 14:12:56 -04:00
2013-11-29 11:11:53 -08:00
2013-03-03 19:36:31 -08:00
2013-08-14 22:59:10 -07:00
2013-12-04 10:55:49 -08:00
2013-05-01 14:08:52 -07:00
2014-10-30 09:35:13 -07:00
2013-05-13 20:12:10 +01:00
2013-03-03 19:36:31 -08:00
2014-02-13 13:48:00 -08:00
2013-02-27 19:10:24 -08:00
2014-10-05 14:54:15 -07:00
2013-07-21 18:21:23 -07:00
2014-09-05 16:28:36 -07:00
2013-05-08 15:11:48 -07:00
2013-05-24 16:22:50 -07:00
2013-03-12 18:55:21 -07:00
2013-04-29 15:54:27 -07:00
2014-07-28 08:00:02 -07:00
2014-01-25 08:27:11 -08:00
2013-05-24 16:22:51 -07:00
2013-05-07 20:16:25 -07:00
2013-05-04 15:48:45 -04:00
2013-07-13 11:42:26 -07:00
2013-04-29 15:41:42 -04:00
2013-10-18 07:45:45 -07:00
2014-09-05 16:28:34 -07:00
2013-05-03 09:56:25 -07:00
2014-09-05 16:28:36 -07:00
2014-04-26 17:15:36 -07:00
2013-11-13 12:05:31 +09:00
2014-10-05 14:54:13 -07:00
2013-03-23 14:15:31 -07:00
2013-03-03 19:36:31 -08:00
2013-06-07 12:15:38 -04:00
2014-10-30 09:35:11 -07:00
2013-02-13 06:15:14 -08:00
2014-09-05 16:28:37 -07:00
2014-10-05 14:54:14 -07:00
2014-10-30 09:35:12 -07:00
2013-05-07 20:16:25 -07:00
2014-10-05 14:54:14 -07:00
2013-03-03 19:36:31 -08:00
2013-03-03 19:36:31 -08:00
2014-09-17 09:03:57 -07:00
2013-05-09 16:42:10 -07:00
2013-03-03 19:36:31 -08:00
2013-05-31 15:17:31 -04:00
2014-03-06 21:30:12 -08:00
2013-02-26 20:16:07 -08:00
2014-07-06 18:54:15 -07:00
2013-04-29 09:17:57 +10:00
2013-03-11 07:09:48 -07:00
2013-04-05 15:35:52 -07:00
2013-10-05 07:13:09 -07:00
2014-07-06 18:54:13 -07:00
2014-10-09 12:18:42 -07:00
2013-04-30 09:36:50 -07:00
2014-09-17 09:04:01 -07:00
2014-06-30 20:09:45 -07:00
2013-02-26 02:46:11 -05:00
2014-06-16 13:42:52 -07:00
2013-05-01 17:51:54 -07:00
2013-05-02 10:16:16 -07:00
2013-10-13 16:08:31 -07:00
2013-04-29 15:40:23 -04:00
2013-04-30 17:04:03 -07:00
2012-12-20 17:40:19 -08:00
2014-03-23 21:38:21 -07:00
2013-10-05 07:13:09 -07:00
2013-07-28 16:29:40 -07:00
2014-10-05 14:54:14 -07:00
2013-05-04 15:03:05 -04:00
2013-05-07 20:16:25 -07:00
2014-07-31 12:53:50 -07:00
2014-09-17 09:04:02 -07:00
2014-02-13 13:48:00 -08:00
2013-05-08 10:13:35 -07:00
2013-05-01 07:21:43 -07:00
2014-06-07 13:25:38 -07:00
2013-02-22 23:31:31 -05:00
2012-12-20 14:04:11 -08:00
2014-05-30 21:52:12 -07:00
2014-02-22 12:41:25 -08:00
2013-03-03 19:36:31 -08:00
2013-03-01 23:51:07 -05:00
2014-04-26 17:15:35 -07:00
2014-06-16 13:42:52 -07:00
2013-06-20 19:02:45 +04:00
2013-02-22 23:31:31 -05:00
2013-04-17 13:25:09 +01:00
2013-04-30 17:04:04 -07:00
2012-12-20 18:46:29 -05:00
2014-05-13 13:59:44 +02:00
2013-05-01 17:51:54 -07:00
2014-02-06 11:08:16 -08:00
2014-09-17 09:04:02 -07:00
2014-10-30 09:35:09 -07:00
2014-05-30 21:52:12 -07:00
2013-12-11 22:36:26 -08:00
2013-05-31 15:16:33 -04:00
2013-05-01 17:51:54 -07:00
2014-06-07 13:25:33 -07:00
2014-02-13 13:48:00 -08:00
2013-02-22 23:31:31 -05:00
2013-02-07 20:51:08 +01:00
2013-11-13 12:05:34 +09:00
2013-03-03 22:58:46 -05:00
2014-02-13 13:47:59 -08:00
2013-02-26 02:46:08 -05:00
2013-10-18 07:45:44 -07:00
2013-08-04 16:51:15 +08:00
2013-03-03 22:46:22 -05:00
2013-03-02 09:35:13 -05:00
2012-12-20 18:50:08 -05:00
2012-12-20 18:50:11 -05:00