The following document contains the results of FindBugs Report
FindBugs Version is 1.2.1
Threshold is Low
Effort is Default
Classes | Bugs | Errors | Missing Classes |
---|---|---|---|
308 | 27 | 0 | 0 |
Bug | Category | Details | Line |
---|---|---|---|
Unchecked/unconfirmed cast from org.sat4j.specs.IVec to org.sat4j.core.Vec in org.sat4j.core.Vec.copyTo(IVec) | STYLE | BC_UNCONFIRMED_CAST | 287 |
org.sat4j.core.Vec.toArray() may expose internal representation by returning Vec.myarray | MALICIOUS_CODE | EI_EXPOSE_REP | 319 |
org.sat4j.core.Vec.Vec(Object[]) may expose internal representation by storing an externally mutable object into Vec.myarray | MALICIOUS_CODE | EI_EXPOSE_REP2 | 87 |
Bug | Category | Details | Line |
---|---|---|---|
Unchecked/unconfirmed cast from org.sat4j.specs.IVecInt to org.sat4j.core.VecInt in org.sat4j.core.VecInt.copyTo(IVecInt) | STYLE | BC_UNCONFIRMED_CAST | 329 |
Unchecked/unconfirmed cast from org.sat4j.specs.IVecInt to org.sat4j.core.VecInt in org.sat4j.core.VecInt.moveTo2(IVecInt) | STYLE | BC_UNCONFIRMED_CAST | 353 |
Unchecked/unconfirmed cast from org.sat4j.specs.IVecInt to org.sat4j.core.VecInt in org.sat4j.core.VecInt.pushAll(IVecInt) | STYLE | BC_UNCONFIRMED_CAST | 565 |
org.sat4j.core.VecInt.toArray() may expose internal representation by returning VecInt.myarray | MALICIOUS_CODE | EI_EXPOSE_REP | 618 |
org.sat4j.core.VecInt.VecInt(int[]) may expose internal representation by storing an externally mutable object into VecInt.myarray | MALICIOUS_CODE | EI_EXPOSE_REP2 | 200 |
Bug | Category | Details | Line |
---|---|---|---|
Class org.sat4j.minisat.constraints.cnf.Lits defines non-transient non-serializable instance field reason | BAD_PRACTICE | SE_BAD_FIELD | Not available |
Bug | Category | Details | Line |
---|---|---|---|
org.sat4j.minisat.core.Heap.Heap(double[]) may expose internal representation by storing an externally mutable object into Heap.activity | MALICIOUS_CODE | EI_EXPOSE_REP2 | 103 |
Bug | Category | Details | Line |
---|---|---|---|
Class org.sat4j.minisat.core.Pair defines non-transient non-serializable instance field reason | BAD_PRACTICE | SE_BAD_FIELD | Not available |
Bug | Category | Details | Line |
---|---|---|---|
org.sat4j.minisat.core.Solver.reduceDB() forces garbage collection; extremely dubious except in benchmarking code | PERFORMANCE | DM_GC | 978 |
Should org.sat4j.minisat.core.Solver.findModel() return a zero length array rather than null? | STYLE | PZLA_PREFER_ZERO_LENGTH_ARRAYS | 1513 |
Should org.sat4j.minisat.core.Solver.findModel(IVecInt) return a zero length array rather than null? | STYLE | PZLA_PREFER_ZERO_LENGTH_ARRAYS | 1522 |
Exception is caught when Exception is not thrown in org.sat4j.minisat.core.Solver.setSimplifier(String) | STYLE | REC_CATCH_EXCEPTION | 588 |
Bug | Category | Details | Line |
---|---|---|---|
AbstractLearning.val not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
Bug | Category | Details | Line |
---|---|---|---|
LimitedLearning.stats not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
Bug | Category | Details | Line |
---|---|---|---|
MiniSATLearning.dsf not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
Bug | Category | Details | Line |
---|---|---|---|
org.sat4j.opt.AbstractSelectorVariablesDecorator.model() may expose internal representation by returning AbstractSelectorVariablesDecorator.prevmodel | MALICIOUS_CODE | EI_EXPOSE_REP | 131 |
Bug | Category | Details | Line |
---|---|---|---|
Class org.sat4j.opt.MaxSatDecorator defines non-transient non-serializable instance field prevConstr | BAD_PRACTICE | SE_BAD_FIELD | Not available |
Bug | Category | Details | Line |
---|---|---|---|
org.sat4j.opt.MinOneDecorator.model() may expose internal representation by returning MinOneDecorator.prevmodel | MALICIOUS_CODE | EI_EXPOSE_REP | 123 |
Class org.sat4j.opt.MinOneDecorator defines non-transient non-serializable instance field previousConstr | BAD_PRACTICE | SE_BAD_FIELD | Not available |
MinOneDecorator.prevmodel not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
Bug | Category | Details | Line |
---|---|---|---|
The field org.sat4j.reader.EfficientScanner.in is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available |
Bug | Category | Details | Line |
---|---|---|---|
LecteurDimacs.in not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
Bug | Category | Details | Line |
---|---|---|---|
org.sat4j.tools.xplain.Pair defines compareTo(Pair) and uses Object.equals() | BAD_PRACTICE | EQ_COMPARETO_USE_OBJECT_EQUALS | 46-51 |
Test for floating point equality in org.sat4j.tools.xplain.Pair.compareTo(Pair) | STYLE | FE_FLOATING_POINT_EQUALITY | 48 |