lttng: fix NPE when segment selection is null

Change-Id: I1ac9dd36c0465b3261805f362c41a595a55f5df5
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58061
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

diff --git a/analysis/org.eclipse.tracecompass.analysis.timing.ui/src/org/eclipse/tracecompass/analysis/timing/ui/views/segmentstore/AbstractSegmentStoreTableViewer.java b/analysis/org.eclipse.tracecompass.analysis.timing.ui/src/org/eclipse/tracecompass/analysis/timing/ui/views/segmentstore/AbstractSegmentStoreTableViewer.java
index 8e518bcd345f041e839587db31a28227803a7bff..91a5ca226651dab22d7724e20314092f6fb010ad 100644 (file)
--- a/analysis/org.eclipse.tracecompass.analysis.timing.ui/src/org/eclipse/tracecompass/analysis/timing/ui/views/segmentstore/AbstractSegmentStoreTableViewer.java
+++ b/analysis/org.eclipse.tracecompass.analysis.timing.ui/src/org/eclipse/tracecompass/analysis/timing/ui/views/segmentstore/AbstractSegmentStoreTableViewer.java
@@ -264,23 +264,24 @@ public abstract class AbstractSegmentStoreTableViewer extends TmfSimpleTableView
     @Override
     protected void appendToTablePopupMenu(IMenuManager manager, IStructuredSelection sel) {
         final ISegment segment = (ISegment) sel.getFirstElement();
+        if (segment != null) {
+            IAction gotoStartTime = new Action(Messages.SegmentStoreTableViewer_goToStartEvent) {
+                @Override
+                public void run() {
+                    broadcast(new TmfSelectionRangeUpdatedSignal(AbstractSegmentStoreTableViewer.this, new TmfNanoTimestamp(segment.getStart())));
+                }
+            };
 
-        IAction gotoStartTime = new Action(Messages.SegmentStoreTableViewer_goToStartEvent) {
-            @Override
-            public void run() {
-                broadcast(new TmfSelectionRangeUpdatedSignal(AbstractSegmentStoreTableViewer.this, new TmfNanoTimestamp(segment.getStart())));
-            }
-        };
-
-        IAction gotoEndTime = new Action(Messages.SegmentStoreTableViewer_goToEndEvent) {
-            @Override
-            public void run() {
-                broadcast(new TmfSelectionRangeUpdatedSignal(AbstractSegmentStoreTableViewer.this, new TmfNanoTimestamp(segment.getEnd())));
-            }
-        };
+            IAction gotoEndTime = new Action(Messages.SegmentStoreTableViewer_goToEndEvent) {
+                @Override
+                public void run() {
+                    broadcast(new TmfSelectionRangeUpdatedSignal(AbstractSegmentStoreTableViewer.this, new TmfNanoTimestamp(segment.getEnd())));
+                }
+            };
 
-       manager.add(gotoStartTime);
-       manager.add(gotoEndTime);
+            manager.add(gotoStartTime);
+            manager.add(gotoEndTime);
+        }
     }
 
     // ------------------------------------------------------------------------