Merge pull request #630 from timll/develop

StevenArzt · web-flow · commit 1c0aa2b71748 · 2023-07-04T13:17:36.000+02:00
Fix getSinkInfo returning no defs
diff --git a/soot-infoflow/src/soot/jimple/infoflow/sourcesSinks/manager/BaseSourceSinkManager.java b/soot-infoflow/src/soot/jimple/infoflow/sourcesSinks/manager/BaseSourceSinkManager.java
@@ -386,7 +386,7 @@ public SinkInfo getSinkInfo(Stmt sCallSite, InfoflowManager manager, AccessPath
 			return null;
 
 		Collection<ISourceSinkDefinition> def = getSinkDefinitions(sCallSite, manager, ap);
-		return def == null ? null : new SinkInfo(def);
+		return def.size() > 0 ? new SinkInfo(def) : null;
 	}
 
 	@Override

Original file line number	Diff line number	Diff line change
`@@ -386,7 +386,7 @@ public SinkInfo getSinkInfo(Stmt sCallSite, InfoflowManager manager, AccessPath`
`386`	`386`	`return null;`
`387`	`387`
`388`	`388`	`Collection<ISourceSinkDefinition> def = getSinkDefinitions(sCallSite, manager, ap);`
`389`		`- return def == null ? null : new SinkInfo(def);`
	`389`	`+ return def.size() > 0 ? new SinkInfo(def) : null;`
`390`	`390`	`}`
`391`	`391`
`392`	`392`	`@Override`