Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: "J. Johnston" <jjohnstn@redhat.com>
To: gdb-patches@sources.redhat.com
Subject: RFA: dbxread patch
Date: Tue, 25 Feb 2003 00:01:00 -0000	[thread overview]
Message-ID: <3E5AB25A.30504@redhat.com> (raw)

[-- Attachment #1: Type: text/plain, Size: 862 bytes --]

The following patch fixes a problem when the stabs debug info
for a file does not contain sline entries.  This happens, for
example with the glibc syscall code which is generated assembler
code.  In process_one_symbol in dbxread.c, a line 0 is recorded for the
end of the function.  When there are no other line numbers, we end up finding this
line 0 as our alt_symtab in symtab:find_pc_sect_line() and we end up
subtracting one resulting in line -1.  This problem manifests itself
when we attempt to break at the generated glibc functions such as
kill.

The patch changes process_one_symbol() to only record line 0 if sline info
has been found for the function.

Ok to commit?

-- Jeff J.


2003-02-24  Jeff Johnston  <jjohnstn@redhat.com>

	* dbxread.c (process_one_symbol): Only record line 0 if one or more sline
	entries have been seen for the function.




[-- Attachment #2: dbxread.patch --]
[-- Type: text/plain, Size: 532 bytes --]

Index: dbxread.c
===================================================================
RCS file: /cvs/src/src/gdb/dbxread.c,v
retrieving revision 1.43
diff -u -r1.43 dbxread.c
--- dbxread.c	20 Feb 2003 18:31:14 -0000	1.43
+++ dbxread.c	24 Feb 2003 23:39:52 -0000
@@ -2831,7 +2831,9 @@
  	      break;
  	    }
 
-	  record_line (current_subfile, 0, last_function_start + valu);
+	  if (sline_found_in_function)
+	    record_line (current_subfile, 0, last_function_start + valu);
+
 	  within_function = 0;
 	  new = pop_context ();
 

             reply	other threads:[~2003-02-25  0:01 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-02-25  0:01 J. Johnston [this message]
2003-02-25 22:52 ` Jim Blandy
2003-02-25 23:17   ` J. Johnston
2003-02-26 21:12     ` Elena Zannoni
2003-02-26 21:46       ` J. Johnston

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=3E5AB25A.30504@redhat.com \
    --to=jjohnstn@redhat.com \
    --cc=gdb-patches@sources.redhat.com \
    /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
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox