[Bug gdb/15161] symfile.c:struct load_section_data::load_offset is "unsigned long" but should be wider.

["cvs-commit at gcc dot gnu.org" <sourceware-bugzilla@sourceware.org>]

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

--- Comment #4 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> 2013-02-19 20:53:04 UTC ---
CVSROOT:    /cvs/src
Module name:    src
Changes by:    palves@sourceware.org    2013-02-19 20:52:58

Modified files:
    gdb/gdbserver  : ChangeLog server.c 

Log message:
    gdbserver:server.c - use unpack_varlen_hex to extract hex numbers.

    Addresses, as most numbers in the RSP are hex encoded, with variable
    length (that just means the width isn't specified, and there's no top
    cap.  So they should be extracted with unpack_varlen_hex.

    A couple spots in server.c are using strto(u)l, which doesn't work on
    LLP64 targets.

    This patch fixes it.

    Tested on x86_64 Fedora 17.

    2013-02-19  Pedro Alves  <palves@redhat.com>
    Kai Tietz <ktietz@redhat.com>

    PR gdb/15161

    * server.c (handle_query) <CRC check>: Use unpack_varlen_hex
    instead of strtoul to extract address from packet.
    (process_serial_event) <'z'>: Likewise.

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gdb/gdbserver/ChangeLog.diff?cvsroot=src&r1=1.681&r2=1.682
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gdb/gdbserver/server.c.diff?cvsroot=src&r1=1.184&r2=1.185

-- 
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.