This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [RFC] Stabs parsing regression from GDB 6.6 to GDB 6.6.90


Daniel Jacobowitz wrote:
On Wed, Oct 03, 2007 at 11:41:56AM -0700, Joel Brobecker wrote:
I definitively think that this patch should go both into
head and branch, but it needs approval from someone else...
Unless someone like Elena who knows this code very well says it's safe,
I would prefer if the patch didn't get applied to the branch. I'm a bit
concerned about it given the few iterations that you both had to go
through in order to get it to that point. It just shows that it is
not obvious.


That's because the original code to read signed octals was never triggered from day 1. I suspect it was due to a bad/incomplete merge from whatever tree it came from, and a testcase that doesn't really trigger it.

I think it's a bad idea to ship a release of GDB with a new bug that
we know about, which wasn't present in the previous release.


Another option is to revert this on the branch:


2007-08-04 Michael Snyder

	* stabsread.c (read_huge_number): Attempt to compute value before
	values that it depends on.

Since nobody complained until then (except for Coverity).  The signed
octal parsing was dead code Before Michail's Patch (BMP), and is broken
after it.  BMP, the basic types happened to be recorded with the
correct bit sizes, but ADA ranges that have a negative edge (-50, 50)
would be misrecorded .  That was what the original code attempted to
fix, but I don't really know where it makes a difference.  Perhaps
a print command which sets a value whose type is covered by a
range, with an out-of-range value, will trigger a warning:

print var = -51
warning: -51 is out of range for type x [-50, 50].

If gdb doesn't do it yet, it could be useful anyway.

What I know of ADA is what one can learn from a 10 minutes HOWTO reading.

I read through the patch.  It seems fine to me, though I am mostly
trusting Pedro, Pierre, and their test cases.




Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]