From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alasdair G Kergon Subject: Re: [PATCH] dm ioctl: prevent stack leak in dm ioctl call Date: Thu, 27 Apr 2017 16:43:48 +0100 Message-ID: <20170427154348.GD5330@agk-dp.fab.redhat.com> References: <20170425233129.GA155598@google.com> <20170426001117.GA7433@agk-dp.fab.redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dm-devel-bounces@redhat.com Errors-To: dm-devel-bounces@redhat.com To: Adrian Salido Cc: dm-devel@redhat.com, mpatocka@redhat.com, snitzer@redhat.com List-Id: dm-devel.ids On Tue, Apr 25, 2017 at 05:33:19PM -0700, Adrian Salido wrote: > Will update the patch to be clear So at the moment, we're leaning towards just: param->data_size = offsetof(struct dm_ioctl, data) to replace param->data_size = sizeof(*param); in the caller. Alasdair