Fix for ticket #1149. At times the current row can be returned as -1 …

…which will cause a crash if you try to fetch its display role. git-svn-id: http://svn.osgeo.org/qgis/trunk/qgis@8754 c8812cc2-4d05-0410-92ff-de0c093fc19c
qgis · Jul 11, 2008 · 154bb2f · 154bb2f
1 parent 93a0708
commit 154bb2f
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/src/app/qgsattributetable.cpp b/src/app/qgsattributetable.cpp
@@ -358,8 +358,15 @@ void QgsAttributeTable::contextMenuEvent(QContextMenuEvent *event)
   mActionValues.clear();
 
   for (int i = 0; i < columnCount(); ++i)
-    mActionValues.push_back(std::make_pair(horizontalHeaderItem(i)->text(), item(row, i)->text()));
-
+  {
+    if (row >= 0) //prevent crash if row is negative, see ticket #1149
+    {
+      mActionValues.push_back(
+        std::make_pair(
+          horizontalHeaderItem( i )->text(),
+          item( row, i )->text() ) );
+    }
+  }
   // The item that was clicked on, stored as an index into the
   // mActionValues vector.
   mClickedOnValue = col;