INTERNAL: remove potential bugs 99/27599/2
authorwoojin <woojin2.jung@samsung.com>
Tue, 16 Sep 2014 11:15:03 +0000 (20:15 +0900)
committerwoojin <woojin2.jung@samsung.com>
Tue, 16 Sep 2014 12:00:54 +0000 (21:00 +0900)
add null check, synchronization check

Change-Id: I6543b822bbe4e17c10cd8e65605df00f738aae24
Signed-off-by: woojin <woojin2.jung@samsung.com>
org.tizen.dynamicanalyzer/src/org/tizen/dynamicanalyzer/swap/callstack/BaseCallstackManager.java
org.tizen.dynamicanalyzer/src/org/tizen/dynamicanalyzer/ui/summary/profiling/ProfileDataMaker.java

index 776d602..8509e6e 100644 (file)
@@ -57,7 +57,7 @@ import org.tizen.dynamicanalyzer.util.Logger;
 import org.tizen.dynamicanalyzer.utils.AnalyzerUtil;
 
 public abstract class BaseCallstackManager {
-       protected class DuplicateUserCall {
+       protected static class DuplicateUserCall {
                private CallStackUnit dupCallstackUnit = null;
                private int dupCallcount = 0;
 
index a0bd724..b4b0e5d 100644 (file)
@@ -51,7 +51,7 @@ import org.tizen.dynamicanalyzer.util.Logger;
 import org.tizen.dynamicanalyzer.utils.Formatter;
 
 public class ProfileDataMaker {
-       class CallstackTree {
+       static class CallstackTree {
                ProfilingData me;
                CallstackTree parent;
                List<CallstackTree> children = null;
@@ -417,6 +417,10 @@ public class ProfileDataMaker {
                                } else {
                                        String pName = fupData.getParent();
                                        ProfilingData inParent = getProfilingDataByKey(pName);
+                                       if (inParent == null) {
+                                               Logger.error("ProfilingData: function item without library item");
+                                               continue;
+                                       }
                                        inParent.addInCount(totalSampleCount);
 
                                        if (null != inParent.getParent()) {