From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756362AbXKEDYG (ORCPT ); Sun, 4 Nov 2007 22:24:06 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754283AbXKEDXx (ORCPT ); Sun, 4 Nov 2007 22:23:53 -0500 Received: from koto.vergenet.net ([210.128.90.7]:47153 "EHLO koto.vergenet.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753089AbXKEDXw (ORCPT ); Sun, 4 Nov 2007 22:23:52 -0500 Date: Mon, 5 Nov 2007 12:23:48 +0900 From: Simon Horman To: Li Zefan Cc: Andrew Morton , LKML , tony.luck@intel.com, linux-ia64@vger.kernel.org Subject: Re: [PATCH 2/6] IA64: fix memset size error Message-ID: <20071105032347.GB15262@verge.net.au> References: <472E7D34.1040509@cn.fujitsu.com> <472E7DDC.9050806@cn.fujitsu.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <472E7DDC.9050806@cn.fujitsu.com> User-Agent: mutt-ng/devel-r804 (Debian) Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Nov 05, 2007 at 10:20:12AM +0800, Li Zefan wrote: > Li Zefan wrote: > > The size arguments passing to memset is wrong. > > > > Signed-off-by Li Zefan This looks correct to me. Acked. > > > > --- > > drivers/video/ps3fb.c | 2 +- > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > Sorry, please ignore the wrong patch, and here is the right one: > > --- > arch/ia64/kernel/efi.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/arch/ia64/kernel/efi.c b/arch/ia64/kernel/efi.c > index 3f7ea13..0e4ef20 100644 > --- a/arch/ia64/kernel/efi.c > +++ b/arch/ia64/kernel/efi.c > @@ -218,7 +218,7 @@ efi_gettimeofday (struct timespec *ts) > { > efi_time_t tm; > > - memset(ts, 0, sizeof(ts)); > + memset(ts, 0, sizeof(*ts)); > if ((*efi.get_time)(&tm, NULL) != EFI_SUCCESS) > return; > > -- > 1.5.3.rc7 > > - > To unsubscribe from this list: send the line "unsubscribe linux-ia64" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Horms H: http://www.vergenet.net/~horms/ W: http://www.valinux.co.jp/en/