[8u-dev] Request for Approval: Backport of 8035653: InetAddress.getLocalHost crash

[Langer, Christoph]

Hi Alex,

I pushed it: http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/f08a4bccd723

Best regards
Christoph

> -----Original Message-----
> From: Langer, Christoph
> Sent: Mittwoch, 26. April 2017 08:42
> To: 'Alex Kashchenko' <akashche at redhat.com>
> Cc: jdk8u-dev at openjdk.java.net
> Subject: RE: [8u-dev] Request for Approval: Backport of 8035653:
> InetAddress.getLocalHost crash
> 
> Hi Alex,
> 
> I'll push this for you.
> 
> As it is a backport and does not contain additional changes, I'll keep the
> original author.
> 
> Best regards
> Christoph
> 
> > -----Original Message-----
> > From: jdk8u-dev [mailto:jdk8u-dev-bounces at openjdk.java.net] On Behalf
> > Of Alex Kashchenko
> > Sent: Dienstag, 25. April 2017 17:34
> > To: jdk8u-dev at openjdk.java.net
> > Subject: Re: [8u-dev] Request for Approval: Backport of 8035653:
> > InetAddress.getLocalHost crash
> >
> > Hi,
> >
> > On 04/25/2017 03:48 PM, Rob McKenna wrote:
> > > Approved
> >
> > The sponsor is required to prepare the changeset (I am not an Author in
> > jdk8u) and push it. It is not clear whether "Contributed-by:" is added
> > to backports, if it is - may I ask the sponsor to include
> > mvala at redhat.com there. Thanks!
> >
> > >
> > >     -Rob
> > >
> > > On 25/04/17 03:16, Alex Kashchenko wrote:
> > >> Hi,
> > >>
> > >> Could you please approve a backport of 8035653 to jdk8u-dev?
> > >>
> > >> webrev:
> http://cr.openjdk.java.net/~akasko/jdk8u/8035653/webrev.00/
> > >> bug: https://bugs.openjdk.java.net/browse/JDK-8035653
> > >> jdk9 changeset:
> > http://hg.openjdk.java.net/jdk9/dev/jdk/rev/8e5e92e0530e
> > >> original review thread:
> > >> http://mail.openjdk.java.net/pipermail/net-dev/2014-
> > February/008276.html
> > >> backport review thread:
> > >> http://mail.openjdk.java.net/pipermail/net-dev/2017-
> April/010729.html
> > >>
> > >> Patch from jdk9/dev does not apply cleanly, change to
> > >> DualStackPlainDatagramSocketImpl.java was already added to jdk8 as
> part
> > of
> > >> 8072466 [1][2].
> > >>
> > >> This patch brings changes to DualStackPlainDatagramSocketImpl.c and
> also
> > >> includes B8035653.java test.
> > >>
> > >> [1] https://bugs.openjdk.java.net/browse/JDK-8072466
> > >> [2] http://hg.openjdk.java.net/jdk8u/jdk8u-
> > dev/jdk/rev/3623f1b29b58#l7.1
> > >>
> > >> --
> > >> -Alex
> >
> >
> > --
> > -Alex