Re: RFR: 8207025: JvmtiEnv::SetSystemProperty() does not handle OOM [v2]

2022-03-28 Thread Serguei Spitsyn
On Mon, 28 Mar 2022 22:06:12 GMT, David Holmes wrote: >> Ioi Lam has updated the pull request incrementally with one additional >> commit since the last revision: >> >> @dholmes-ora comments: changed implementation to work with >> JvmtiEnv::SetSystemProperty > > src/hotspot/share/prims/jvmti

Re: RFR: 8207025: JvmtiEnv::SetSystemProperty() does not handle OOM [v2]

2022-03-28 Thread David Holmes
On Mon, 28 Mar 2022 18:34:31 GMT, Ioi Lam wrote: >> `JvmtiEnv::SetSystemProperty` eventually calls `PathString::set_value` in >> arguments.cpp, which aborts the VM when it fails to allocate a string copy >> of the property value. >> >> >> bool PathString::set_value(const char *value) { >> i