Received: by 2002:a25:683:0:0:0:0:0 with SMTP id 125csp1232735ybg; Tue, 2 Jun 2020 04:55:16 -0700 (PDT) X-Google-Smtp-Source: ABdhPJw/H1dWowOIHQgPyVpQIgneu6oJ7FUYSNP0rAoH35E4HfIUDG2RemYUFmSUXBqj59FIwApP X-Received: by 2002:a50:ee01:: with SMTP id g1mr1581462eds.44.1591098916349; Tue, 02 Jun 2020 04:55:16 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1591098916; cv=none; d=google.com; s=arc-20160816; b=SDwxL/0wH+yiSfLk+BybpliKj/EPRyJcrTz6V8FCUx+RjulVCDAGzYs5YjTQ9yY8oR 9oJqozU9Njtf0tzPRCRQ9n0NxUkXfshJ709vTei5whd49LEYcfRE2Xc2cdwjyHvrwXsy ovqcdx8NRcSDkmahjBI0PEpFtCdlT0OZwOQT1dxoCmLEunLBDYWK+1IMeJN9IVkD9K5l C8WCQXwW43lWo87ujhsYRIg/Y1TLrbXBgNzWTKm1pqMBAvvVVdRKBUpD4Qtyh3Mn5m0Z qKF8VPwx4CEgSUwSvycwG1XZeYXzV3JSzKmu4+9TgXP2+UgjqCzVhCUD6qRcS8ZUD9xW agRw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:references:cms-type :content-transfer-encoding:content-language:in-reply-to:mime-version :user-agent:date:message-id:cc:to:subject:from:dkim-signature :dkim-filter; bh=DFymYLd+WxEKwPEsz1W/bGa1Yq98RkN02bjpEnWFsTI=; b=N55owA0cVEBt4RVq/IOF2zyYxeFpXkp4SEsivBzXqLcbKvmERdiNhAw9PEMDTIs1L3 sKJMBtSv0ditVhFksX2YNzbAyKqT0FZ/JwVNn3/U2rkN9skJ1CDjUh2KBovrMNDWebuw p0fPF3EroU01z9wCA05TvtYOzzZovalkfYLKgI2XnZV2vkFsQPJD6okDgvnGJEicBg9Y tEe+uAF+NBMos7fanyieiHgvYI6L3Rygi9QLdDtFCnyF1jkfi2eBHg3NKj7WcCdJIVqC OvDhglum5soLFnOm5SwORJcr+0WWtiMU5dLkMdaNM6zklTayQHCYki+6liGulEt3KNDF EWfQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@samsung.com header.s=mail20170921 header.b=b2ReVjeC; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=samsung.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id d16si1341088edx.546.2020.06.02.04.54.53; Tue, 02 Jun 2020 04:55:16 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@samsung.com header.s=mail20170921 header.b=b2ReVjeC; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=samsung.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728373AbgFBLwQ (ORCPT + 99 others); Tue, 2 Jun 2020 07:52:16 -0400 Received: from mailout1.w1.samsung.com ([210.118.77.11]:60236 "EHLO mailout1.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728368AbgFBLwP (ORCPT ); Tue, 2 Jun 2020 07:52:15 -0400 Received: from eucas1p2.samsung.com (unknown [182.198.249.207]) by mailout1.w1.samsung.com (KnoxPortal) with ESMTP id 20200602115214euoutp01fdae2ff0687a588a95c5ba6b77afae9a~Ut95w7_zi0115001150euoutp01P for ; Tue, 2 Jun 2020 11:52:14 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 mailout1.w1.samsung.com 20200602115214euoutp01fdae2ff0687a588a95c5ba6b77afae9a~Ut95w7_zi0115001150euoutp01P DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=samsung.com; s=mail20170921; t=1591098734; bh=DFymYLd+WxEKwPEsz1W/bGa1Yq98RkN02bjpEnWFsTI=; h=From:Subject:To:Cc:Date:In-Reply-To:References:From; b=b2ReVjeCHIbTEYLUNo7df+Ejz6XJk0/Ccdc8vOpFl0fsiA8cUHYj/N6QTpK+TFrd5 gCBBAwkOtprApXI9BUNfrRBkwqJsYtAvhXeMGDNzx/Y9iCgyNh1KJ8RX5dxTzQrhhw +LMqZJ82sjEL2Ce4rkhavDoCcfnjvjpJznYdUPt8= Received: from eusmges1new.samsung.com (unknown [203.254.199.242]) by eucas1p1.samsung.com (KnoxPortal) with ESMTP id 20200602115214eucas1p1ba66739f7837d7083674aa6543cdb6a4~Ut95oRZM40113401134eucas1p14; Tue, 2 Jun 2020 11:52:14 +0000 (GMT) Received: from eucas1p2.samsung.com ( [182.198.249.207]) by eusmges1new.samsung.com (EUCPMTA) with SMTP id AA.39.61286.E6D36DE5; Tue, 2 Jun 2020 12:52:14 +0100 (BST) Received: from eusmtrp2.samsung.com (unknown [182.198.249.139]) by eucas1p2.samsung.com (KnoxPortal) with ESMTPA id 20200602115213eucas1p2a9a2ebb2afccca0354cf196caf762c81~Ut95XPpzJ2114421144eucas1p2H; Tue, 2 Jun 2020 11:52:13 +0000 (GMT) Received: from eusmgms1.samsung.com (unknown [182.198.249.179]) by eusmtrp2.samsung.com (KnoxPortal) with ESMTP id 20200602115213eusmtrp22e94988dfefaa16a48822c76e001af9f~Ut95WrdIp2672126721eusmtrp2H; Tue, 2 Jun 2020 11:52:13 +0000 (GMT) X-AuditID: cbfec7f2-ef1ff7000001ef66-79-5ed63d6e409c Received: from eusmtip1.samsung.com ( [203.254.199.221]) by eusmgms1.samsung.com (EUCPMTA) with SMTP id 7D.4A.08375.D6D36DE5; Tue, 2 Jun 2020 12:52:13 +0100 (BST) Received: from [106.120.51.71] (unknown [106.120.51.71]) by eusmtip1.samsung.com (KnoxPortal) with ESMTPA id 20200602115213eusmtip1bf6e913a99c16fcd84809478e5a5e394~Ut95AXli90275702757eusmtip1h; Tue, 2 Jun 2020 11:52:13 +0000 (GMT) From: Bartlomiej Zolnierkiewicz Subject: [PATCH v2 2/2] video: fbdev: amifb: add FIXMEs about {put,get}_user() failures To: linux-fbdev@vger.kernel.org, dri-devel@lists.freedesktop.org Cc: linux-kernel@vger.kernel.org, linux-m68k , Al Viro , Geert Uytterhoeven Message-ID: <380c0494-ed02-b2be-65b0-d385627fb894@samsung.com> Date: Tue, 2 Jun 2020 13:52:13 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.8.0 MIME-Version: 1.0 In-Reply-To: Content-Language: en-US Content-Transfer-Encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFlrKKsWRmVeSWpSXmKPExsWy7djP87p5ttfiDL7s5ba48vU9m8WzW3uZ LE70fWC1uLxrDpvF8idrmS3O/z3O6sDmcehwB6PH/e7jTB4nW7+xeHzeJOex6clbpgDWKC6b lNSczLLUIn27BK6MM8cPMhUs4qyY/vsEawPjNvYuRk4OCQETiQ/nJrGB2EICKxglvr0p7WLk ArK/MErsuNbGBOF8ZpRYsGwWXMe0llY2iMRyRokDT7YzQzhvGSVmv1kFVsUmYCUxsX0VI4gt LBApMfFiA9gOEQEHiSk3JoDVMAssYJTYcsgZxOYVsJN4/WAfM4jNIqAicfjpEzBbVCBC4tOD w6wQNYISJ2c+YQGxOQU8JXb0XmaCmCMucevJfChbXmL72zlgB0kIrGOXOHnzFdAyDiDHReLq IimID4QlXh3fAvWNjMT/nfOZoOoZJf52vIBq3s4osXzyPzaIKmuJO+d+sYEMYhbQlFi/Sx8i 7Chx5sJLJoj5fBI33gpC3MAnMWnbdGaIMK9ER5sQRLWaxIZlG9hg1nbtXMk8gVFpFpLPZiH5 ZhaSb2Yh7F3AyLKKUTy1tDg3PbXYMC+1XK84Mbe4NC9dLzk/dxMjMPGc/nf80w7Gr5eSDjEK cDAq8fAaGF6LE2JNLCuuzD3EKMHBrCTC63T2dJwQb0piZVVqUX58UWlOavEhRmkOFiVxXuNF L2OFBNITS1KzU1MLUotgskwcnFINjOZbnVs5tvH0Wtg0MMnebKmUXvpTQDREqn1pI9c/SaPe 2/Lzi4N1jiUaHZzxvJXvYG/dS1bDq5W2O/MmdWwOiVvzuK9kR60bsylrQP73uRpyLCJZrEqG V/g23Xl8o9a45t9nRx3PHVNZjPqcZyQJ837c5vSnOvlywqQ7D0XZ96mfWXz+laKrEktxRqKh FnNRcSIAnQ8NdjgDAAA= X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFtrJIsWRmVeSWpSXmKPExsVy+t/xu7q5ttfiDHbcM7O48vU9m8WzW3uZ LE70fWC1uLxrDpvF8idrmS3O/z3O6sDmcehwB6PH/e7jTB4nW7+xeHzeJOex6clbpgDWKD2b ovzSklSFjPziElulaEMLIz1DSws9IxNLPUNj81grI1MlfTublNSczLLUIn27BL2MM8cPMhUs 4qyY/vsEawPjNvYuRk4OCQETiWktrWxdjFwcQgJLGSXuNc8CcjiAEjISx9eXQdQIS/y51gVV 85pR4t3v/YwgCTYBK4mJ7avAbGGBSIlXR6+xgNgiAg4SU25MYAdpYBZYwChxtu0gK0T3HEaJ Ha9ugHXwCthJvH6wjxnEZhFQkTj89AmYLSoQIXF4xyyoGkGJkzOfgE3lFPCU2NF7mQnEZhZQ l/gz7xIzhC0ucevJfKi4vMT2t3OYJzAKzULSPgtJyywkLbOQtCxgZFnFKJJaWpybnltsqFec mFtcmpeul5yfu4kRGG3bjv3cvIPx0sbgQ4wCHIxKPLwGhtfihFgTy4orcw8xSnAwK4nwOp09 HSfEm5JYWZValB9fVJqTWnyI0RTouYnMUqLJ+cBEkFcSb2hqaG5haWhubG5sZqEkztshcDBG SCA9sSQ1OzW1ILUIpo+Jg1OqgXHShuc1174L9Kw/2l2n2/OiSHFO45F1GVwCSU9z3u1f7Vf2 +bXp8wOqGxsP6OasTFmn9zP8zLnld14ev33p4qQ+nnnvLEKuV8w7z+q4KOz9olOPP1l9li/l vv/Y9NeaL9t23ohae+2YyB/7UwJzX7Wd9hB55VQaL7cm+vgZvVv8T3/d2epx+ceHcCWW4oxE Qy3mouJEANp7p+rMAgAA X-CMS-MailID: 20200602115213eucas1p2a9a2ebb2afccca0354cf196caf762c81 X-Msg-Generator: CA Content-Type: text/plain; charset="utf-8" X-RootMTR: 20200504232908eucas1p296927bc7c736ad924cefaea9a546459d X-EPHeader: CA CMS-TYPE: 201P X-CMS-RootMailID: 20200504232908eucas1p296927bc7c736ad924cefaea9a546459d References: Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Since we lack the hardware (or proper emulator setup) for testing needed changes add FIXMEs to document the issues (so at least they are not forgotten). Cc: Al Viro Cc: Geert Uytterhoeven Signed-off-by: Bartlomiej Zolnierkiewicz --- v2: - rebased on top of updated patch #1/2 drivers/video/fbdev/amifb.c | 2 ++ 1 file changed, 2 insertions(+) Index: b/drivers/video/fbdev/amifb.c =================================================================== --- a/drivers/video/fbdev/amifb.c +++ b/drivers/video/fbdev/amifb.c @@ -1892,6 +1892,7 @@ static int ami_get_var_cursorinfo(struct | ((datawords >> 15) & 1)); datawords <<= 1; #endif + /* FIXME: check the return value + test the change */ put_user(color, data++); } if (bits > 0) { @@ -1962,6 +1963,7 @@ static int ami_set_var_cursorinfo(struct bits = 16; words = delta; datawords = 0; for (width = (short)var->width - 1; width >= 0; width--) { unsigned long tdata = 0; + /* FIXME: check the return value + test the change */ get_user(tdata, data); data++; #ifdef __mc68000__