linux-kernel - Re: [PATCH 4/7] modpost: factorize symbol pretty print in get_pretty

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20150318090846.GI19131@chrystal.uk.oracle.com>
Date:	Wed, 18 Mar 2015 10:08:47 +0100
From:	Quentin Casasnovas <quentin.casasnovas@...cle.com>
To:	Quentin Casasnovas <quentin.casasnovas@...cle.com>,
	Rusty Russell <rusty@...tcorp.com.au>,
	Michal Marek <mmarek@...e.cz>
Cc:	lkml <linux-kernel@...r.kernel.org>,
	Oleg Nesterov <oleg@...hat.com>,
	Borislav Petkov <bp@...en8.de>,
	Linus Torvalds <torvalds@...ux-foundation.org>
Subject: Re: [PATCH 4/7] modpost: factorize symbol pretty print in
 get_pretty_name().

Adding Rusty and Michal to CC.

On Tue, Mar 17, 2015 at 01:39:59PM +0100, Quentin Casasnovas wrote:
> Signed-off-by: Quentin Casasnovas <quentin.casasnovas@...cle.com>
> ---
>  scripts/mod/modpost.c | 23 ++++++++++++-----------
>  1 file changed, 12 insertions(+), 11 deletions(-)
> 
> diff --git a/scripts/mod/modpost.c b/scripts/mod/modpost.c
> index 0f48f8b..c69681e 100644
> --- a/scripts/mod/modpost.c
> +++ b/scripts/mod/modpost.c
> @@ -1270,6 +1270,15 @@ static void print_section_list(const char * const list[20])
>  	fprintf(stderr, "\n");
>  }
>  
> +static inline void get_pretty_name(int is_func, const char** name, const char** name_p)
> +{
> +	switch (is_func) {
> +	case 0:	*name = "variable"; *name_p = ""; break;
> +	case 1:	*name = "function"; *name_p = "()"; break;
> +	default: *name = "(unknown reference)"; *name_p = ""; break;
> +	}
> +}
> +
>  /*
>   * Print a warning about a section mismatch.
>   * Try to find symbols near it so user can find it.
> @@ -1289,21 +1298,13 @@ static void report_sec_mismatch(const char *modname,
>  	char *prl_from;
>  	char *prl_to;
>  
> -	switch (from_is_func) {
> -	case 0: from = "variable"; from_p = "";   break;
> -	case 1: from = "function"; from_p = "()"; break;
> -	default: from = "(unknown reference)"; from_p = ""; break;
> -	}
> -	switch (to_is_func) {
> -	case 0: to = "variable"; to_p = "";   break;
> -	case 1: to = "function"; to_p = "()"; break;
> -	default: to = "(unknown reference)"; to_p = ""; break;
> -	}
> -
>  	sec_mismatch_count++;
>  	if (!sec_mismatch_verbose)
>  		return;
>  
> +	get_pretty_name(from_is_func, &from, &from_p);
> +	get_pretty_name(to_is_func, &to, &to_p);
> +
>  	warn("%s(%s+0x%llx): Section mismatch in reference from the %s %s%s "
>  	     "to the %s %s:%s%s\n",
>  	     modname, fromsec, fromaddr, from, fromsym, from_p, to, tosec,
> -- 
> 2.0.5
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/