<div dir="ltr">Hi,<div><br></div><div>Should we not be also incrementing i to get the next Ace? Otherwise, we are always getting the same one and probably just waiting till the count is 0 which would terminate the loop (if that is what we want, why not just break?).</div><div><br></div><div>Thanks,</div><div>Jc</div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Oct 1, 2018 at 4:57 PM Jonathan Gibbons <<a href="mailto:jonathan.gibbons@oracle.com">jonathan.gibbons@oracle.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  
    
  
  <div text="#000000" bgcolor="#FFFFFF">
    <p>I'm guessing serviceability-dev is a better place for this
      message.</p>
    <p>-- Jon<br>
    </p>
    <div class="m_8090995368929075399moz-forward-container"><br>
    </div>
    <br>
    <div class="m_8090995368929075399moz-cite-prefix">On 10/1/18 5:44 AM, Ladislav Jahoda
      wrote:<br>
    </div>
    <blockquote type="cite">
      
      <div dir="ltr">
        <div style="FONT-SIZE:10pt;FONT-FAMILY:'Arial';COLOR:#000000">
          <div>Hello guys,</div>
          <div> </div>
          <div>the revokeall utility has an infinite loop in its
            revokeAll method. Please note in the loop (line #344 in
            revokeall.c):</div>
          <div> </div>
          <div>while (count > 0) {</div>
          <div> </div>
          <div>the condition (line #354):</div>
          <div> </div>
          <div>if (((ACCESS_ALLOWED_ACE *)ace)->Header.AceType !=
            ACCESS_ALLOWED_ACE_TYPE) {<br>
                 continue;</div>
          <div>}</div>
          <div> </div>
          <div>which loops forever if the condition is met. Note that
            the “count” counter needs to be decremented before the
            continue statement, that is:</div>
          <div> </div>
          <div>
            <div>if (((ACCESS_ALLOWED_ACE *)ace)->Header.AceType !=
              ACCESS_ALLOWED_ACE_TYPE) {</div>
            <div>     count--;<br>
                   continue;</div>
            <div>}</div>
            <div> </div>
            <div>I checked the OpenJDK 8, 9 and 10, the bug is in all
              versions.</div>
            <div> </div>
            <div>Thanks ahead for the fix,</div>
            <div>Lada Jahoda</div>
            <div> </div>
          </div>
          <div><br>
             </div>
          <div> </div>
        </div>
      </div>
    </blockquote>
    <br>
  </div>

</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><br></div>Thanks,<div>Jc</div></div></div>