RFR(xxs): 8139037: [aix] Crash in ResolverConfigurationImpl.c - pointer shearing

Thomas Stüfe thomas.stuefe at gmail.com
Tue Oct 13 08:52:14 UTC 2015

Still need a second reviewer and a sponsor... please.


On Wed, Oct 7, 2015 at 4:28 PM, Thomas Stüfe <thomas.stuefe at gmail.com>

> Hi,
> please review and sponsor this AIX fix.
> in ResolverConfigurationImpl.c, we may crash because strchr() is used
> without prototype, which causes the C compiler to default to an int return
> type, which means we loose the upper 32bit of the pointer returned by
> strchr().
> The solution is to include string.h.
> Before, string.h was conditionally included for all platforms but AIX -
> reason is not really clear to me. But as strchr() is POSIX and always in
> string.h, I included string.h unconditionally for all Unices.
> webrev:
> http://cr.openjdk.java.net/~stuefe/webrevs/8139037/webrev.00/webrev/
> bug: https://bugs.openjdk.java.net/browse/JDK-8139037
> Kind Regards, Thomas

More information about the core-libs-dev mailing list