RFR: 8230466: check malloc/calloc results in jdk.hotspot.agent

Baesken, Matthias matthias.baesken at sap.com
Tue Sep 3 07:42:55 UTC 2019


Hello, please review the following small fix .

In   jdk.hotspot.agent  native code (linux / macosx)   we miss to check the  result of malloc/calloc a few times .
This should be  adjusted.
Additionally  I added initialization  to the symtab  array  in  symtab.c   (by calling memset  to make sure we have a defined state )  .



One question (was not really sure about this one so I did not change it so far) :

http://cr.openjdk.java.net/~mbaesken/webrevs/8230466.0/src/jdk.hotspot.agent/macosx/native/libsaproc/symtab.c.frames.html

359 void destroy_symtab(symtab_t* symtab) {
360   if (!symtab) return;
361   free(symtab->strs);
362   free(symtab->symbols);
363   free(symtab);
364 }



Here we miss to close   symtab->hash_table   (opened by  dbopen) ,  is it needed  (haven't  used dbopen much - maybe someone can comment on this)?


bug/webrev :

https://bugs.openjdk.java.net/browse/JDK-8230466

http://cr.openjdk.java.net/~mbaesken/webrevs/8230466.0/


Thanks and best regards, Matthias


More information about the hotspot-dev mailing list