FindBugs Bug Detector Report
The following document contains the results of FindBugs
FindBugs Version is 3.0.1
Threshold is medium
Effort is min
Summary
| Classes |
Bugs |
Errors |
Missing Classes |
| 72 |
23 |
0 |
0 |
org.oxerr.okcoin.rest.dto.AccountRecords
| Bug |
Category |
Details |
Line |
Priority |
| org.oxerr.okcoin.rest.dto.AccountRecords.getRecords() may expose internal representation by returning AccountRecords.records |
MALICIOUS_CODE |
EI_EXPOSE_REP |
23 |
Medium |
| new org.oxerr.okcoin.rest.dto.AccountRecords(Boolean, int, AccountRecord[], String) may expose internal representation by storing an externally mutable object into AccountRecords.records |
MALICIOUS_CODE |
EI_EXPOSE_REP2 |
18 |
Medium |
org.oxerr.okcoin.rest.dto.BatchTradeResult
| Bug |
Category |
Details |
Line |
Priority |
| org.oxerr.okcoin.rest.dto.BatchTradeResult.getOrderInfo() may expose internal representation by returning BatchTradeResult.orderInfo |
MALICIOUS_CODE |
EI_EXPOSE_REP |
25 |
Medium |
| new org.oxerr.okcoin.rest.dto.BatchTradeResult(boolean, BatchTradeResult$OrderInfo[]) may expose internal representation by storing an externally mutable object into BatchTradeResult.orderInfo |
MALICIOUS_CODE |
EI_EXPOSE_REP2 |
21 |
Medium |
org.oxerr.okcoin.rest.dto.CancelOrderResult
| Bug |
Category |
Details |
Line |
Priority |
| Boxed value is unboxed and then immediately reboxed in new org.oxerr.okcoin.rest.dto.CancelOrderResult(Boolean, Long, String, String) |
PERFORMANCE |
BX_UNBOXING_IMMEDIATELY_REBOXED |
18 |
Medium |
org.oxerr.okcoin.rest.dto.CandlestickChart
| Bug |
Category |
Details |
Line |
Priority |
| org.oxerr.okcoin.rest.dto.CandlestickChart.getCandlesticks() may expose internal representation by returning CandlestickChart.candlesticks |
MALICIOUS_CODE |
EI_EXPOSE_REP |
19 |
Medium |
| new org.oxerr.okcoin.rest.dto.CandlestickChart(Candlestick[]) may expose internal representation by storing an externally mutable object into CandlestickChart.candlesticks |
MALICIOUS_CODE |
EI_EXPOSE_REP2 |
15 |
Medium |
org.oxerr.okcoin.rest.dto.Depth
| Bug |
Category |
Details |
Line |
Priority |
| org.oxerr.okcoin.rest.dto.Depth.getAsks() may expose internal representation by returning Depth.asks |
MALICIOUS_CODE |
EI_EXPOSE_REP |
23 |
Medium |
| org.oxerr.okcoin.rest.dto.Depth.getBids() may expose internal representation by returning Depth.bids |
MALICIOUS_CODE |
EI_EXPOSE_REP |
27 |
Medium |
| new org.oxerr.okcoin.rest.dto.Depth(BigDecimal[][], BigDecimal[][]) may expose internal representation by storing an externally mutable object into Depth.asks |
MALICIOUS_CODE |
EI_EXPOSE_REP2 |
18 |
Medium |
| new org.oxerr.okcoin.rest.dto.Depth(BigDecimal[][], BigDecimal[][]) may expose internal representation by storing an externally mutable object into Depth.bids |
MALICIOUS_CODE |
EI_EXPOSE_REP2 |
19 |
Medium |
org.oxerr.okcoin.rest.dto.IcebergOrderHistory
| Bug |
Category |
Details |
Line |
Priority |
| org.oxerr.okcoin.rest.dto.IcebergOrderHistory.getOrders() may expose internal representation by returning IcebergOrderHistory.orders |
MALICIOUS_CODE |
EI_EXPOSE_REP |
27 |
Medium |
| new org.oxerr.okcoin.rest.dto.IcebergOrderHistory(int, boolean, IcebergOrder[]) may expose internal representation by storing an externally mutable object into IcebergOrderHistory.orders |
MALICIOUS_CODE |
EI_EXPOSE_REP2 |
19 |
Medium |
org.oxerr.okcoin.rest.dto.LendDepth
| Bug |
Category |
Details |
Line |
Priority |
| org.oxerr.okcoin.rest.dto.LendDepth.getLendDepth() may expose internal representation by returning LendDepth.lendDepth |
MALICIOUS_CODE |
EI_EXPOSE_REP |
16 |
Medium |
| new org.oxerr.okcoin.rest.dto.LendDepth(Lend[]) may expose internal representation by storing an externally mutable object into LendDepth.lendDepth |
MALICIOUS_CODE |
EI_EXPOSE_REP2 |
12 |
Medium |
org.oxerr.okcoin.rest.dto.OrderHistory
| Bug |
Category |
Details |
Line |
Priority |
| org.oxerr.okcoin.rest.dto.OrderHistory.getOrders() may expose internal representation by returning OrderHistory.orders |
MALICIOUS_CODE |
EI_EXPOSE_REP |
54 |
Medium |
| new org.oxerr.okcoin.rest.dto.OrderHistory(boolean, int, int, int, Order[]) may expose internal representation by storing an externally mutable object into OrderHistory.orders |
MALICIOUS_CODE |
EI_EXPOSE_REP2 |
28 |
Medium |
org.oxerr.okcoin.rest.dto.OrderResult
| Bug |
Category |
Details |
Line |
Priority |
| org.oxerr.okcoin.rest.dto.OrderResult.getOrders() may expose internal representation by returning OrderResult.orders |
MALICIOUS_CODE |
EI_EXPOSE_REP |
20 |
Medium |
| new org.oxerr.okcoin.rest.dto.OrderResult(boolean, int, Order[]) may expose internal representation by storing an externally mutable object into OrderResult.orders |
MALICIOUS_CODE |
EI_EXPOSE_REP2 |
16 |
Medium |
org.oxerr.okcoin.rest.dto.UnrepaymentsInfo
| Bug |
Category |
Details |
Line |
Priority |
| org.oxerr.okcoin.rest.dto.UnrepaymentsInfo.getUnrepayments() may expose internal representation by returning UnrepaymentsInfo.unrepayments |
MALICIOUS_CODE |
EI_EXPOSE_REP |
19 |
Medium |
| new org.oxerr.okcoin.rest.dto.UnrepaymentsInfo(BorrowOrder[], boolean) may expose internal representation by storing an externally mutable object into UnrepaymentsInfo.unrepayments |
MALICIOUS_CODE |
EI_EXPOSE_REP2 |
15 |
Medium |
org.oxerr.okcoin.rest.dto.valuereader.IcebergOrdersReader
| Bug |
Category |
Details |
Line |
Priority |
| Boxing/unboxing to parse a primitive org.oxerr.okcoin.rest.dto.valuereader.IcebergOrdersReader.read(HTMLDocument) |
PERFORMANCE |
DM_BOXED_PRIMITIVE_FOR_PARSING |
97 |
High |
org.oxerr.okcoin.rest.service.OKCoinAccountService
| Bug |
Category |
Details |
Line |
Priority |
| Call to org.knowm.xchange.currency.Currency.equals(String) in org.oxerr.okcoin.rest.service.OKCoinAccountService.withdrawFunds(Currency, BigDecimal, String) |
CORRECTNESS |
EC_UNRELATED_TYPES |
47 |
High |
