From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chen Gang Subject: [PATCH] arch: frv: kernel: using vsnprintf() instead of vsprintf() Date: Mon, 27 May 2013 13:46:42 +0800 Message-ID: <51A2F342.8060907@asianux.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: Received: from intranet.asianux.com ([58.214.24.6]:46326 "EHLO intranet.asianux.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752281Ab3E0Frd (ORCPT ); Mon, 27 May 2013 01:47:33 -0400 Sender: linux-arch-owner@vger.kernel.org List-ID: To: "dhowells@redhat.com" , jesper.nilsson@axis.com, Richard Kuo , Tejun Heo Cc: Andrew Morton , David Miller , "linux-kernel@vger.kernel.org" , Linux-Arch Since die_if_kernel() is an extern common used function, better always check the buffer length to avoid memory overflow by a long 'str'. Signed-off-by: Chen Gang --- arch/frv/kernel/traps.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/arch/frv/kernel/traps.c b/arch/frv/kernel/traps.c index 4bff48c..a6d105d 100644 --- a/arch/frv/kernel/traps.c +++ b/arch/frv/kernel/traps.c @@ -523,7 +523,7 @@ void die_if_kernel(const char *str, ...) return; va_start(va, str); - vsprintf(buffer, str, va); + vsnprintf(buffer, sizeof(buffer), str, va); va_end(va); console_verbose(); -- 1.7.7.6