|
|
|
FindBugs Bug Detector Report
|
The following document contains the results of FindBugs
|
FindBugs Version is 3.0.1
|
Threshold is medium
|
Effort is min
|
|
|
Files
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
org.apache.harmony.pack200.AttributeDefinitionBands
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Switch statement found in org.apache.harmony.pack200.AttributeDefinitionBands.addAttributeDefinitions(Map, int[], int) where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 212 | Medium |
| org.apache.harmony.pack200.AttributeDefinitionBands.addAttributeDefinitions(Map, int[], int) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 206 | Medium |
|
|
org.apache.harmony.pack200.BHSDCodec
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.pack200.BHSDCodec.encode(int, int) invokes inefficient new Byte(byte) constructor; use Byte.valueOf(byte) instead | PERFORMANCE | DM_NUMBER_CTOR | 367 | Medium |
|
|
org.apache.harmony.pack200.BandSet
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.pack200.BandSet.encodeWithPopulationCodec(String, int[], BHSDCodec, BandSet$BandData, BandSet$BandAnalysisResults) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 388 | Medium |
| Switch statement found in org.apache.harmony.pack200.BandSet.encodeWithPopulationCodec(String, int[], BHSDCodec, BandSet$BandData, BandSet$BandAnalysisResults) where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 438 | Medium |
| org.apache.harmony.pack200.BandSet.encodeWithPopulationCodec(String, int[], BHSDCodec, BandSet$BandData, BandSet$BandAnalysisResults) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 369 | Medium |
|
|
org.apache.harmony.pack200.BandSet$BandAnalysisResults
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Should org.apache.harmony.pack200.BandSet$BandAnalysisResults be a static inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 769 | Medium |
|
|
org.apache.harmony.pack200.BandSet$BandData
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| new org.apache.harmony.pack200.BandSet$BandData(BandSet, int[]) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 677 | Medium |
| new org.apache.harmony.pack200.BandSet$BandData(BandSet, int[]) may expose internal representation by storing an externally mutable object into BandSet$BandData.band | MALICIOUS_CODE | EI_EXPOSE_REP2 | 676 | Medium |
|
|
org.apache.harmony.pack200.BcBands
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.pack200.BcBands.getIndexInClassForConstructor(List) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 255 | Medium |
| org.apache.harmony.pack200.BcBands.visitEnd() invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 279 | Medium |
| org.apache.harmony.pack200.BcBands.visitLabel(Label) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 293 | Medium |
| Switch statement found in org.apache.harmony.pack200.BcBands.visitIntInsn(int, int) where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 368 | Medium |
| Switch statement found in org.apache.harmony.pack200.BcBands.visitVarInsn(int, int) where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 566 | Medium |
|
|
org.apache.harmony.pack200.CPClass
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.pack200.CPClass defines compareTo(Object) and uses Object.equals() | BAD_PRACTICE | EQ_COMPARETO_USE_OBJECT_EQUALS | 42 | Medium |
|
|
org.apache.harmony.pack200.CPDouble
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.pack200.CPDouble defines compareTo(Object) and uses Object.equals() | BAD_PRACTICE | EQ_COMPARETO_USE_OBJECT_EQUALS | 31 | Medium |
|
|
org.apache.harmony.pack200.CPFloat
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.pack200.CPFloat defines compareTo(Object) and uses Object.equals() | BAD_PRACTICE | EQ_COMPARETO_USE_OBJECT_EQUALS | 31 | Medium |
|
|
org.apache.harmony.pack200.CPInt
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.pack200.CPInt defines compareTo(Object) and uses Object.equals() | BAD_PRACTICE | EQ_COMPARETO_USE_OBJECT_EQUALS | 31 | Medium |
|
|
org.apache.harmony.pack200.CPLong
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.pack200.CPLong defines compareTo(Object) and uses Object.equals() | BAD_PRACTICE | EQ_COMPARETO_USE_OBJECT_EQUALS | 31 | Medium |
|
|
org.apache.harmony.pack200.CPMethodOrField
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.pack200.CPMethodOrField defines compareTo(Object) and uses Object.equals() | BAD_PRACTICE | EQ_COMPARETO_USE_OBJECT_EQUALS | 39 | Medium |
|
|
org.apache.harmony.pack200.CPNameAndType
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.pack200.CPNameAndType defines compareTo(Object) and uses Object.equals() | BAD_PRACTICE | EQ_COMPARETO_USE_OBJECT_EQUALS | 37 | Medium |
|
|
org.apache.harmony.pack200.CPSignature
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.pack200.CPSignature defines compareTo(Object) and uses Object.equals() | BAD_PRACTICE | EQ_COMPARETO_USE_OBJECT_EQUALS | 39 | Medium |
|
|
org.apache.harmony.pack200.CPString
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.pack200.CPString defines compareTo(Object) and uses Object.equals() | BAD_PRACTICE | EQ_COMPARETO_USE_OBJECT_EQUALS | 33 | Medium |
|
|
org.apache.harmony.pack200.CPUTF8
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.pack200.CPUTF8 defines compareTo(Object) and uses Object.equals() | BAD_PRACTICE | EQ_COMPARETO_USE_OBJECT_EQUALS | 31 | Medium |
|
|
org.apache.harmony.pack200.ClassBands
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Dead store to flags in org.apache.harmony.pack200.ClassBands.addClass(int, int, String, String, String, String[]) | STYLE | DLS_DEAD_LOCAL_STORE | 219 | High |
| org.apache.harmony.pack200.ClassBands.addAnnotation(int, String, boolean, List, List, List, List, List, List, List) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 1437 | Medium |
| org.apache.harmony.pack200.ClassBands.addAnnotationDefault(List, List, List, List, List, List, List) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 1476 | Medium |
| org.apache.harmony.pack200.ClassBands.addCode() invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 1201 | Medium |
| org.apache.harmony.pack200.ClassBands.addCodeAttribute(NewAttribute) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 1178 | Medium |
| org.apache.harmony.pack200.ClassBands.addField(int, String, String, String, Object) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 277 | Medium |
| org.apache.harmony.pack200.ClassBands.addFieldAttribute(NewAttribute) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 1148 | Medium |
| org.apache.harmony.pack200.ClassBands.addLineNumber(int, Label) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 1220 | Medium |
| org.apache.harmony.pack200.ClassBands.addLocalVariable(String, String, String, Label, Label, int) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 1236 | Medium |
| org.apache.harmony.pack200.ClassBands.addMaxStack(int, int) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 1188 | Medium |
| org.apache.harmony.pack200.ClassBands.addMethod(int, String, String, String, String[]) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 1001 | Medium |
| org.apache.harmony.pack200.ClassBands.addMethodAttribute(NewAttribute) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 1163 | Medium |
| org.apache.harmony.pack200.ClassBands.addParameterAnnotation(int, String, boolean, List, List, List, List, List, List, List) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 1361 | Medium |
| org.apache.harmony.pack200.ClassBands.endOfMethod() invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 1034 | Medium |
| org.apache.harmony.pack200.ClassBands.finaliseBands() invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 326 | Medium |
| org.apache.harmony.pack200.ClassBands.renumberBci(List, IntList, Map) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 1301 | Medium |
| org.apache.harmony.pack200.ClassBands.renumberDoubleOffsetBci(List, List, List, IntList, Map) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 1335 | Medium |
| org.apache.harmony.pack200.ClassBands.renumberOffsetBci(List, List, IntList, Map) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 1317 | Medium |
| Switch statement found in org.apache.harmony.pack200.ClassBands.addAnnotation(int, String, boolean, List, List, List, List, List, List, List) where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 1408 | Medium |
|
|
org.apache.harmony.pack200.CodecEncoding
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.pack200.CodecEncoding.getSpecifier(Codec, Codec) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 231 | Medium |
| Useless condition: it's known that value >= 117 ('u') at this point | STYLE | UC_USELESS_CONDITION | 158 | Medium |
| Useless condition: it's known that value >= 141 (0x8d) at this point | STYLE | UC_USELESS_CONDITION | 184 | Medium |
|
|
org.apache.harmony.pack200.CpBands
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.pack200.CpBands.addCharacters(List, char[]) invokes inefficient new Character(char) constructor; use Character.valueOf(char) instead | PERFORMANCE | DM_NUMBER_CTOR | 189 | Medium |
| org.apache.harmony.pack200.CpBands.addIndices() invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 440 | Medium |
| org.apache.harmony.pack200.CpBands.writeCpUtf8(OutputStream) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 134 | Medium |
|
|
org.apache.harmony.pack200.IcBands$IcTuple
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.pack200.IcBands$IcTuple defines equals and uses Object.hashCode() | BAD_PRACTICE | HE_EQUALS_USE_HASHCODE | 166 | Medium |
| Should org.apache.harmony.pack200.IcBands$IcTuple be a static inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 158 | Medium |
|
|
org.apache.harmony.pack200.NewAttribute
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Switch statement found in org.apache.harmony.pack200.NewAttribute.addContext(int) where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 60 | Medium |
|
|
org.apache.harmony.pack200.NewAttributeBands
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.pack200.NewAttributeBands.readNumber(StringReader) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 932 | Medium |
|
|
org.apache.harmony.pack200.NewAttributeBands$Integral
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.pack200.NewAttributeBands$Integral.addAttributeToBand(NewAttribute, InputStream) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 423 | Medium |
| org.apache.harmony.pack200.NewAttributeBands$Integral.renumberBci(IntList, Map) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 453 | Medium |
| org.apache.harmony.pack200.NewAttributeBands$Integral.renumberOffsetBci(List, IntList, Map) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 471 | Medium |
|
|
org.apache.harmony.pack200.NewAttributeBands$LayoutElement
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Switch statement found in org.apache.harmony.pack200.NewAttributeBands$LayoutElement.getLength(char) where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 339 | Medium |
|
|
org.apache.harmony.pack200.NewAttributeBands$Reference
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Read of unwritten field band in org.apache.harmony.pack200.NewAttributeBands$Reference.addAttributeToBand(NewAttribute, InputStream) | CORRECTNESS | NP_UNWRITTEN_FIELD | 671 | Medium |
| Unwritten field: org.apache.harmony.pack200.NewAttributeBands$Reference.band | CORRECTNESS | UWF_UNWRITTEN_FIELD | 671 | Medium |
|
|
org.apache.harmony.pack200.NewAttributeBands$UnionCase
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.pack200.NewAttributeBands$UnionCase.hasTag(long) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 778 | Medium |
|
|
org.apache.harmony.pack200.PackingOptions
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.pack200.PackingOptions.getUnknownAttributePrototypes() may expose internal representation by returning PackingOptions.unknownAttributeTypes | MALICIOUS_CODE | EI_EXPOSE_REP | 318 | Medium |
| org.apache.harmony.pack200.PackingOptions.addOrUpdateAttributeActions(List, Map, int) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 270 | Medium |
|
|
org.apache.harmony.pack200.PopulationCodec
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.pack200.PopulationCodec.getFavoured() may expose internal representation by returning PopulationCodec.favoured | MALICIOUS_CODE | EI_EXPOSE_REP | 132 | Medium |
|
|
org.apache.harmony.pack200.Segment$ArrayVisitor
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.pack200.Segment$ArrayVisitor.visit(String, Object) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 633 | Medium |
| org.apache.harmony.pack200.Segment$ArrayVisitor.visitArray(String) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 653 | Medium |
| org.apache.harmony.pack200.Segment$ArrayVisitor.visitEnum(String, String, String) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 664 | Medium |
|
|
org.apache.harmony.pack200.Segment$SegmentAnnotationVisitor
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.pack200.Segment$SegmentAnnotationVisitor.visitAnnotation(String, String) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 527 | Medium |
| org.apache.harmony.pack200.Segment$SegmentAnnotationVisitor.visitArray(String) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 538 | Medium |
|
|
org.apache.harmony.pack200.Segment$SegmentAnnotationVisitor$AnnotationVisitorImpl
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.pack200.Segment$SegmentAnnotationVisitor$AnnotationVisitorImpl.visit(String, Object) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 579 | Medium |
| org.apache.harmony.pack200.Segment$SegmentAnnotationVisitor$AnnotationVisitorImpl.visitEnum(String, String, String) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 604 | Medium |
|
|
org.apache.harmony.pack200.SegmentHeader
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unread field: org.apache.harmony.pack200.SegmentHeader.have_file_modtime; should this field be static? | PERFORMANCE | SS_SHOULD_BE_STATIC | 67 | Medium |
| Unread field: org.apache.harmony.pack200.SegmentHeader.have_file_options; should this field be static? | PERFORMANCE | SS_SHOULD_BE_STATIC | 68 | Medium |
|
|
org.apache.harmony.pack200.SegmentHeader$Counter
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Should org.apache.harmony.pack200.SegmentHeader$Counter be a static inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 300 | Medium |
|
|
org.apache.harmony.unpack200.AttributeLayoutMap
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.unpack200.AttributeLayoutMap.add(AttributeLayout) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 219 | Medium |
| org.apache.harmony.unpack200.AttributeLayoutMap.getAttributeLayout(int, int) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 242 | Medium |
|
|
org.apache.harmony.unpack200.BcBands
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.unpack200.BcBands.read(InputStream) invokes inefficient Boolean constructor; use Boolean.valueOf(...) instead | PERFORMANCE | DM_BOOLEAN_CTOR | 194 | Medium |
| org.apache.harmony.unpack200.BcBands.read(InputStream) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 259 | Medium |
| Unread field: org.apache.harmony.unpack200.BcBands.bcEscByte | PERFORMANCE | URF_UNREAD_FIELD | 357 | Medium |
| Unread field: org.apache.harmony.unpack200.BcBands.bcEscRef | PERFORMANCE | URF_UNREAD_FIELD | 352 | Medium |
| Unread field: org.apache.harmony.unpack200.BcBands.bcEscRefSize | PERFORMANCE | URF_UNREAD_FIELD | 354 | Medium |
|
|
org.apache.harmony.unpack200.ClassBands
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Bad comparison of nonnegative value with 0 in org.apache.harmony.unpack200.ClassBands.parseCodeBands(InputStream) | CORRECTNESS | INT_BAD_COMPARISON_WITH_NONNEGATIVE_VALUE | 762 | Medium |
| Switch statement found in org.apache.harmony.unpack200.ClassBands.parseMetadata(InputStream, String[], int[], int[], String) where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 1048 | Medium |
|
|
org.apache.harmony.unpack200.CpBands
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.unpack200.CpBands.cpIntegerValue(int) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 575 | Medium |
| org.apache.harmony.unpack200.CpBands.cpLongValue(int) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 565 | Medium |
| org.apache.harmony.unpack200.CpBands.parseCpClass(InputStream) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 160 | Medium |
| org.apache.harmony.unpack200.CpBands.parseCpDescriptor(InputStream) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 194 | Medium |
| org.apache.harmony.unpack200.CpBands.parseCpSignature(InputStream) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 375 | Medium |
| org.apache.harmony.unpack200.CpBands.parseCpUtf8(InputStream) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 413 | Medium |
| Useless object stored in variable list of method org.apache.harmony.unpack200.CpBands.parseCpSignature(InputStream) | STYLE | UC_USELESS_OBJECT | 363 | Medium |
|
|
org.apache.harmony.unpack200.IcBands
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.unpack200.IcBands.getIcTuples() may expose internal representation by returning IcBands.icAll | MALICIOUS_CODE | EI_EXPOSE_REP | 153 | Medium |
|
|
org.apache.harmony.unpack200.IcBands$1
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.unpack200.IcBands$1.compare(Object, Object) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 234 | Medium |
|
|
org.apache.harmony.unpack200.NewAttributeBands
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.unpack200.NewAttributeBands.readNumber(StringReader) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 998 | Medium |
|
|
org.apache.harmony.unpack200.NewAttributeBands$LayoutElement
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Switch statement found in org.apache.harmony.unpack200.NewAttributeBands$LayoutElement.getLength(char) where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 374 | Medium |
|
|
org.apache.harmony.unpack200.NewAttributeBands$UnionCase
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.unpack200.NewAttributeBands$UnionCase.hasTag(long) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 861 | Medium |
|
|
org.apache.harmony.unpack200.Segment
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Found reliance on default encoding in org.apache.harmony.unpack200.Segment.setLogStream(OutputStream): new java.io.PrintWriter(OutputStream) | I18N | DM_DEFAULT_ENCODING | 619 | High |
| org.apache.harmony.unpack200.Segment.unpackRead(InputStream) ignores result of java.io.InputStream.read(byte[]) | BAD_PRACTICE | RR_NOT_CHECKED | 497 | Medium |
|
|
org.apache.harmony.unpack200.SegmentConstantPoolArrayCache$CachedArray
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.unpack200.SegmentConstantPoolArrayCache$CachedArray.cacheIndexes() invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 178 | Medium |
| new org.apache.harmony.unpack200.SegmentConstantPoolArrayCache$CachedArray(SegmentConstantPoolArrayCache, String[]) may expose internal representation by storing an externally mutable object into SegmentConstantPoolArrayCache$CachedArray.primaryArray | MALICIOUS_CODE | EI_EXPOSE_REP2 | 126 | Medium |
| Should org.apache.harmony.unpack200.SegmentConstantPoolArrayCache$CachedArray be a static inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 124 | Medium |
|
|
org.apache.harmony.unpack200.SegmentHeader
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unread field: org.apache.harmony.unpack200.SegmentHeader.archiveMajor | PERFORMANCE | URF_UNREAD_FIELD | 161 | Medium |
| Unread field: org.apache.harmony.unpack200.SegmentHeader.archiveMinor | PERFORMANCE | URF_UNREAD_FIELD | 146 | Medium |
|
|
org.apache.harmony.unpack200.bytecode.AnnotationsAttribute$Annotation
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| new org.apache.harmony.unpack200.bytecode.AnnotationsAttribute$Annotation(int, CPUTF8, CPUTF8[], AnnotationsAttribute$ElementValue[]) may expose internal representation by storing an externally mutable object into AnnotationsAttribute$Annotation.element_names | MALICIOUS_CODE | EI_EXPOSE_REP2 | 47 | Medium |
| new org.apache.harmony.unpack200.bytecode.AnnotationsAttribute$Annotation(int, CPUTF8, CPUTF8[], AnnotationsAttribute$ElementValue[]) may expose internal representation by storing an externally mutable object into AnnotationsAttribute$Annotation.element_values | MALICIOUS_CODE | EI_EXPOSE_REP2 | 48 | Medium |
|
|
org.apache.harmony.unpack200.bytecode.ByteCode
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.unpack200.bytecode.ByteCode.getByteCodeTargets() may expose internal representation by returning ByteCode.byteCodeTargets | MALICIOUS_CODE | EI_EXPOSE_REP | 325 | Medium |
| org.apache.harmony.unpack200.bytecode.ByteCode.getNestedClassFileEntries() may expose internal representation by returning ByteCode.nested | MALICIOUS_CODE | EI_EXPOSE_REP | 96 | Medium |
| org.apache.harmony.unpack200.bytecode.ByteCode.getNestedPositions() may expose internal representation by returning ByteCode.nestedPositions | MALICIOUS_CODE | EI_EXPOSE_REP | 272 | Medium |
| org.apache.harmony.unpack200.bytecode.ByteCode.getRewrite() may expose internal representation by returning ByteCode.rewrite | MALICIOUS_CODE | EI_EXPOSE_REP | 364 | Medium |
| org.apache.harmony.unpack200.bytecode.ByteCode.setByteCodeTargets(int[]) may expose internal representation by storing an externally mutable object into ByteCode.byteCodeTargets | MALICIOUS_CODE | EI_EXPOSE_REP2 | 321 | Medium |
| org.apache.harmony.unpack200.bytecode.ByteCode.setNested(ClassFileEntry[]) may expose internal representation by storing an externally mutable object into ByteCode.nested | MALICIOUS_CODE | EI_EXPOSE_REP2 | 250 | Medium |
| org.apache.harmony.unpack200.bytecode.ByteCode.setNestedPositions(int[][]) may expose internal representation by storing an externally mutable object into ByteCode.nestedPositions | MALICIOUS_CODE | EI_EXPOSE_REP2 | 268 | Medium |
| org.apache.harmony.unpack200.bytecode.ByteCode.setRewrite(int[]) may expose internal representation by storing an externally mutable object into ByteCode.rewrite | MALICIOUS_CODE | EI_EXPOSE_REP2 | 352 | Medium |
|
|
org.apache.harmony.unpack200.bytecode.CPMethod
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.unpack200.bytecode.CPMethod doesn't override CPMember.equals(Object) | STYLE | EQ_DOESNT_OVERRIDE_EQUALS | 1 | Medium |
|
|
org.apache.harmony.unpack200.bytecode.ClassConstantPool
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.unpack200.bytecode.ClassConstantPool.sortClassPool() invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 235 | Medium |
|
|
org.apache.harmony.unpack200.bytecode.ClassFile
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unread field: org.apache.harmony.unpack200.bytecode.ClassFile.magic; should this field be static? | PERFORMANCE | SS_SHOULD_BE_STATIC | 29 | Medium |
|
|
org.apache.harmony.unpack200.bytecode.CodeAttribute
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| new org.apache.harmony.unpack200.bytecode.CodeAttribute(int, int, byte[], Segment, OperandManager, List) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 45 | Medium |
|
|
org.apache.harmony.unpack200.bytecode.ExceptionsAttribute
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| new org.apache.harmony.unpack200.bytecode.ExceptionsAttribute(CPClass[]) may expose internal representation by storing an externally mutable object into ExceptionsAttribute.exceptions | MALICIOUS_CODE | EI_EXPOSE_REP2 | 48 | Medium |
|
|
org.apache.harmony.unpack200.bytecode.LineNumberTableAttribute
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| new org.apache.harmony.unpack200.bytecode.LineNumberTableAttribute(int, int[], int[]) may expose internal representation by storing an externally mutable object into LineNumberTableAttribute.line_numbers | MALICIOUS_CODE | EI_EXPOSE_REP2 | 41 | Medium |
| new org.apache.harmony.unpack200.bytecode.LineNumberTableAttribute(int, int[], int[]) may expose internal representation by storing an externally mutable object into LineNumberTableAttribute.start_pcs | MALICIOUS_CODE | EI_EXPOSE_REP2 | 40 | Medium |
|
|
org.apache.harmony.unpack200.bytecode.LocalVariableTableAttribute
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| new org.apache.harmony.unpack200.bytecode.LocalVariableTableAttribute(int, int[], int[], CPUTF8[], CPUTF8[], int[]) may expose internal representation by storing an externally mutable object into LocalVariableTableAttribute.descriptors | MALICIOUS_CODE | EI_EXPOSE_REP2 | 54 | Medium |
| new org.apache.harmony.unpack200.bytecode.LocalVariableTableAttribute(int, int[], int[], CPUTF8[], CPUTF8[], int[]) may expose internal representation by storing an externally mutable object into LocalVariableTableAttribute.indexes | MALICIOUS_CODE | EI_EXPOSE_REP2 | 55 | Medium |
| new org.apache.harmony.unpack200.bytecode.LocalVariableTableAttribute(int, int[], int[], CPUTF8[], CPUTF8[], int[]) may expose internal representation by storing an externally mutable object into LocalVariableTableAttribute.lengths | MALICIOUS_CODE | EI_EXPOSE_REP2 | 52 | Medium |
| new org.apache.harmony.unpack200.bytecode.LocalVariableTableAttribute(int, int[], int[], CPUTF8[], CPUTF8[], int[]) may expose internal representation by storing an externally mutable object into LocalVariableTableAttribute.names | MALICIOUS_CODE | EI_EXPOSE_REP2 | 53 | Medium |
| new org.apache.harmony.unpack200.bytecode.LocalVariableTableAttribute(int, int[], int[], CPUTF8[], CPUTF8[], int[]) may expose internal representation by storing an externally mutable object into LocalVariableTableAttribute.start_pcs | MALICIOUS_CODE | EI_EXPOSE_REP2 | 51 | Medium |
|
|
org.apache.harmony.unpack200.bytecode.LocalVariableTypeTableAttribute
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| new org.apache.harmony.unpack200.bytecode.LocalVariableTypeTableAttribute(int, int[], int[], CPUTF8[], CPUTF8[], int[]) may expose internal representation by storing an externally mutable object into LocalVariableTypeTableAttribute.indexes | MALICIOUS_CODE | EI_EXPOSE_REP2 | 55 | Medium |
| new org.apache.harmony.unpack200.bytecode.LocalVariableTypeTableAttribute(int, int[], int[], CPUTF8[], CPUTF8[], int[]) may expose internal representation by storing an externally mutable object into LocalVariableTypeTableAttribute.lengths | MALICIOUS_CODE | EI_EXPOSE_REP2 | 52 | Medium |
| new org.apache.harmony.unpack200.bytecode.LocalVariableTypeTableAttribute(int, int[], int[], CPUTF8[], CPUTF8[], int[]) may expose internal representation by storing an externally mutable object into LocalVariableTypeTableAttribute.names | MALICIOUS_CODE | EI_EXPOSE_REP2 | 53 | Medium |
| new org.apache.harmony.unpack200.bytecode.LocalVariableTypeTableAttribute(int, int[], int[], CPUTF8[], CPUTF8[], int[]) may expose internal representation by storing an externally mutable object into LocalVariableTypeTableAttribute.signatures | MALICIOUS_CODE | EI_EXPOSE_REP2 | 54 | Medium |
| new org.apache.harmony.unpack200.bytecode.LocalVariableTypeTableAttribute(int, int[], int[], CPUTF8[], CPUTF8[], int[]) may expose internal representation by storing an externally mutable object into LocalVariableTypeTableAttribute.start_pcs | MALICIOUS_CODE | EI_EXPOSE_REP2 | 51 | Medium |
|
|
org.apache.harmony.unpack200.bytecode.NewAttribute
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.unpack200.bytecode.NewAttribute.addBCIndex(int, int) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 107 | Medium |
| org.apache.harmony.unpack200.bytecode.NewAttribute.addBCLength(int, int) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 112 | Medium |
| org.apache.harmony.unpack200.bytecode.NewAttribute.addBCOffset(int, int) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 102 | Medium |
| org.apache.harmony.unpack200.bytecode.NewAttribute.addInteger(int, long) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 97 | Medium |
| org.apache.harmony.unpack200.bytecode.NewAttribute.addInteger(int, long) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 98 | Medium |
| org.apache.harmony.unpack200.bytecode.NewAttribute.addToBody(int, Object) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 117 | Medium |
|
|
org.apache.harmony.unpack200.bytecode.OperandManager
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| new org.apache.harmony.unpack200.bytecode.OperandManager(int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[]) may expose internal representation by storing an externally mutable object into OperandManager.bcByte | MALICIOUS_CODE | EI_EXPOSE_REP2 | 89 | Medium |
| new org.apache.harmony.unpack200.bytecode.OperandManager(int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[]) may expose internal representation by storing an externally mutable object into OperandManager.bcCaseCount | MALICIOUS_CODE | EI_EXPOSE_REP2 | 87 | Medium |
| new org.apache.harmony.unpack200.bytecode.OperandManager(int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[]) may expose internal representation by storing an externally mutable object into OperandManager.bcCaseValue | MALICIOUS_CODE | EI_EXPOSE_REP2 | 88 | Medium |
| new org.apache.harmony.unpack200.bytecode.OperandManager(int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[]) may expose internal representation by storing an externally mutable object into OperandManager.bcClassRef | MALICIOUS_CODE | EI_EXPOSE_REP2 | 98 | Medium |
| new org.apache.harmony.unpack200.bytecode.OperandManager(int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[]) may expose internal representation by storing an externally mutable object into OperandManager.bcDoubleRef | MALICIOUS_CODE | EI_EXPOSE_REP2 | 96 | Medium |
| new org.apache.harmony.unpack200.bytecode.OperandManager(int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[]) may expose internal representation by storing an externally mutable object into OperandManager.bcFieldRef | MALICIOUS_CODE | EI_EXPOSE_REP2 | 99 | Medium |
| new org.apache.harmony.unpack200.bytecode.OperandManager(int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[]) may expose internal representation by storing an externally mutable object into OperandManager.bcFloatRef | MALICIOUS_CODE | EI_EXPOSE_REP2 | 94 | Medium |
| new org.apache.harmony.unpack200.bytecode.OperandManager(int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[]) may expose internal representation by storing an externally mutable object into OperandManager.bcIMethodRef | MALICIOUS_CODE | EI_EXPOSE_REP2 | 101 | Medium |
| new org.apache.harmony.unpack200.bytecode.OperandManager(int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[]) may expose internal representation by storing an externally mutable object into OperandManager.bcInitRef | MALICIOUS_CODE | EI_EXPOSE_REP2 | 107 | Medium |
| new org.apache.harmony.unpack200.bytecode.OperandManager(int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[]) may expose internal representation by storing an externally mutable object into OperandManager.bcIntRef | MALICIOUS_CODE | EI_EXPOSE_REP2 | 93 | Medium |
| new org.apache.harmony.unpack200.bytecode.OperandManager(int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[]) may expose internal representation by storing an externally mutable object into OperandManager.bcLabel | MALICIOUS_CODE | EI_EXPOSE_REP2 | 92 | Medium |
| new org.apache.harmony.unpack200.bytecode.OperandManager(int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[]) may expose internal representation by storing an externally mutable object into OperandManager.bcLocal | MALICIOUS_CODE | EI_EXPOSE_REP2 | 91 | Medium |
| new org.apache.harmony.unpack200.bytecode.OperandManager(int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[]) may expose internal representation by storing an externally mutable object into OperandManager.bcLongRef | MALICIOUS_CODE | EI_EXPOSE_REP2 | 95 | Medium |
| new org.apache.harmony.unpack200.bytecode.OperandManager(int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[]) may expose internal representation by storing an externally mutable object into OperandManager.bcMethodRef | MALICIOUS_CODE | EI_EXPOSE_REP2 | 100 | Medium |
| new org.apache.harmony.unpack200.bytecode.OperandManager(int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[]) may expose internal representation by storing an externally mutable object into OperandManager.bcShort | MALICIOUS_CODE | EI_EXPOSE_REP2 | 90 | Medium |
| new org.apache.harmony.unpack200.bytecode.OperandManager(int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[]) may expose internal representation by storing an externally mutable object into OperandManager.bcStringRef | MALICIOUS_CODE | EI_EXPOSE_REP2 | 97 | Medium |
| new org.apache.harmony.unpack200.bytecode.OperandManager(int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[]) may expose internal representation by storing an externally mutable object into OperandManager.bcSuperField | MALICIOUS_CODE | EI_EXPOSE_REP2 | 104 | Medium |
| new org.apache.harmony.unpack200.bytecode.OperandManager(int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[]) may expose internal representation by storing an externally mutable object into OperandManager.bcSuperMethod | MALICIOUS_CODE | EI_EXPOSE_REP2 | 106 | Medium |
| new org.apache.harmony.unpack200.bytecode.OperandManager(int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[]) may expose internal representation by storing an externally mutable object into OperandManager.bcThisField | MALICIOUS_CODE | EI_EXPOSE_REP2 | 103 | Medium |
| new org.apache.harmony.unpack200.bytecode.OperandManager(int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[]) may expose internal representation by storing an externally mutable object into OperandManager.bcThisMethod | MALICIOUS_CODE | EI_EXPOSE_REP2 | 105 | Medium |
| new org.apache.harmony.unpack200.bytecode.OperandManager(int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[], int[]) may expose internal representation by storing an externally mutable object into OperandManager.wideByteCodes | MALICIOUS_CODE | EI_EXPOSE_REP2 | 108 | Medium |
|
|
org.apache.harmony.unpack200.bytecode.RuntimeVisibleorInvisibleAnnotationsAttribute
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| new org.apache.harmony.unpack200.bytecode.RuntimeVisibleorInvisibleAnnotationsAttribute(CPUTF8, AnnotationsAttribute$Annotation[]) may expose internal representation by storing an externally mutable object into RuntimeVisibleorInvisibleAnnotationsAttribute.annotations | MALICIOUS_CODE | EI_EXPOSE_REP2 | 38 | Medium |
|
|
org.apache.harmony.unpack200.bytecode.RuntimeVisibleorInvisibleParameterAnnotationsAttribute
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| new org.apache.harmony.unpack200.bytecode.RuntimeVisibleorInvisibleParameterAnnotationsAttribute(CPUTF8, RuntimeVisibleorInvisibleParameterAnnotationsAttribute$ParameterAnnotation[]) may expose internal representation by storing an externally mutable object into RuntimeVisibleorInvisibleParameterAnnotationsAttribute.parameter_annotations | MALICIOUS_CODE | EI_EXPOSE_REP2 | 39 | Medium |
|
|
org.apache.harmony.unpack200.bytecode.RuntimeVisibleorInvisibleParameterAnnotationsAttribute$ParameterAnnotation
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| new org.apache.harmony.unpack200.bytecode.RuntimeVisibleorInvisibleParameterAnnotationsAttribute$ParameterAnnotation(AnnotationsAttribute$Annotation[]) may expose internal representation by storing an externally mutable object into RuntimeVisibleorInvisibleParameterAnnotationsAttribute$ParameterAnnotation.annotations | MALICIOUS_CODE | EI_EXPOSE_REP2 | 79 | Medium |
|
|
org.apache.harmony.unpack200.bytecode.forms.ByteCodeForm
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.harmony.unpack200.bytecode.forms.ByteCodeForm.getRewrite() may expose internal representation by returning ByteCodeForm.rewrite | MALICIOUS_CODE | EI_EXPOSE_REP | 464 | Medium |
| new org.apache.harmony.unpack200.bytecode.forms.ByteCodeForm(int, String, int[]) may expose internal representation by storing an externally mutable object into ByteCodeForm.rewrite | MALICIOUS_CODE | EI_EXPOSE_REP2 | 401 | Medium |
| org.apache.harmony.unpack200.bytecode.forms.ByteCodeForm.byteCodesByName is a mutable collection which should be package protected | MALICIOUS_CODE | MS_MUTABLE_COLLECTION_PKGPROTECT | 31 | Medium |
| org.apache.harmony.unpack200.bytecode.forms.ByteCodeForm.byteCodeArray should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 30 | Medium |
|
|
