LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Ananth N Mavinakayanahalli <ananth@in.ibm.com>
To: Masami Hiramatsu <mhiramat@redhat.com>
Cc: LKML <linux-kernel@vger.kernel.org>,
	Andrew Morton <akpm@linux-foundation.org>,
	systemtap-ml <systemtap@sources.redhat.com>,
	Jim Keniston <jkenisto@us.ibm.com>
Subject: Re: [PATCH][2.6.25-rc2-mm1] fix a null pointer bug in register_kretprobe
Date: Tue, 4 Mar 2008 07:43:15 +0530	[thread overview]
Message-ID: <20080304021315.GA7126@in.ibm.com> (raw)
In-Reply-To: <47CC7B1C.1060305@redhat.com>

On Mon, Mar 03, 2008 at 05:26:36PM -0500, Masami Hiramatsu wrote:
> X-Enigmail-Version: 0.95.6
> Content-Type: text/plain; charset=ISO-8859-1
> Content-Transfer-Encoding: 7bit
> X-Scanned-By: MIMEDefang 2.58 on 172.16.52.254
> 
>  Fix a bug in regiseter_kretprobe() which does not check
>  rp->kp.symbol_name == NULL before calling kprobe_lookup_name.
> 
>  For maintainability, this introduces kprobe_addr helper function which
>  resolves addr field. It is used by register_kprobe and register_kretprobe.
> 
> Signed-off-by: Masami Hiramatsu <mhiramat@redhat.com>
> CC: Ananth N Mavinakayanahalli <ananth@in.ibm.com>
> CC: Jim Keniston <jkenisto@us.ibm.com>

Acked-by: Ananth N Mavinakayanahalli <ananth@in.ibm.com>

Thanks Masami!

> ---
>  kernel/kprobes.c |   43 ++++++++++++++++++++++++++-----------------
>  1 file changed, 26 insertions(+), 17 deletions(-)
> 
> Index: 2.6.25-rc2-mm1/kernel/kprobes.c
> ===================================================================
> --- 2.6.25-rc2-mm1.orig/kernel/kprobes.c	2008-02-18 11:37:18.000000000 -0500
> +++ 2.6.25-rc2-mm1/kernel/kprobes.c	2008-02-18 11:37:18.000000000 -0500
> @@ -498,27 +498,36 @@ static int __kprobes in_kprobes_function
>  	return 0;
>  }
> 
> +/*
> + * If we have a symbol_name argument, look it up and add the offset field
> + * to it. This way, we can specify a relative address to a symbol.
> + */
> +static kprobe_opcode_t __kprobes *kprobe_addr(struct kprobe *p)
> +{
> +	kprobe_opcode_t *addr = p->addr;
> +	if (p->symbol_name) {
> +		if (addr)
> +			return NULL;
> +		kprobe_lookup_name(p->symbol_name, addr);
> +	}
> +
> +	if (!addr)
> +		return NULL;
> +	return (kprobe_opcode_t *)(((char *)addr) + p->offset);
> +}
> +
>  static int __kprobes __register_kprobe(struct kprobe *p,
>  	unsigned long called_from)
>  {
>  	int ret = 0;
>  	struct kprobe *old_p;
>  	struct module *probed_mod;
> +	kprobe_opcode_t *addr;
> 
> -	/*
> -	 * If we have a symbol_name argument look it up,
> -	 * and add it to the address.  That way the addr
> -	 * field can either be global or relative to a symbol.
> -	 */
> -	if (p->symbol_name) {
> -		if (p->addr)
> -			return -EINVAL;
> -		kprobe_lookup_name(p->symbol_name, p->addr);
> -	}
> -
> -	if (!p->addr)
> +	addr = kprobe_addr(p);
> +	if (!addr)
>  		return -EINVAL;
> -	p->addr = (kprobe_opcode_t *)(((char *)p->addr)+ p->offset);
> +	p->addr = addr;
> 
>  	if (!kernel_text_address((unsigned long) p->addr) ||
>  	    in_kprobes_functions((unsigned long) p->addr))
> @@ -722,12 +731,12 @@ int __kprobes register_kretprobe(struct
>  	int ret = 0;
>  	struct kretprobe_instance *inst;
>  	int i;
> -	void *addr = rp->kp.addr;
> +	void *addr;
> 
>  	if (kretprobe_blacklist_size) {
> -		if (addr == NULL)
> -			kprobe_lookup_name(rp->kp.symbol_name, addr);
> -		addr += rp->kp.offset;
> +		addr = kprobe_addr(&rp->kp);
> +		if (!addr)
> +			return -EINVAL;
> 
>  		for (i = 0; kretprobe_blacklist[i].name != NULL; i++) {
>  			if (kretprobe_blacklist[i].addr == addr)
> -- 
> Masami Hiramatsu
> 
> Software Engineer
> Hitachi Computer Products (America) Inc.
> Software Solutions Division
> 
> e-mail: mhiramat@redhat.com
> 

      parent reply	other threads:[~2008-03-04  2:13 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-03-03 22:26 Masami Hiramatsu
2008-03-03 22:36 ` Andrew Morton
2008-03-03 22:59   ` Masami Hiramatsu
2008-03-04  2:13 ` Ananth N Mavinakayanahalli [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20080304021315.GA7126@in.ibm.com \
    --to=ananth@in.ibm.com \
    --cc=akpm@linux-foundation.org \
    --cc=jkenisto@us.ibm.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mhiramat@redhat.com \
    --cc=systemtap@sources.redhat.com \
    --subject='Re: [PATCH][2.6.25-rc2-mm1] fix a null pointer bug in register_kretprobe' \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).