RFR 8199154: Accessibility issues in jdk.security.auth
Weijun Wang
weijun.wang at oracle.com
Fri Mar 9 03:34:39 UTC 2018
> On Mar 9, 2018, at 11:30 AM, Valerie Peng <valerie.peng at oracle.com> wrote:
>
>
> Line 196 of LdapLoginModule.java seems to miss a </dd>?
Maybe this is one of those inner tags (DD always inside DL) that will be automatically closed when its outer tag is closed?
I'll add it anyway. Thanks.
--Max
> Rest of webrev look fine.
> Valerie
>
> On 3/6/2018 5:38 PM, Weijun Wang wrote:
>> Please take a review at
>>
>> http://cr.openjdk.java.net/~weijun/8199154/webrev.00/
>>
>> In LdapLoginModule.java, there seems to be an erroneous DL inside DL. I removed the outer DL and its incomplete DD. There is no visual difference after the change.
>>
>> In Krb5LoginModule.java, there were only DDs and no DT. I've simply removed all D? tags. The output is little more compact. See the attached screenshot.
>>
>>
>>
>> Thanks
>> Max
>
More information about the security-dev
mailing list