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: <20070220231549.GA15844@Krystal>
Date:	Tue, 20 Feb 2007 18:15:49 -0500
From:	Mathieu Desnoyers <mathieu.desnoyers@...ymtl.ca>
To:	Adrian Bunk <bunk@...sta.de>
Cc:	Andrew Morton <akpm@...ux-foundation.org>,
	linux-kernel@...r.kernel.org
Subject: Re: [-mm patch] marker exports must be EXPORT_SYMBOL_GPL

* Adrian Bunk (bunk@...sta.de) wrote:
> On Thu, Feb 15, 2007 at 05:14:08AM -0800, Andrew Morton wrote:
> >...
> > - Added the Linux Kernel Markers code.  No idea how to use it and it
> >   seems we're not to be told.
> >...
> > Changes since 2.6.20-rc6-mm3:
> >...
> > +linux-kernel-markers-kconfig-menus.patch
> > +linux-kernel-markers-architecture-independant-code.patch
> > +linux-kernel-markers-powerpc-optimization.patch
> > +linux-kernel-markers-i386-optimization.patch
> > +linux-kernel-markers-non-optimized-architectures.patch
> > 
> >  Linux Kenrel Markers
> >...
> 
> 
> This is definitely functionality illegal modules have no business to 
> use.
> 
> Signed-off-by: Adrian Bunk <bunk@...sta.de>

Acked-by: Mathieu Desnoyers <mathieu.desnoyers@...ymtl.ca>

> 
> ---
> 
>  arch/i386/kernel/marker.c |    2 +-
>  kernel/module.c           |   10 +++++-----
>  2 files changed, 6 insertions(+), 6 deletions(-)
> 
> --- linux-2.6.20-mm1/kernel/module.c.old	2007-02-15 16:02:09.000000000 +0100
> +++ linux-2.6.20-mm1/kernel/module.c	2007-02-15 16:03:08.000000000 +0100
> @@ -304,7 +304,7 @@
>  void __mark_empty_function(const char *fmt, ...)
>  {
>  }
> -EXPORT_SYMBOL(__mark_empty_function);
> +EXPORT_SYMBOL_GPL(__mark_empty_function);
>  
>  #ifdef MARK_POLYMORPHIC
>  static int marker_set_ins_enable(void *address, char enable)
> @@ -461,7 +461,7 @@
>  	mutex_unlock(&module_mutex);
>  	return found;
>  }
> -EXPORT_SYMBOL(marker_set_probe);
> +EXPORT_SYMBOL_GPL(marker_set_probe);
>  
>  int marker_remove_probe(marker_probe_func *probe)
>  {
> @@ -481,7 +481,7 @@
>  	mutex_unlock(&module_mutex);
>  	return found;
>  }
> -EXPORT_SYMBOL(marker_remove_probe);
> +EXPORT_SYMBOL_GPL(marker_remove_probe);
>  
>  int marker_list_probe(marker_probe_func *probe)
>  {
> @@ -505,7 +505,7 @@
>  	mutex_unlock(&module_mutex);
>  	return found;
>  }
> -EXPORT_SYMBOL(marker_list_probe);
> +EXPORT_SYMBOL_GPL(marker_list_probe);
>  #endif
>  
>  #ifdef CONFIG_SMP
> @@ -2510,7 +2510,7 @@
>  	}
>  	mutex_unlock(&module_mutex);
>  }
> -EXPORT_SYMBOL(list_modules);
> +EXPORT_SYMBOL_GPL(list_modules);
>  
>  /* Given an address, look for it in the module exception tables. */
>  const struct exception_table_entry *search_module_extables(unsigned long addr)
> --- linux-2.6.20-mm1/arch/i386/kernel/marker.c.old	2007-02-15 16:03:22.000000000 +0100
> +++ linux-2.6.20-mm1/arch/i386/kernel/marker.c	2007-02-15 16:03:34.000000000 +0100
> @@ -90,4 +90,4 @@
>  	flush_icache_range(address, size);
>  	return 0;
>  }
> -EXPORT_SYMBOL(arch_marker_set_ins_enable);
> +EXPORT_SYMBOL_GPL(arch_marker_set_ins_enable);
> 
> 
> 
> 
> 

-- 
Mathieu Desnoyers
Computer Engineering Ph.D. Candidate, Ecole Polytechnique de Montreal
OpenPGP key fingerprint: 8CD5 52C3 8E3C 4140 715F  BA06 3F25 A8FE 3BAE 9A68
-
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/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ