[Bug target/39064] New: libiberty md5.h needs uintptr_t

[Bug target/39064] New: libiberty md5.h needs uintptr_t

[sezeroz at gmail dot com]

md5.h of libiberty assumes unsigned long to be of equal size to the size
of a pointer, but that isn't true for all sys tems (ie. win64). this patch
is a workaround for that, probably ugly, but md5.h should include stdint.h
not for _LIBC only..  here is the generated warning:
../../../gcc-svn/libiberty/md5.c: In function 'md5_process_bytes':
../../../gcc-svn/libiberty/md5.c:234: warning: cast from pointer to integer of
different size

(the patch should also be applied to binutils and gdb, or whoever else uses
libiberty)


-- 
           Summary: libiberty md5.h needs uintptr_t
           Product: gcc
           Version: 4.4.0
            Status: UNCONFIRMED
          Severity: trivial
          Priority: P3
         Component: target
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: sezeroz at gmail dot com
GCC target triplet: x86_64-pc-mingw32


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=39064

[Bug target/39064] libiberty md5.h needs uintptr_t

[sezeroz at gmail dot com]

------- Comment #1 from sezeroz at gmail dot com  2009-02-01 16:57 -------
Created an attachment (id=17223)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=17223&action=view)
libiberty md5.h win64 patch


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=39064

[Bug target/39064] libiberty md5.h needs uintptr_t

[ktietz at gcc dot gnu dot org]

-- 

ktietz at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ktietz at gcc dot gnu dot
                   |                            |org
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2009-02-21 09:53:55
               date|                            |


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=39064

[Bug target/39064] libiberty md5.h needs uintptr_t

[nightstrike at gmail dot com]

------- Comment #2 from nightstrike at gmail dot com  2009-03-14 15:46 -------
http://gcc.gnu.org/ml/gcc-patches/2009-03/msg00638.html


-- 

nightstrike at gmail dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |nightstrike at gmail dot com


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=39064

[Bug target/39064] libiberty md5.h needs uintptr_t

[sezeroz at gmail dot com]

------- Comment #3 from sezeroz at gmail dot com  2009-09-03 10:10 -------
The discussion thread at
http://gcc.gnu.org/ml/gcc-patches/2009-03/msg00778.html still haven't provide a
solution for this one even after the recent build system changes..  Will md5.h,
splay-tree.h and sha1.h be modified to use autotools support, or, should the
suggested patch be applied to md5.h?


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=39064

[Bug target/39064] libiberty md5.h needs uintptr_t

[ktietz at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=39064

Kai Tietz <ktietz at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--- Comment #8 from Kai Tietz <ktietz at gcc dot gnu.org> 2013-01-30 17:00:12 UTC ---
Fixed

[Bug target/39064] libiberty md5.h needs uintptr_t

[ktietz at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=39064

--- Comment #7 from Kai Tietz <ktietz at gcc dot gnu.org> 2013-01-30 16:58:22 UTC ---
Author: ktietz
Date: Wed Jan 30 16:58:10 2013
New Revision: 195581

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=195581
Log:
    PR other/54620
    PR target/39064
    * md5.h (md5_uintptr, md5_uint32): Define as uintptr_t/uint32_t if
    stdint.h and sys/types.h headers are present.
    * sha1.h (sha1_uintptr, sha1_uint32): Likewise.


Modified:
    branches/gcc-4_6-branch/include/ChangeLog
    branches/gcc-4_6-branch/include/md5.h
    branches/gcc-4_6-branch/include/sha1.h

[Bug target/39064] libiberty md5.h needs uintptr_t

[ktietz at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=39064

--- Comment #6 from Kai Tietz <ktietz at gcc dot gnu.org> 2013-01-30 16:56:50 UTC ---
Author: ktietz
Date: Wed Jan 30 16:56:36 2013
New Revision: 195580

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=195580
Log:
2013-01-30  Kai Tietz  <ktietz@redhat.com>

    PR other/54620
    PR target/39064
    * md5.h (md5_uintptr, md5_uint32): Define as uintptr_t/uint32_t if
    stdint.h and sys/types.h headers are present.
    * sha1.h (sha1_uintptr, sha1_uint32): Likewise.


Modified:
    branches/gcc-4_7-branch/include/ChangeLog
    branches/gcc-4_7-branch/include/md5.h
    branches/gcc-4_7-branch/include/sha1.h

[Bug target/39064] libiberty md5.h needs uintptr_t

[ktietz at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=39064

--- Comment #5 from Kai Tietz <ktietz at gcc dot gnu.org> 2013-01-30 16:51:03 UTC ---
Author: ktietz
Date: Wed Jan 30 16:50:49 2013
New Revision: 195579

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=195579
Log:
    PR other/54620
    PR target/39064
    * md5.h (md5_uintptr, md5_uint32): Define as uintptr_t/uint32_t if
    stdint.h and sys/types.h headers are present.
    * sha1.h (sha1_uintptr, sha1_uint32): Likewise.


Modified:
    trunk/include/ChangeLog
    trunk/include/md5.h
    trunk/include/sha1.h

[Bug target/39064] libiberty md5.h needs uintptr_t

[nightstrike at gmail dot com]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=39064

--- Comment #4 from nightstrike <nightstrike at gmail dot com> 2010-11-17 13:13:44 UTC ---
*** Bug 46464 has been marked as a duplicate of this bug. ***