出厂测试整改报告

出厂测试整改报告2016年4月23日测试结果概况此次测试，为3月31日上线的功能模块。送审的模块共有13个，页面53个，代码总量大约为21.7万行，源程序文件1375个。此次测试结果如下：主要分为代码扫描问题，安全功能问题和渗透测试问题。其中代码扫描问题高危风险1个，中危风险256个，低位风险1281个，共1538个；安全功能问题16个；渗透测试问题7个。按照测试标准，高危风险必须为0个，中危风险小于50个，低危风险不用整改。因此此次测试的结果是不理想的，没有通过测试。原因分析由于业务需求进行了调整，3月上线的模块与5月的上线模块出现了很大的差异。工作量也比较大，当时项目组忙着赶工5月份模块了，没有将精力重点放在3月模块上。我方内部对代码的编写评审是很严格的，但此次由于精力有些分散，最后有部分代码未能内部走查，导致出现了大量了代码扫描的风险。测试结果分析后，发现大量的错误（155个）发生在一本代码上，我们已经内部对出现问题的责任PG、管理SE、PL等一系列相关人员进行了批评教育，同时加强内部代码走查管理，以保证杜绝此类错误的再次发生。缺乏出厂测试的测试经验，对出厂测试的理解仅仅停留在功能测试，对安全测试准备不充分。同时，测试结果的存在同样代码被多次统计的情况，比如：同样的代码被多次统计（DataUtils.java的1行代码），涉及到的问题为155个。若修改此处代码即可解决155个问题。getString，Parse方法异常发生时UAP中已经对异常进行了处理，但是代码扫描工具没有识别到UAP中的Jar包,涉及到的问题1007个。整体整改措施措施时间状态认真分析问题原因，并制订整改方案。整理出适用于国网出厂测试的内部测试流程，提升测试的效率效果。2016年4月23日完成对报告中的问题逐项制定整改措施并全员宣贯，包括低危风险。2016年4月25日根据整改措施进行修改并完成自查。2016年4月25日-2016年5月1日使用同样的代码扫描软件在出厂测试前进行代码扫描。2016年5月3日正在沟通代码扫描问题及具体整改措施问题分类数量级别整改措施ReflectedXSSAllClients1High改变获取参数的方法UncheckedInputforLoopCondition228Medium所有保存限制行数所有删除限制行数PotentialXSS12Medium甘特图功能采用了另外的组件，删除文件jquerysvg.js，jquerysvgmin.jsUseofCryptographicallyWeakPRNG6Medium不使用随机函数UseofInsufficientlyRandomValues6Medium不使用随机函数TrustBoundaryViolation1Medium登录成功后，设置session前，清除当前sessionrequest.getSession().invalidate();HttpSessionsession=request.getSession(true);UseofaOneWayHashwithoutaSalt1Medium使用BASE64+MD5进行加密SessionFixation1Medium登录成功后，设置session前，清除当前sessionrequest.getSession().invalidate();HttpSessionsession=request.getSession(true);HttpOnlyCookiesInConfig1Mediumweb.xml追加配置<session-config><cookie-config><http-only>true</http-only></cookie-config><session-config>ImproperExceptionHandling1007Low修改从数据库取值方式IntegerOverflow96Low追加计算溢出判断IntegerUnderflow96Low追加计算溢出判断RaceConditionFormatFlaw49Lowparse()和format()不能和其他函数一同使用SerializableClassContainingSensitiveData19Low去掉序列化HeuristicSQLInjection4Low1、在系统全局增加过滤器，对select、and、union、order、by、=、’、”、等参数与符号进行输入过滤。2、在输入变量处，对select、and、union、order、by、=、’、”、等参数与符号进行输入过滤。3、sql语句的参数都使用传参方式，严禁拼接参数。JQueryDeprecatedSymbols4Low废弃使用此方法。HeuristicXSRF2Lowsql语句的参数都使用传参方式，严禁拼接参数。DivideByZero2Low判断除数是否为0UseofBrokenorRiskyCryptographicAlgorithm1Low使用BASE64+MD5进行加密ReversibleOneWayHash1Low使用BASE64+MD5进行加密安全功能问题及具体整改措施问题整改措施系统登录无用户身份选测选项。与测试中心沟通，不需要整改。系统存在用户多点并发。增加多点判断以禁止多点并发。系统存在弱口令，要求密码为8位以上数字加字符。增加密码约束条件为8位以上数字加字符。系统存在越权现象。追加拦截器，检查访问此URL的用户是否具有此权限新建用户不会强制修改密码：要求：新建用户登录需要强制修改密码。修改内容：判断用户的”密码修改日期”是否为空。为空则是新建用户，强制用户修改密码。密码不会过期，要求：密码有效期为三