From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753869Ab0CYMwQ (ORCPT ); Thu, 25 Mar 2010 08:52:16 -0400 Received: from mail.parknet.co.jp ([210.171.160.6]:33441 "EHLO mail.parknet.co.jp" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752746Ab0CYMwP (ORCPT ); Thu, 25 Mar 2010 08:52:15 -0400 From: OGAWA Hirofumi To: Nikolaus Schulz Cc: Al Viro , Marton Balint , Alexey Dobriyan , Kevin Dankwardt , Christoph Hellwig , linux-kernel@vger.kernel.org, stable@kernel.org Subject: Re: [PATCH] fat: fix buffer overflow in vfat_create_shortname() References: <20100325035010.GA3242@penelope.zusammrottung.local> <8739zp6k9s.fsf@devron.myhome.or.jp> <20100325122419.GA8896@luigi.zusammrottung.local> <87y6hg60nw.fsf@devron.myhome.or.jp> Date: Thu, 25 Mar 2010 21:52:12 +0900 In-Reply-To: <87y6hg60nw.fsf@devron.myhome.or.jp> (OGAWA Hirofumi's message of "Thu, 25 Mar 2010 21:33:39 +0900") Message-ID: <87bpec4l8j.fsf@devron.myhome.or.jp> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/23.1.93 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org OGAWA Hirofumi writes: > Nikolaus Schulz writes: > >>> However, anyway, your patch is much better off than current one. So, >>> I'll apply this in the next merge window. >>> >>> Or should we apply this immediately? >>> >>> Thanks. >> >> Given that this fixes a stack corruption which triggers the gcc stack >> smashing protection and thus basically a crash, I vote for not >> postponing it, but applying it immediately. > > Um..., where is a stack corruption? sprintf() overflow? But, it's > actually snprintf(), not sprintf()... Whoops, it was really sprintf(). -- OGAWA Hirofumi