Bug 524787 - crash [@mozStorageResultSet::GetNextRow(mozIStorageRow**) ]. r=sdwilsh
authorDavid Dahl <ddahl@mozilla.com>
Sat, 31 Oct 2009 11:38:31 +0100
changeset 34439 4f972fa92cd01682668fe166a0ce9db9df1f8d74
parent 34438 7e0336a4f6fdfd8a911c9a0f19777de9799dc868
child 34440 293f92d1a482515d10b0b24762671c7fdb41b368
push id1
push userroot
push dateTue, 26 Apr 2011 22:38:44 +0000
treeherdermozilla-beta@bfdb6e623a36 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerssdwilsh
bugs524787
milestone1.9.3a1pre
Bug 524787 - crash [@mozStorageResultSet::GetNextRow(mozIStorageRow**) ]. r=sdwilsh
storage/src/mozStorageResultSet.cpp
--- a/storage/src/mozStorageResultSet.cpp
+++ b/storage/src/mozStorageResultSet.cpp
@@ -72,16 +72,18 @@ NS_IMPL_THREADSAFE_ISUPPORTS1(
 )
 
 ////////////////////////////////////////////////////////////////////////////////
 //// mozIStorageResultSet
 
 NS_IMETHODIMP
 ResultSet::GetNextRow(mozIStorageRow **_row)
 {
+  NS_ENSURE_ARG_POINTER(_row);
+
   if (mCurrentIndex >= mData.Count()) {
     // Just return null here
     return NS_OK;
   }
 
   NS_ADDREF(*_row = mData.ObjectAt(mCurrentIndex++));
   return NS_OK;
 }