Code Review for hotspot

Prepared by:goetz on Mon May 9 12:21:06 CEST 2016
Workspace:/net/usr.work/d045726/oJ/8140594-covCo-hs-comp/hotspot
Compare against: http://hg.openjdk.java.net/jdk9/hs-comp/hotspot
Compare against version:11178
Summary of changes: 81 lines changed: 25 ins; 4 del; 52 mod; 49117 unchg
Changeset: hotspot.changeset
Author comments:
os_linux.cpp

- Buffer not null terminated

- size of pointer passed to jio_snprintf()

os_linux_x86.cpp

- wrong size passed, scaled automatically by C compiler.

dfa.cpp/formssel./cpp

- no real issues as this is a tool, but nevertheless...

c1_LinearScan.cpp

- Access with negative index if called with any_reg

- Coverity doesn't grok that two calls to base_counter(i) result in same value.

ciMethodBlocks.cpp

- Out of bounds access

classFileParser.cpp

- Don't access array at BAD_ALLOCATION_TYPE, oob.

- CHECK macro results in two statements, second is not in if body.

classLoader.cpp

- jio_snprintf does null termination. But it might return -1 if truncated, in this case array access at -1.

systemDictionary.cpp

- avoid %s in message text.

compileBroker.cpp

_last_method_compiled is an array, not a pointer.

compileLog.cpp

- buffer not null terminated

disassembler.cpp

- lib_offset might be -1.

logTagSet.cpp

- possible buffer overrun

constantPool.cpp

- flagts not initialized

generateOopMap.cpp

- Remaining fields not initialized.

block.cpp

- assertion assigns instead of comparing.

callnode.cpp

_bci not initialized.

graphKit.cpp

- assertion assings instead of comparing

matcher.cpp

- find_receiver might return any_reg

relocator.cpp

- delta might be -4 ... assert returns.

Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/os/linux/vm/os_linux.cpp 

rev 11179 : 8140594: Various minor code improvements (compiler)
Reviewed-by: thartmann
14 lines changed: 1 ins; 0 del; 13 mod; 6290 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/os_cpu/linux_x86/vm/os_linux_x86.cpp 

rev 11179 : 8140594: Various minor code improvements (compiler)
Reviewed-by: thartmann
1 line changed: 0 ins; 0 del; 1 mod; 944 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/adlc/formssel.cpp 

rev 11179 : 8140594: Various minor code improvements (compiler)
Reviewed-by: thartmann
7 lines changed: 1 ins; 0 del; 6 mod; 4269 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/c1/c1_LinearScan.cpp 

rev 11179 : 8140594: Various minor code improvements (compiler)
Reviewed-by: thartmann
5 lines changed: 2 ins; 0 del; 3 mod; 6775 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/ci/ciMethodBlocks.cpp 

rev 11179 : 8140594: Various minor code improvements (compiler)
Reviewed-by: thartmann
1 line changed: 0 ins; 0 del; 1 mod; 401 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/classfile/classFileParser.cpp 

rev 11179 : 8140594: Various minor code improvements (compiler)
Reviewed-by: thartmann
7 lines changed: 3 ins; 0 del; 4 mod; 5899 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/classfile/classLoader.cpp 

rev 11179 : 8140594: Various minor code improvements (compiler)
Reviewed-by: thartmann
5 lines changed: 0 ins; 3 del; 2 mod; 1744 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/classfile/systemDictionary.cpp 

rev 11179 : 8140594: Various minor code improvements (compiler)
Reviewed-by: thartmann
3 lines changed: 0 ins; 0 del; 3 mod; 2912 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/compiler/compileBroker.cpp 

rev 11179 : 8140594: Various minor code improvements (compiler)
Reviewed-by: thartmann
4 lines changed: 0 ins; 1 del; 3 mod; 2404 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/compiler/compileLog.cpp 

rev 11179 : 8140594: Various minor code improvements (compiler)
Reviewed-by: thartmann
2 lines changed: 1 ins; 0 del; 1 mod; 344 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/compiler/disassembler.cpp 

rev 11179 : 8140594: Various minor code improvements (compiler)
Reviewed-by: thartmann
3 lines changed: 0 ins; 0 del; 3 mod; 592 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/logging/logTagSet.cpp 

rev 11179 : 8140594: Various minor code improvements (compiler)
Reviewed-by: thartmann
8 lines changed: 6 ins; 0 del; 2 mod; 119 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/oops/constantPool.cpp 

rev 11179 : 8140594: Various minor code improvements (compiler)
Reviewed-by: thartmann
2 lines changed: 1 ins; 0 del; 1 mod; 2083 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/oops/generateOopMap.cpp 

rev 11179 : 8140594: Various minor code improvements (compiler)
Reviewed-by: thartmann
8 lines changed: 6 ins; 0 del; 2 mod; 2565 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/opto/block.cpp 

rev 11179 : 8140594: Various minor code improvements (compiler)
Reviewed-by: thartmann
1 line changed: 0 ins; 0 del; 1 mod; 1777 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/opto/callnode.cpp 

rev 11179 : 8140594: Various minor code improvements (compiler)
Reviewed-by: thartmann
1 line changed: 1 ins; 0 del; 0 mod; 2040 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/opto/graphKit.cpp 

rev 11179 : 8140594: Various minor code improvements (compiler)
Reviewed-by: thartmann
1 line changed: 0 ins; 0 del; 1 mod; 4492 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/opto/matcher.cpp 

rev 11179 : 8140594: Various minor code improvements (compiler)
Reviewed-by: thartmann
6 lines changed: 3 ins; 0 del; 3 mod; 2691 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/runtime/relocator.cpp 

rev 11179 : 8140594: Various minor code improvements (compiler)
Reviewed-by: thartmann
2 lines changed: 0 ins; 0 del; 2 mod; 776 unchg

This code review page was prepared using /sapmnt/home/d045726/bin/webrev.ksh (vers 25.12-hg+openjdk.java.net).