RFR: 8253435: Cgroup: 'stomping of _mount_path' crash if manually mounted cpusets exist [v2]

[Volker Simonis]

On Thu, 24 Sep 2020 12:26:20 GMT, Severin Gehwolf <sgehwolf at openjdk.org> wrote:

>> Volker Simonis has refreshed the contents of this pull request, and previous commits have been removed. The incremental
>> views will show differences compared to the previous content of the PR.
>
> src/hotspot/os/linux/cgroupSubsystem_linux.cpp line 122:
> 
>> 120:   if (*mount_path == NULL) {
>> 121:     *mount_path = os::strdup(tmpmount);
>> 122:     if (strrchr(*mount_path, '/')) {
> 
> I guess we could avoid calling `strrchr` twice here by using a local variable. How about this?
> 
> char* last_slash = strrchr(*mount_path, '/');
> if (last_slash != NULL) {
>   *last_slash = '\0'; // truncate controller
> }

Changed as suggested.

-------------

PR: https://git.openjdk.java.net/jdk/pull/295