Received: by 10.223.185.116 with SMTP id b49csp1086249wrg; Sat, 3 Mar 2018 15:10:00 -0800 (PST) X-Google-Smtp-Source: AG47ELugljfjvIxgssP43mzQSKWxsy8jckAjJWXtCq7JTajRYkvqL/I4DHQiBid+gW06TorvdgAF X-Received: by 10.99.143.88 with SMTP id r24mr8230022pgn.160.1520118600328; Sat, 03 Mar 2018 15:10:00 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1520118600; cv=none; d=google.com; s=arc-20160816; b=yEq5EsgCc0oNoVWB9Z+lQDeXxiNsuRbguMjp4NVAKAeKFy20wqXhIZ2//IlliWYT3E MYNo6CJ8crEtY4ORl61giZSpgmG+Px9F+rMkwSifh/Q4lYQOJVius92wt4YYLpZyXpTj xM+Fj5H4ql9Gx/hme4gG+wjIIS04durrFRBfSg2EoilxQ2Wss2O3J1FK8nVxYRXWDXU5 Nx1mV9jID7197RE91baGEFsDN+vCBj5z6YlP0ewq3gOMJ3K1IHAh54ew1kOhNmY1+iOE G3t54XBFFDF5qV7IfP8fnn9Ru0M478FmwTptwGVMioUhvJ7riSCs4A8rzurHrvcJ1PB/ RURQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:mime-version:content-transfer-encoding :spamdiagnosticmetadata:spamdiagnosticoutput:content-language :accept-language:in-reply-to:references:message-id:date:thread-index :thread-topic:subject:cc:to:from:dkim-signature :arc-authentication-results; bh=nPHAq11Q420IXlAEGV9qz0zcuO+RH9LXLTUBCs5R/WM=; b=uyStvgo7vTko8/HheI9rIZuegXgW0EOAB4ai1Lm73dmOFHI/mqTk6R1AWOF0pTPi6F rxx90me6Tv5chstyKW++GqY+F7Vgp+fGsN8c1zMIr3Svze7PJI+Suk1BtpuaBICja59f ApUtUo3VfIj9PDRGNivn6YbqStbHbnwY6nm6hgyIWybV/fPG23oOb0bd5xPcLkv3mh0J o5jR5V4UWoQ1RcolL54B+/rdKmEB5i2JvJAeGiuyKZqtQ0IAQdTvsRcCPR7bFP9RIbxQ TkTt/PVhbYDYK0jOKcn+xLWC5jlao5zpggIVq+L9ZBDX/KPy9Bzuvi72AqSGeRhm29Zl Gdow== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@microsoft.com header.s=selector1 header.b=On+5t9Bd; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=microsoft.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id c63si7473606pfd.51.2018.03.03.15.09.46; Sat, 03 Mar 2018 15:10:00 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@microsoft.com header.s=selector1 header.b=On+5t9Bd; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=microsoft.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934906AbeCCWjy (ORCPT + 99 others); Sat, 3 Mar 2018 17:39:54 -0500 Received: from mail-dm3nam03on0102.outbound.protection.outlook.com ([104.47.41.102]:24173 "EHLO NAM03-DM3-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S935101AbeCCWj1 (ORCPT ); Sat, 3 Mar 2018 17:39:27 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=nPHAq11Q420IXlAEGV9qz0zcuO+RH9LXLTUBCs5R/WM=; b=On+5t9BdE1wnnk/VMRqN+ifmWSlfbSV+ndFD8dkEw7Qlb9RpDxqhJ3zFVmC8D2htAR4cMWC0iwe1wt9Mo6iCr7Z46dfNv+d1+Z4eBPJ9AgKA4LC4CytCZlpjYC8omxxrjwBxdb4IGigpkQwZ5W9JfFKRVx88Y0diH7nQQWcAR1o= Received: from MW2PR2101MB1034.namprd21.prod.outlook.com (52.132.149.10) by MW2PR2101MB1113.namprd21.prod.outlook.com (52.132.149.30) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.588.1; Sat, 3 Mar 2018 22:39:21 +0000 Received: from MW2PR2101MB1034.namprd21.prod.outlook.com ([fe80::1d56:338f:e2b:cec0]) by MW2PR2101MB1034.namprd21.prod.outlook.com ([fe80::1d56:338f:e2b:cec0%3]) with mapi id 15.20.0567.006; Sat, 3 Mar 2018 22:39:21 +0000 From: Sasha Levin To: "linux-kernel@vger.kernel.org" , "stable@vger.kernel.org" CC: John Johansen , James Morris , Sasha Levin Subject: [PATCH AUTOSEL for 4.4 060/115] apparmor: Make path_max parameter readonly Thread-Topic: [PATCH AUTOSEL for 4.4 060/115] apparmor: Make path_max parameter readonly Thread-Index: AQHTsz9cMZ/KH0dpbU212Q8P+2Up/A== Date: Sat, 3 Mar 2018 22:31:23 +0000 Message-ID: <20180303223010.27106-60-alexander.levin@microsoft.com> References: <20180303223010.27106-1-alexander.levin@microsoft.com> In-Reply-To: <20180303223010.27106-1-alexander.levin@microsoft.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [52.168.54.252] x-ms-publictraffictype: Email x-microsoft-exchange-diagnostics: 1;MW2PR2101MB1113;7:AWgtLyp+qfCDawWFxalNmAxWp9Ili9n92CKImB1YRJjby1Y95/t+QR5BYj61HtMFch0Az0+IU8Q/VCP410VtyGn0OS8twjySifGojMhQM30SlUCiTHZ+GgyHJzBFQ29TFzwwkDbpgbw1fDQofhAMGK9KqVVGWcXj0fJMmJ/F+5FJVf8pSS4jmr7UA4Vd7ewGFhUn8iD0wM4rkou7TNVKITjdKQeWYAQNYFtGMBrLCgg/k6Ecjuht88xiA7gdFI+Q x-ms-office365-filtering-ht: Tenant x-ms-office365-filtering-correlation-id: cb7cd477-8976-413a-25de-08d581579bbe x-microsoft-antispam: UriScan:;BCL:0;PCL:0;RULEID:(7020095)(4652020)(48565401081)(5600026)(4604075)(3008032)(4534165)(4627221)(201703031133081)(201702281549075)(2017052603307)(7193020);SRVR:MW2PR2101MB1113; x-ms-traffictypediagnostic: MW2PR2101MB1113: x-microsoft-antispam-prvs: x-exchange-antispam-report-test: UriScan:(28532068793085)(274141268393343)(89211679590171)(192374486261705)(146099531331640)(198206253151910); x-exchange-antispam-report-cfa-test: BCL:0;PCL:0;RULEID:(8211001083)(61425038)(6040501)(2401047)(5005006)(8121501046)(3002001)(3231220)(944501244)(52105095)(10201501046)(93006095)(93001095)(6055026)(61426038)(61427038)(6041288)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123560045)(20161123564045)(20161123558120)(20161123562045)(6072148)(201708071742011);SRVR:MW2PR2101MB1113;BCL:0;PCL:0;RULEID:;SRVR:MW2PR2101MB1113; x-forefront-prvs: 0600F93FE1 x-forefront-antispam-report: SFV:NSPM;SFS:(10019020)(39380400002)(366004)(346002)(39860400002)(396003)(376002)(199004)(189003)(68736007)(7736002)(10090500001)(6116002)(3846002)(1076002)(2906002)(5250100002)(6486002)(2501003)(6436002)(76176011)(86362001)(22452003)(6512007)(53936002)(66066001)(86612001)(99286004)(59450400001)(2900100001)(316002)(36756003)(54906003)(106356001)(110136005)(6506007)(102836004)(186003)(97736004)(26005)(6346003)(14454004)(3660700001)(8936002)(107886003)(10290500003)(5660300001)(81166006)(478600001)(3280700002)(105586002)(8676002)(81156014)(4326008)(72206003)(2950100002)(305945005)(25786009)(22906009)(217873001);DIR:OUT;SFP:1102;SCL:1;SRVR:MW2PR2101MB1113;H:MW2PR2101MB1034.namprd21.prod.outlook.com;FPR:;SPF:None;PTR:InfoNoRecords;A:1;MX:1;LANG:en; received-spf: None (protection.outlook.com: microsoft.com does not designate permitted sender hosts) authentication-results: spf=none (sender IP is ) smtp.mailfrom=Alexander.Levin@microsoft.com; x-microsoft-antispam-message-info: ZdZssPQrDzhxnqNh+qc88l/pHYw628/5fzSD+Rl8k6HbAf2GlokTBVI3sa4oZWFNm0k13XmTSqes+Lp/SyqfQSJHVaAqe4VTLV/mOKMPLEUFW8LnpmTxJnpniYXJiMrd5HCRhtwzh0+NnHL0vXVW/B2ng1Z7CwvLAiXsQL9hIYM= spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-OriginatorOrg: microsoft.com X-MS-Exchange-CrossTenant-Network-Message-Id: cb7cd477-8976-413a-25de-08d581579bbe X-MS-Exchange-CrossTenant-originalarrivaltime: 03 Mar 2018 22:31:23.6978 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 72f988bf-86f1-41af-91ab-2d7cd011db47 X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW2PR2101MB1113 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: John Johansen [ Upstream commit 622f6e3265707ebf02ba776ac6e68003bcc31213 ] The path_max parameter determines the max size of buffers allocated but it should not be setable at run time. If can be used to cause an oops root@ubuntu:~# echo 16777216 > /sys/module/apparmor/parameters/path_max root@ubuntu:~# cat /sys/module/apparmor/parameters/path_max Killed [ 122.141911] BUG: unable to handle kernel paging request at ffff880080945= fff [ 122.143497] IP: [] d_absolute_path+0x44/0xa0 [ 122.144742] PGD 220c067 PUD 0 [ 122.145453] Oops: 0002 [#1] SMP [ 122.146204] Modules linked in: vmw_vsock_vmci_transport vsock ppdev vmw_= balloon snd_ens1371 btusb snd_ac97_codec gameport snd_rawmidi btrtl snd_seq= _device ac97_bus btbcm btintel snd_pcm input_leds bluetooth snd_timer snd j= oydev soundcore serio_raw coretemp shpchp nfit parport_pc i2c_piix4 8250_fi= ntek vmw_vmci parport mac_hid ib_iser rdma_cm iw_cm ib_cm ib_sa ib_mad ib_c= ore ib_addr iscsi_tcp libiscsi_tcp libiscsi scsi_transport_iscsi autofs4 bt= rfs raid10 raid456 async_raid6_recov async_memcpy async_pq async_xor async_= tx xor raid6_pq libcrc32c raid1 raid0 multipath linear hid_generic usbhid h= id crct10dif_pclmul crc32_pclmul ghash_clmulni_intel aesni_intel aes_x86_64= lrw gf128mul glue_helper ablk_helper cryptd vmwgfx psmouse mptspi ttm mpts= csih drm_kms_helper mptbase syscopyarea scsi_transport_spi sysfillrect [ 122.163365] ahci sysimgblt e1000 fb_sys_fops libahci drm pata_acpi fjes [ 122.164747] CPU: 3 PID: 1501 Comm: bash Not tainted 4.4.0-59-generic #80= -Ubuntu [ 122.166250] Hardware name: VMware, Inc. VMware Virtual Platform/440BX De= sktop Reference Platform, BIOS 6.00 07/02/2015 [ 122.168611] task: ffff88003496aa00 ti: ffff880076474000 task.ti: ffff880= 076474000 [ 122.170018] RIP: 0010:[] [] d_absol= ute_path+0x44/0xa0 [ 122.171525] RSP: 0018:ffff880076477b90 EFLAGS: 00010206 [ 122.172462] RAX: ffff880080945fff RBX: 0000000000000000 RCX: 00000000010= 00000 [ 122.173709] RDX: 0000000000ffffff RSI: ffff880080946000 RDI: ffff8800348= a1010 [ 122.174978] RBP: ffff880076477bb8 R08: ffff880076477c80 R09: 00000000000= 00000 [ 122.176227] R10: 00007ffffffff000 R11: ffff88007f946000 R12: ffff88007f9= 46000 [ 122.177496] R13: ffff880076477c80 R14: ffff8800348a1010 R15: ffff8800348= a2400 [ 122.178745] FS: 00007fd459eb4700(0000) GS:ffff88007b6c0000(0000) knlGS:= 0000000000000000 [ 122.180176] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033 [ 122.181186] CR2: ffff880080945fff CR3: 0000000073422000 CR4: 00000000001= 406e0 [ 122.182469] Stack: [ 122.182843] 00ffffff00000001 ffff880080946000 0000000000000000 00000000= 00000000 [ 122.184409] 00000000570f789c ffff880076477c30 ffffffff81385671 ffff8800= 7a2e7a58 [ 122.185810] 0000000000000000 ffff880076477c88 01000000008a1000 00000000= 00000000 [ 122.187231] Call Trace: [ 122.187680] [] aa_path_name+0x81/0x370 [ 122.188637] [] profile_transition+0xbd/0xb80 [ 122.190181] [] ? zone_statistics+0x7c/0xa0 [ 122.191674] [] apparmor_bprm_set_creds+0x9b0/0xac0 [ 122.193288] [] ? ext4_xattr_get+0x81/0x220 [ 122.194793] [] ? ext4_xattr_security_get+0x1c/0x30 [ 122.196392] [] ? get_vfs_caps_from_disk+0x69/0x110 [ 122.198004] [] ? mnt_may_suid+0x3f/0x50 [ 122.199737] [] ? cap_bprm_set_creds+0xa3/0x600 [ 122.201377] [] security_bprm_set_creds+0x33/0x50 [ 122.203024] [] prepare_binprm+0x85/0x190 [ 122.204515] [] do_execveat_common.isra.33+0x485/0x710 [ 122.206200] [] SyS_execve+0x3a/0x50 [ 122.207615] [] stub_execve+0x5/0x5 [ 122.208978] [] ? entry_SYSCALL_64_fastpath+0x16/0x71 [ 122.210615] Code: f8 31 c0 48 63 c2 83 ea 01 48 c7 45 e8 00 00 00 00 48 = 01 c6 85 d2 48 c7 45 f0 00 00 00 00 48 89 75 e0 89 55 dc 78 0c 48 8d 46 ff = 46 ff 00 48 89 45 e0 48 8d 55 e0 48 8d 4d dc 48 8d 75 e8 e8 [ 122.217320] RIP [] d_absolute_path+0x44/0xa0 [ 122.218860] RSP [ 122.219919] CR2: ffff880080945fff [ 122.220936] ---[ end trace 506cdbd85eb6c55e ]--- Reported-by: Tetsuo Handa Signed-off-by: John Johansen Signed-off-by: James Morris Signed-off-by: Sasha Levin --- security/apparmor/lsm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/security/apparmor/lsm.c b/security/apparmor/lsm.c index dec607c17b64..6dc4ce47580f 100644 --- a/security/apparmor/lsm.c +++ b/security/apparmor/lsm.c @@ -722,7 +722,7 @@ module_param_named(logsyscall, aa_g_logsyscall, aabool,= S_IRUSR | S_IWUSR); =20 /* Maximum pathname length before accesses will start getting rejected */ unsigned int aa_g_path_max =3D 2 * PATH_MAX; -module_param_named(path_max, aa_g_path_max, aauint, S_IRUSR | S_IWUSR); +module_param_named(path_max, aa_g_path_max, aauint, S_IRUSR); =20 /* Determines how paranoid loading of policy is and how much verification * on the loaded policy is done. --=20 2.14.1