From: Sylvain 'ythier' Hitier Subject: [radeon] radeon_kick_out_firmware_fb() can deref NULL Date: Fri, 14 Jan 2011 18:49:40 +0100 Message-ID: <20110114174940.GA9168@begonia> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline Sender: linux-kernel-owner@vger.kernel.org To: dri-devel@lists.freedesktop.org Cc: Benjamin Herrenschmidt , Daniel Vetter , Dave Airlie , linux-kernel@vger.kernel.org List-Id: dri-devel@lists.freedesktop.org Hello, In commit a56f7428d7534f162fbb089c5c79012bf38a7c29 [1], the result of alloc_apertures() function call is used without checking for a possible NULL result, which may happen, as seen in [2]. The latest commit [3] in the drm-fixes branch of the linux/kernel/git/airlied/drm-2.6.git repo still has the issue. [1] http://git.kernel.org/?p=linux/kernel/git/airlied/drm-2.6.git;a=commitdiff;h=a56f7428d7534f162fbb089c5c79012bf38a7c29 [2] http://git.kernel.org/?p=linux/kernel/git/airlied/drm-2.6.git;hb=refs/heads/drm-fixes;a=blob;f=include/linux/fb.h#l890 [3] http://git.kernel.org/?p=linux/kernel/git/airlied/drm-2.6.git;a=commit;h=d15eda5c6edff4987af6f4423af0bab0c3251e74 Regards, Sylvain "ythier" Hitier -- Free hugs for Free Software Fellows! Living in a Debian package => http://en.wikipedia.org/wiki/Apt,_Vaucluse