From: Seungwon Jeon <tgih.jun@samsung.com>
To: "'James Hogan'" <james.hogan@imgtec.com>, "'Chris Ball'" <cjb@laptop.org>,
        "'Will Newton'" <will.newton@imgtec.com>,
        "'Thomas Abraham'" <thomas.abraham@linaro.org>
Cc: "'Jaehoon Chung'" <jh80.chung@samsung.com>, linux-mmc@vger.kernel.org,
        linux-kernel@vger.kernel.org
References: <1350376988-27477-1-git-send-email-james.hogan@imgtec.com>
In-reply-to: <1350376988-27477-1-git-send-email-james.hogan@imgtec.com>
Subject: RE: [PATCH] dw_mmc: fix multiple drv_data NULL dereferences
Date: Tue, 16 Oct 2012 19:54:12 +0900
Message-id: <001701cdab8c$93e1eac0$bba5c040$%jun@samsung.com>
MIME-version: 1.0
Content-type: text/plain; charset=ks_c_5601-1987
Content-transfer-encoding: 7bit
Thread-index: Ac2rfJeFv+Up+NZOQYWdKkAhKIa+EQADeoAw
Content-language: ko
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 836
Lines: 23

On Tuesday, October 16, James Hogan <james.hogan@imgtec.com>
> Commit 800d78bfccb3d38116abfda2a5b9c8afdbd5ea21 ("mmc: dw_mmc: add
> support for implementation specific callbacks") merged in v3.7-rc1.
> 
> The above commit introduced multiple NULL pointer dereferences when
> the default dw_mci_pltfm_probe() is used, as it sets host->drv_data to
> NULL, and that's only checked against NULL in 1 out of the 7 cases where
> it is dereferenced.
> 
> Signed-off-by: James Hogan <james.hogan@imgtec.com>

Acked-by: Seungwon Jeon <tgih.jun@samsung.com>

Looks good to me.
Thanks.

Seungwon Jeon

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/