From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752254AbdKHMAv (ORCPT ); Wed, 8 Nov 2017 07:00:51 -0500 Received: from lhrrgout.huawei.com ([194.213.3.17]:39857 "EHLO lhrrgout.huawei.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752006AbdKHMAt (ORCPT ); Wed, 8 Nov 2017 07:00:49 -0500 Subject: Re: [PATCH v2 00/15] ima: digest list feature To: Matthew Garrett CC: linux-integrity , , , , , References: <20171107103710.10883-1-roberto.sassu@huawei.com> <899b68a6-fefe-a6db-d624-ea83f597caf1@huawei.com> From: Roberto Sassu Message-ID: Date: Wed, 8 Nov 2017 13:00:37 +0100 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset="utf-8"; format=flowed Content-Language: en-GB Content-Transfer-Encoding: 7bit X-Originating-IP: [10.220.96.228] X-CFilter-Loop: Reflected X-Mirapoint-Virus-RAPID-Raw: score=unknown(0), refid=str=0001.0A020201.5A02F1EF.012A,ss=1,re=0.000,recu=0.000,reip=0.000,cl=1,cld=1,fgs=0, ip=0.0.0.0, so=2013-06-18 04:22:30, dmn=2013-03-21 17:37:32 X-Mirapoint-Loop-Id: d392086e7ca9da2fbadb14e6d9fc2f46 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 11/7/2017 7:06 PM, Matthew Garrett wrote: > On Tue, Nov 7, 2017 at 12:53 PM, Roberto Sassu wrote: >> On 11/7/2017 3:49 PM, Matthew Garrett wrote: >>> RPM's hardly universal, and distributions are in the process of moving >>> away from using it for distributing non-core applications (Flatpak and >>> Snap are becoming increasingly popular here). I think this needs to be >>> a generic solution rather than having the kernel tied to a specific >>> package format. >> >> >> Support for new digest list formats can be easily added. Digest list >> metadata includes the digest list type, so that the appropriate parser >> is selected. > > But we're still left in a state where the kernel has to end up > supporting a number of very niche formats, and userland agility is > tied to the kernel. I think it makes significantly more sense to push > the problem out to userland. At least for appraisal, digest lists must be parsed by the kernel. If the parser is moved to userspace, I don't know if we are able to provide the same guarantee, that the correct set of digests has been uploaded to IMA. A new measurement can be added, when IMA receives the digests, but a verifier has to verify the signature of the original file, perform format conversion, calculate the digest and compare it with that in the new IMA measurement. If digest lists are parsed directly by the kernel, then the signature can be verified directly. >> Digest lists should be parsed directly by the kernel, because processing >> the lists in userspace would increase the chances that a compromised >> tool does not upload to the kernel the expected digests. Also, digest >> lists must be processed before init, otherwise appraisal will deny the >> execution. Lastly, the mechanism of parsing files from the kernel is >> already used to parse the IMA policy. > > Isn't failing to upload the expected digest list just a DoS? We > already expect to load keys from initramfs, so it seems fine to parse > stuff there - what's the problem with extracting information from > RPMs, translating them to the generic format and pushing that into the > kernel? The main problem is that the digest list measurement, performed when the parser accesses the file containing the RPM header, might not reflect what IMA uses for digest lookup. Roberto -- HUAWEI TECHNOLOGIES Duesseldorf GmbH, HRB 56063 Managing Director: Bo PENG, Qiuen PENG, Shengli WANG