Bug 4613 - Memory Window: Invalid addresses (outside of any accessible segment) cause a hang
Summary: Memory Window: Invalid addresses (outside of any accessible segment) cause a...
Status: RESOLVED FIXED
Alias: None
Product: frysk
Classification: Unclassified
Component: general (show other bugs)
Version: unspecified
: P2 critical
Target Milestone: ---
Assignee: Kris Van Hees
URL:
Keywords:
Depends on:
Blocks: 3119 3200
  Show dependency treegraph
 
Reported: 2007-06-11 18:22 UTC by Kris Van Hees
Modified: 2007-09-07 06:49 UTC (History)
0 users

See Also:
Host:
Target:
Build:
Last reconfirmed:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Kris Van Hees 2007-06-11 18:22:12 UTC
The spin buttons that define the address range in the memory window have
statically defined lower and upper bounds that are not at all related to the
actual valid address range that applies to the application being debugged.  As
such, it is possible to attempt to access invalid memory locations.
Comment 1 Andrew Cagney 2007-06-20 17:34:40 UTC
[patch changed from: Memory Window: Spin buttons do not enforce correct address
limits; to Memory Window: Spin buttons do not recover from incorrect address limits]

The user, using the spin button can set any limit; for an invalid address,
things need to not hang (which is apparently the case).

For instance, specifying a lower bound of 0 should get "invalid" indications,
and not a crash.
Comment 2 Kris Van Hees 2007-06-20 18:46:11 UTC
Changed the topic (as I planned on doing later today in view of the discussion
on the call 2 weeks ago and follow-up today) to reflect the real problem.  It
essentially has nothing to do with the spin buttons themselves if we allow the
user to use any arbitrary address.

The real problem is that invalid addresses should be handled gracefully rather
than causing a hang.
Comment 3 Zhao Shujing 2007-09-07 06:48:25 UTC
7-09-07  Zhao Shujing <pearly.zhao@oracle.com>

        * memroy/MemoryWindow.java: When right-click up or down spin
        button, it would be ignored. When access invalid memory address,
        the requirement would be ignored. Fixes #4621 #4622 and # 4613
        * disassembler/DisassemblyWindow.java: Ditto
Comment 4 Zhao Shujing 2007-09-07 06:49:41 UTC
resolved