<html>
  <head>
    <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <p>+1</p>
    <p>--Semyon<br>
    </p>
    <br>
    <div class="moz-cite-prefix">On 11/17/17 6:38 AM, Dmitry Markov
      wrote:<br>
    </div>
    <blockquote
      cite="mid:148EB85B-9B2E-40BE-A400-B3E0D3542906@oracle.com"
      type="cite">
      <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
      <font class="" face="Arial">Hello,<br class="">
        <br class="">
        Could you review a fix for jdk10, please?<br class="">
        <br class="">
         bug: <a moz-do-not-send="true"
          href="https://bugs.openjdk.java.net/browse/JDK-8154405"
          class="">https://bugs.openjdk.java.net/browse/JDK-8154405</a><br
          class="">
         webrev: <a moz-do-not-send="true"
          href="http://cr.openjdk.java.net/%7Edmarkov/8154405/webrev.00/"
          class="">http://cr.openjdk.java.net/~dmarkov/8154405/webrev.00/</a><br
          class="">
        <br class="">
      </font>
      <div style="margin: 0px; font-size: 13px; line-height: normal;
        background-color: rgb(255, 255, 255);" class=""><span
          style="font-kerning: none" class=""><font class=""
            face="Arial">Problem description:</font></span></div>
      <div style="margin: 0px; font-size: 13px; line-height: normal;
        background-color: rgb(255, 255, 255);" class=""><span
          style="font-kerning: none" class=""><font class=""
            face="Arial">The current implementation of
            Toolkit.getImage(URL url) and Toolkit.createImage(URL url)
            uses URLPermission to perform security checks. However the
            documentation says that security manager’s method
            checkPermission() is called with
            url.openConnection().getPermission() permission and
            url.openConnection().getPermission() returns
            SocketPermission. So there is an inconsistency between the
            documentation and the implementation.</font></span></div>
      <div style="margin: 0px; font-size: 13px; line-height: normal;
        background-color: rgb(255, 255, 255); min-height: 15px;"
        class=""><font class="" face="Arial"><span style="font-kerning:
            none" class=""></span><br class="">
        </font></div>
      <div style="margin: 0px; font-size: 13px; line-height: normal;
        background-color: rgb(255, 255, 255);" class=""><span
          style="font-kerning: none" class=""><font class=""
            face="Arial">Fix:</font></span></div>
      <div style="margin: 0px; font-size: 13px; line-height: normal;
        background-color: rgb(255, 255, 255);" class=""><span
          style="font-kerning: none" class=""><font class=""
            face="Arial">It is necessary to update the description of
            the methods to get rid of confusing statements.</font></span></div>
      <div style="margin: 0px; font-size: 13px; line-height: normal;
        background-color: rgb(255, 255, 255);" class=""><span
          style="font-kerning: none" class=""><font class=""
            face="Arial"><br class="">
          </font></span></div>
      <font class="" face="Arial">Thanks,<br class="">
        Dmitry</font>
    </blockquote>
    <br>
  </body>
</html>