Assertion failure in debugging code

[Carl Worth]

Argh. I hadn't fully thought through all of this. So my reply above was
incorrect. Or rather, I correctly described what was happening, but
didn't correctly find the bug.

Carl Worth <cworth at cworth.org> writes:
>> It looks like there is a bug in either of these functions calls
>>
>>     RegionMap::iterator start = lowerBound(address);
>>     RegionMap::iterator stop = upperBound(address + size);
>
> OK. I looked at those carefully for any bugs, but they actually look
> correct to me.

I still believe the function bodies are correct for both of the above
functions.

The bug is that the call to upperBound should use the last valid address
for the range (which is "address + size - 1", not "address + size").

I'll follow-up (one last time) with an actual patch.

-Carl

-- 
carl.d.worth at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/apitrace/attachments/20120824/bdad2dde/attachment.pgp>