stan/jdk-24
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

8206470: Incorrect use of os::lasterror in ClassListParser

Browse Source 
The change is for future-proof the code in case errno gets overwritten inside the allocation logic.

Reviewed-by: dholmes
This commit is contained in:
Patricio Chilano Mateo 2018-07-13 13:58:17 -04:00 committed by Coleen Phillimore
parent b5f0945a72
commit 2b397c3c13
1 changed files with 2 additions and 3 deletions

5
src/hotspot/share/classfile/classListParser.cpp

@ -48,14 +48,13 @@ ClassListParser::ClassListParser(const char* file) {
_instance = this;
_classlist_file = file;
_file = fopen(file, "r");
_line_no = 0;
_interfaces = new (ResourceObj::C_HEAP, mtClass) GrowableArray<int>(10, true);
if (_file == NULL) {
char errmsg[JVM_MAXPATHLEN];
os::lasterror(errmsg, JVM_MAXPATHLEN);
vm_exit_during_initialization("Loading classlist failed", errmsg);
}
_line_no = 0;
_interfaces = new (ResourceObj::C_HEAP, mtClass) GrowableArray<int>(10, true);
}
ClassListParser::~ClassListParser() {