Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757192Ab1F2Pk1 (ORCPT ); Wed, 29 Jun 2011 11:40:27 -0400 Received: from mail-ew0-f46.google.com ([209.85.215.46]:51698 "EHLO mail-ew0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756620Ab1F2Ph4 convert rfc822-to-8bit (ORCPT ); Wed, 29 Jun 2011 11:37:56 -0400 MIME-Version: 1.0 In-Reply-To: <1309212931-19772-1-git-send-email-richard@nod.at> References: <1309212931-19772-1-git-send-email-richard@nod.at> Date: Wed, 29 Jun 2011 18:37:54 +0300 Message-ID: Subject: Re: [PATCH 1/4] um: Fix _FORTIFY_SOURCE=2 support for kernel modules From: Vitaliy Ivanov To: Richard Weinberger Cc: akpm@linux-foundation.org, linux-kernel@vger.kernel.org, user-mode-linux-devel@lists.sourceforge.net Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1374 Lines: 37 On Tue, Jun 28, 2011 at 1:15 AM, Richard Weinberger wrote: > When UML is compiled with _FORTIFY_SOURCE we have to > export all _chk() functions which are used in modules. > For now it's only the case for __sprintf_chk(). > > Tested-by: Florian Fainelli > Reported-by: Florian Fainelli > Signed-off-by: Richard Weinberger > --- > ?arch/um/os-Linux/user_syms.c | ? ?5 +++++ > ?1 files changed, 5 insertions(+), 0 deletions(-) > > diff --git a/arch/um/os-Linux/user_syms.c b/arch/um/os-Linux/user_syms.c > index 05f5ea8..45ffe46 100644 > --- a/arch/um/os-Linux/user_syms.c > +++ b/arch/um/os-Linux/user_syms.c > @@ -113,3 +113,8 @@ EXPORT_SYMBOL(__stack_smash_handler); > > ?extern long __guard __attribute__((weak)); > ?EXPORT_SYMBOL(__guard); > + > +#ifdef _FORTIFY_SOURCE > +extern int __sprintf_chk(char *str, int flag, size_t strlen, const char *format); > +EXPORT_SYMBOL(__sprintf_chk); > +#endif > -- Can you please clarify why it's needed? I use FORTIFY_SOURCES on Linus' tree UML on Ubuntu w/ no problems and w/o this patch... Thanks, Vitaliy -- 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/