Use existing utility to copy jstring

facebook · Nov 20, 2024 · a5cc7ab · a5cc7ab
1 parent 49f36bd
commit a5cc7ab
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/java/rocksjni/options.cc b/java/rocksjni/options.cc
@@ -2311,13 +2311,13 @@ void Java_org_rocksdb_Options_setDailyOffpeakTimeUTC(JNIEnv* env, jclass,
                                                      jlong jhandle,
                                                      jstring jtimeutc) {
   auto* opt = reinterpret_cast<ROCKSDB_NAMESPACE::Options*>(jhandle);
-  const jsize jtimesz = env->GetStringUTFLength(jtimeutc);
-  const char* timeutc = env->GetStringUTFChars(jtimeutc, nullptr);
-  if (env->ExceptionCheck()) {
+  jboolean has_exception;
+  auto timeutc =
+      ROCKSDB_NAMESPACE::JniUtil::copyStdString(env, jtimeutc, &has_exception);
+  if (has_exception == JNI_TRUE) {
     return;
   }
-  opt->daily_offpeak_time_utc = std::string(timeutc, jtimesz);
-  env->ReleaseStringUTFChars(jtimeutc, timeutc);
+  opt->daily_offpeak_time_utc = timeutc;
 }
 
 /*