SootUp icon indicating copy to clipboard operation
SootUp copied to clipboard

Spark Framework

Open kadirayk opened this issue 3 years ago • 9 comments

kadirayk avatar May 10 '21 16:05 kadirayk

@kadirayk what is the plan for this PR? Are you going to comment the classes which I should take a look?

linghuiluo avatar May 20 '21 18:05 linghuiluo

@linghuiluo there is a bug after merging, that fails some tests. I wanted to fix that first.

kadirayk avatar May 21 '21 07:05 kadirayk

@linghuiluo there is a bug after merging, that fails some tests. I wanted to fix that first.

No problem, take your time

linghuiluo avatar May 21 '21 07:05 linghuiluo

@linghuiluo added some comments now, maybe you can have a look at these classes in the meantime. Thanks :)

kadirayk avatar May 21 '21 07:05 kadirayk

@kadirayk I didn't see any handling of implicit edges such as Thread.run() -> Thread.start() or Executor.execute() to Thread.run(). or AsyncTask.execute() -> AsyncTask.doInBackground() Probably you just considered the basic edges at first, but don't forget to consider those edges in future. https://github.com/soot-oss/soot/blob/492fb4713f7e124ffdb0c991ac5453f96c23cc29/src/main/resources/virtualedges.xml https://github.com/soot-oss/soot/blob/3966f565db6dc2882c3538ffc39e44f4c14b5bcf/src/main/java/soot/jimple/toolkits/callgraph/OnFlyCallGraphBuilder.java#L853

linghuiluo avatar Jun 17 '21 09:06 linghuiluo

@linghuiluo thank you, just created an issue for that.

kadirayk avatar Jun 21 '21 07:06 kadirayk

Test Coverage Total coverage 0.6463063969156577 Amount of covered (bytecode-)branches 0.53099173553719

github-actions[bot] avatar Oct 25 '22 17:10 github-actions[bot]

May I ask if spark is already available?

struce2 avatar Aug 31 '23 07:08 struce2

@kadirayk may I ask what the status of this PR is? cc: @linghuiluo @swissiety

Jclavo avatar Dec 11 '23 22:12 Jclavo