From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1753170AbcHUTgA (ORCPT <rfc822;w@1wt.eu>);
        Sun, 21 Aug 2016 15:36:00 -0400
Received: from mout.gmx.net ([212.227.15.15]:65403 "EHLO mout.gmx.net"
        rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
        id S1752017AbcHUTf7 (ORCPT <rfc822;linux-kernel@vger.kernel.org>);
        Sun, 21 Aug 2016 15:35:59 -0400
Subject: Re: [PATCH] drm/gma500: dont expose bytes from kernel stack
To: Joe Perches <joe@perches.com>,
        Patrik Jakobsson <patrik.r.jakobsson@gmail.com>
References: <1471804778-3656-1-git-send-email-xypron.glpk@gmx.de>
 <1471805161.3746.10.camel@perches.com>
Cc: David Airlie <airlied@linux.ie>, dri-devel@lists.freedesktop.org,
        linux-kernel@vger.kernel.org
From: Heinrich Schuchardt <xypron.glpk@gmx.de>
Message-ID: <acbed5ba-de02-17f4-c22e-0c3e36ca1c5e@gmx.de>
Date: Sun, 21 Aug 2016 21:35:40 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101
 Icedove/45.2.0
MIME-Version: 1.0
In-Reply-To: <1471805161.3746.10.camel@perches.com>
Content-Type: text/plain; charset=windows-1252
Content-Transfer-Encoding: 7bit
X-Provags-ID: V03:K0:M1mAu7xxF5g5I89wI6Ag4oQUJ8NBA4zbexao1UTM6JIAAIcQMpQ
 KgO6utAZZAyko0U9GF3hvDp0AAIXYZsGu6BepgXCLh5uPcWY0seUbrc6MTduSGmr795J/8/
 5UFixiBkhkhHxSy2GcaZd8OG0L749zafshNYooedowX8dTzslaK5sZWe1/CilY56pJbpMlD
 W5efpqiMT8xua79H78aWA==
X-UI-Out-Filterresults: notjunk:1;V01:K0:uX+vSVTAq+8=:qUiea5YVS0lsqUdiYaiMB9
 xJ7VoQMrKRzQTubCGaGrrtfd3eUDSz0ejgWrKguZxPrq5JF5hzC3VFuJTXgFWyQxwzUBaQecS
 H9qaV2+ssBZGhmwHv6bRpUjrHM74k2aeViDJsscpQdlAlMydC1AqsmZW95gFF2C/NKUTBRH+G
 pgS8FNBrJkV3FJXP5nBEDhTWAZyVbAzRJy7SG08zjOkTRkGBxPNm1MD0SK2ESlqUQM0b6jag7
 ooNCOb1u7+/4ShJaebtDXp8GuJVLwEeL0ADXnnUSkSJq6LaUZfrgZ1Qrc/6XKMkfsugxjhPp1
 sv7Nc7uUfFPAEE6gDC0Z+MsK1VxtRCYGuODAzjhxqQFAeIOn+KZG+EC3eu+R0ESf85UL+rjEr
 baXqDul/d/MqWRe4wtxFQ6aJOPwRXgI6B36UqhqXRboiXounZ3g59XA7okf0xmxxkGoIjMXyF
 WFc8dWIogv2VdbWRAmprpxckal34aMDg86UG11Q0K8zmpBmnepOn01UgAh/bR+B+VNwCnEt3L
 8ZQd5cbuLzuLDXCEmUZ2SuI1xjJhLCHhwnhczEngBsAj4UT2d8HKHN5dfoBazmyRcG758fi2e
 CNCfLIqtvOgeNFMx7rg+9dppjIntxt5os3ceIJEw5l5KkNlEvCKpzqB/SDwC2XBPFFDP4JCD+
 M+WF3/fwvlOwKlwfSER/s4XOYsJ5KQnb09Q51yPcZ8YNF+lWlvDRGkTyxJyJnDC18IPoDbKgv
 loTOJvHSQ1Zs65nsRzwyi6s7Dtq7GiENXkVDwWcrCcISVx15E5qqVuhR6IiMdLkCLFn+jI0Yy
 r9d/yKI
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 08/21/2016 08:46 PM, Joe Perches wrote:
> On Sun, 2016-08-21 at 20:39 +0200, Heinrich Schuchardt wrote:
>> Components m1, m2, p2, dot, vco of variable clock should be
>> initialized to avoid bytes from the kernel stack to be
>> exposed.
> 
> How was this found? visual code inspection?

cppcheck (http://cppcheck.sourceforge.net/)