don't advance iterator if empty/at end for geo cursor SERVER-3428

moonwatcher · Jul 14, 2011 · 146ef71 · 146ef71
1 parent 4ef7ea1
commit 146ef71
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/db/geo/2d.cpp b/db/geo/2d.cpp
@@ -1928,7 +1928,14 @@ namespace mongo {
         virtual Record* _current() { assert(ok()); return _cur->_loc.rec(); }
         virtual BSONObj current() { assert(ok()); return _cur->_o; }
         virtual DiskLoc currLoc() { assert(ok()); return _cur->_loc; }
-        virtual bool advance() { _cur++; incNscanned(); return ok(); }
+        virtual bool advance() {
+            bool isOk = ok();
+            if( isOk ){
+                _cur++;
+                incNscanned();
+            }
+            return isOk;
+        }
         virtual BSONObj currKey() const { return _cur->_key; }
 
         virtual string toString() {