mirror of
https://github.com/ysoftdevs/DependencyCheck.git
synced 2026-03-21 16:49:43 +01:00
null checking proposed by coverity
This commit is contained in:
@@ -248,7 +248,7 @@ public class ArchiveAnalyzer extends AbstractFileTypeAnalyzer {
|
|||||||
//make a copy
|
//make a copy
|
||||||
final List<Dependency> dependencySet = findMoreDependencies(engine, tmpDir);
|
final List<Dependency> dependencySet = findMoreDependencies(engine, tmpDir);
|
||||||
|
|
||||||
if (!dependencySet.isEmpty()) {
|
if (dependencySet != null && !dependencySet.isEmpty()) {
|
||||||
for (Dependency d : dependencySet) {
|
for (Dependency d : dependencySet) {
|
||||||
if (d.getFilePath().startsWith(tmpDir.getAbsolutePath())) {
|
if (d.getFilePath().startsWith(tmpDir.getAbsolutePath())) {
|
||||||
//fix the dependency's display name and path
|
//fix the dependency's display name and path
|
||||||
@@ -314,7 +314,7 @@ public class ArchiveAnalyzer extends AbstractFileTypeAnalyzer {
|
|||||||
dependency.setSha1sum("");
|
dependency.setSha1sum("");
|
||||||
org.apache.commons.io.FileUtils.copyFile(dependency.getActualFile(), tmpLoc);
|
org.apache.commons.io.FileUtils.copyFile(dependency.getActualFile(), tmpLoc);
|
||||||
final List<Dependency> dependencySet = findMoreDependencies(engine, tmpLoc);
|
final List<Dependency> dependencySet = findMoreDependencies(engine, tmpLoc);
|
||||||
if (!dependencySet.isEmpty()) {
|
if (dependencySet != null && !dependencySet.isEmpty()) {
|
||||||
for (Dependency d : dependencySet) {
|
for (Dependency d : dependencySet) {
|
||||||
//fix the dependency's display name and path
|
//fix the dependency's display name and path
|
||||||
if (d.getActualFile().equals(tmpLoc)) {
|
if (d.getActualFile().equals(tmpLoc)) {
|
||||||
|
|||||||
Reference in New Issue
Block a user