Code Review Request for 7196009
Matthew Hall
mhall at mhcomputing.net
Fri Apr 19 06:38:50 UTC 2013
One question. Why does it parse these manually instead of using the Properties class to do it? That would work a lot better and reduce complexity and bugs.
--
Sent from my mobile device.
"Valerie (Yu-Ching) Peng" <valerie.peng at oracle.com> wrote:
>Max,
>
>Do you have time to review the following fix?
>7196009: SunPkcs11 provider fails to parse config path containing
>parenthesis
>
>I made the adjustments to support quoted strings for the library path
>settings in the PKCS11 provider configuration file.
>If the path contain parenthesis or spaces, then the value should be
>quoted, so the whole thing would be taken as is.
>
>Webrev: http://cr.openjdk.java.net/~valeriep/7196009/webrev.00/
>
>Thanks,
>Valerie
More information about the security-dev
mailing list