Received: by 2002:a25:ab43:0:0:0:0:0 with SMTP id u61csp331384ybi; Fri, 7 Jun 2019 08:45:35 -0700 (PDT) X-Google-Smtp-Source: APXvYqwbmkpVqWrR2/NpHH44LmSrrvRJjwrrNk9XASdMUUE5h3cNKMb2KkOEGjMl68VOsine3dkR X-Received: by 2002:a63:e304:: with SMTP id f4mr3343421pgh.187.1559922335330; Fri, 07 Jun 2019 08:45:35 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1559922335; cv=none; d=google.com; s=arc-20160816; b=xRyN8yBGSlDelgRGM2NGuxWSldGxpLG/nyAh6z9pX5OeEd+MrgwmUPHRd0iSm+Yi98 bB6SJZAp5U8gMdvElgd6lvHM942qNTKou5BpOE4kjiqxElDIlx8M6hO22kaeVlPaNT3t jP/hsRYkB7tWWa4suUp6Y8LXdMHzsgNhbN2RJEV32BOKPoxScG2OTse1WbR8L6RJ7E/p Mq96t+Wo0r2O50LQ9AeyIm1v166qbZcJ/6Fj1G98F+Owd9jk0WFmv2o8ocoTxSXQh6l1 hQQB9I5sqWKwGBz0oDCle8BYlVWyc/L+1OSGTy81iJXmGrQoxcGcCBsk5G2uXuq+2kss ZZcg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :user-agent:references:in-reply-to:message-id:date:subject:cc:to :from:dkim-signature; bh=i/F5E+f2AfgqeUq3Z2ahEYlT/XxMGhHFd9ZJsOMEUUE=; b=AhTfbvvKXkmK9KZ7W3A5KjkcCGrjCDho7lgbHmhopZ1429CHmdx7CBcFLpJ+6K+nwV yVSfoqNEb//ptZ0MNqBpD+x4BljkuMycwnqwCXd3bVi5FxGQHPh6OiOb/vMH9xAb2vSd TAXx7from7C+cyaCzr00B3mYxaXMtNhS9opHnstNYFpxzsDTfi5W+Hl86aG6XczTusqG of9Coc5K9twXwx+dShkQne2frKt/8YtEDRye1ih2NZEoWs5GSjH95CjttVS/N/hUGxDl FXO1wywcOw+y/noCDqe5jTV5lBEBW5tdbVqX3QzfRlnuBTrDuAa2stPT4qXfBGDlXukp tttQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=C9VqTehg; 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 Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id q16si2073721pgc.422.2019.06.07.08.45.18; Fri, 07 Jun 2019 08:45:35 -0700 (PDT) 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=@kernel.org header.s=default header.b=C9VqTehg; 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 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1730781AbfFGPnJ (ORCPT + 99 others); Fri, 7 Jun 2019 11:43:09 -0400 Received: from mail.kernel.org ([198.145.29.99]:54114 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1730769AbfFGPnH (ORCPT ); Fri, 7 Jun 2019 11:43:07 -0400 Received: from localhost (83-86-89-107.cable.dynamic.v4.ziggo.nl [83.86.89.107]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 42DD82133D; Fri, 7 Jun 2019 15:43:06 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1559922186; bh=bUkpDoWOEbrLwB7zrUSqeSUqVZH9xXs2KqtxME+PaVA=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=C9VqTehgwpJ4P97JRv3z18r1Il1vJWCa5KuKXYbtp0UBfKRGN4v25oIsuGcYaPbc4 QLdBqXFgtfGvQprOIJ0/kBdtwNM+2cXWkEo4ZLZNA3xP4YI6EHBrYgriMpOnRs63an sDKLInFylc27Dal0Lt/YVgcEoPGbPMc9yfjx9DMQ= From: Greg Kroah-Hartman To: linux-kernel@vger.kernel.org Cc: Greg Kroah-Hartman , stable@vger.kernel.org, Ben Hutchings , Todd Kjos Subject: [PATCH 4.14 67/69] Revert "binder: fix handling of misaligned binder object" Date: Fri, 7 Jun 2019 17:39:48 +0200 Message-Id: <20190607153856.089839424@linuxfoundation.org> X-Mailer: git-send-email 2.21.0 In-Reply-To: <20190607153848.271562617@linuxfoundation.org> References: <20190607153848.271562617@linuxfoundation.org> User-Agent: quilt/0.66 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Todd Kjos This reverts commit 33c6b9ca70a8b066a613e2a3d0331ae8f82aa31a. The commit message is for a different patch. Reverting and then adding the same patch back with the correct commit message. Reported-by: Ben Hutchings Cc: stable # 4.14 Signed-off-by: Todd Kjos Signed-off-by: Greg Kroah-Hartman --- drivers/android/binder_alloc.c | 18 ++++++++++-------- 1 file changed, 10 insertions(+), 8 deletions(-) --- a/drivers/android/binder_alloc.c +++ b/drivers/android/binder_alloc.c @@ -945,13 +945,14 @@ enum lru_status binder_alloc_free_page(s index = page - alloc->pages; page_addr = (uintptr_t)alloc->buffer + index * PAGE_SIZE; - - mm = alloc->vma_vm_mm; - if (!mmget_not_zero(mm)) - goto err_mmget; - if (!down_write_trylock(&mm->mmap_sem)) - goto err_down_write_mmap_sem_failed; vma = binder_alloc_get_vma(alloc); + if (vma) { + if (!mmget_not_zero(alloc->vma_vm_mm)) + goto err_mmget; + mm = alloc->vma_vm_mm; + if (!down_write_trylock(&mm->mmap_sem)) + goto err_down_write_mmap_sem_failed; + } list_lru_isolate(lru, item); spin_unlock(lock); @@ -964,9 +965,10 @@ enum lru_status binder_alloc_free_page(s PAGE_SIZE); trace_binder_unmap_user_end(alloc, index); + + up_write(&mm->mmap_sem); + mmput(mm); } - up_write(&mm->mmap_sem); - mmput(mm); trace_binder_unmap_kernel_start(alloc, index);