public inbox for gdb-prs@sourceware.org
help / color / mirror / Atom feed
* [Bug breakpoints/14145] New: BranchDest at arm-tdep.c overflowed
@ 2012-05-23 13:53 manjian2006 at gmail dot com
  2013-10-29 18:44 ` [Bug breakpoints/14145] " will.newton at gmail dot com
  0 siblings, 1 reply; 2+ messages in thread
From: manjian2006 at gmail dot com @ 2012-05-23 13:53 UTC (permalink / raw)
  To: gdb-prs

http://sourceware.org/bugzilla/show_bug.cgi?id=14145

             Bug #: 14145
           Summary: BranchDest at arm-tdep.c overflowed
           Product: gdb
           Version: 7.4
            Status: NEW
          Severity: critical
          Priority: P2
         Component: breakpoints
        AssignedTo: unassigned@sourceware.org
        ReportedBy: manjian2006@gmail.com
    Classification: Unclassified


BranchDest was intended to find the branch address from the instruction
provided.But its define:

#define BranchDest(addr,instr) \
  ((CORE_ADDR) (((long) (addr)) + 8 + (sbits (instr, 0, 23) << 2)))
tried to convert the 32-bit integer to 64-bit integer without considering the
overflow,which caused the result which has signed bit(e.g 0x859127d4) expanded
its higher bits to 0xffffffff (e.g. 0xffffffff859127d4).

The solution is to fix the define like this

  ((CORE_ADDR) (((CORE_ADDR) (addr)) + 8 + (sbits (instr, 0, 23) << 2)))

This bug might result in failing to execute the "next" command in the machine
which had no hardware debug registers.

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.


^ permalink raw reply	[flat|nested] 2+ messages in thread

* [Bug breakpoints/14145] BranchDest at arm-tdep.c overflowed
  2012-05-23 13:53 [Bug breakpoints/14145] New: BranchDest at arm-tdep.c overflowed manjian2006 at gmail dot com
@ 2013-10-29 18:44 ` will.newton at gmail dot com
  0 siblings, 0 replies; 2+ messages in thread
From: will.newton at gmail dot com @ 2013-10-29 18:44 UTC (permalink / raw)
  To: gdb-prs

https://sourceware.org/bugzilla/show_bug.cgi?id=14145

Will Newton <will.newton at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |will.newton at gmail dot com
         Resolution|---                         |FIXED

--- Comment #1 from Will Newton <will.newton at gmail dot com> ---

This issue was fixed in commit 9991b20795a23272b07dd2214aa3ee0196b61f0b.

-- 
You are receiving this mail because:
You are on the CC list for the bug.


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2013-10-29 18:44 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-05-23 13:53 [Bug breakpoints/14145] New: BranchDest at arm-tdep.c overflowed manjian2006 at gmail dot com
2013-10-29 18:44 ` [Bug breakpoints/14145] " will.newton at gmail dot com

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 read-only IMAP folder(s) and NNTP newsgroup(s).