Hmm, good question.
When I checked JNI spec, it turns out that it didn't list any JNI exception under JNIEnv->GetStringUTFChars(...). Since this JNI method Java_sun_security_smartcardio_PlatformPCSC_initialize(..) does not return any object, perhaps we should throw some exception here to indicate a failure despite that it's very unlikely to happen in reality.

I think either NullPointerException or IOException will do here. Do you have a preference?
Valerie

On 02/14/14 18:01, Xuelei Fan wrote:
Looks fine to me.

Just want to make sure, no exception should be thrown if libName is
null, right?

   92     if (libName == NULL) {
   93         return;
   94     }

Xuelei

On 2/14/2014 7:29 AM, Valerie (Yu-Ching) Peng wrote:
Can someone please review the fixes which checks for pending exceptions
in native code "pcsc_md.c"?
The fix is trivial scope-wise.

Webrev: http://cr.openjdk.java.net/~valeriep/8033571/webrev.00/

Thanks,
Valerie

Reply via email to