[findbug]Bug type EI_EXPOSE_REP2

使用findbugs扫描出该问题

findbug提示原文:

new top.jieziss.common.OldFileBodyUtil(byte[], String) may expose internal representation by storing an externally mutable object into OldFileBodyUtil.text
Bug type EI_EXPOSE_REP2 (click for details)
In class top.jieziss.common.OldFileBodyUtil
In method new top.jieziss.common.OldFileBodyUtil(byte[], String)
Field top.jieziss.common.OldFileBodyUtil.text
Local variable named text
At OldFileBodyUtil.java from InputFileObject:[line 22]

产生原因:

大概意思就是OldFileBodyUtil(byte[],String)可以通过将外部可变对象存储到OldFileBodyUtil.text中来公开内部表示,在许多情况下,存储对象的副本是更好的方法。

这里是重写了FileBody,使用httpClient上传文件流。问题改正如下:

原代码:

public OldFileBodyUtil(byte[] text, String fileName) {
        super(file);
        this.text = text;
        this.fileName = fileName;
    }

改正代码:

public OldFileBodyUtil(byte[] text, String fileName) {
        super(file);
        this.text = text.clone();
        this.fileName = fileName;
    }

  • 2
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 1
    评论

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论 1
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值