Fixing null pointer dereference.

sql/connection.h

Index: sql/connection.h
diff --git a/sql/connection.h b/sql/connection.h
index b9f45ec480c8af44e03309b060abd07360582a1c..8e7f5f549ada70a8396929b8749d13b6559fb40e 100644
--- a/sql/connection.h
+++ b/sql/connection.h
@@ -370,8 +370,9 @@ class SQL_EXPORT Connection {
     // When true, the statement can be used.
     bool is_valid() const { return !!stmt_; }
 
-    // If we've not been linked to a connection, this will be NULL. Guaranteed
-    // non-NULL when is_valid().
+    // If we've not been linked to a connection, this will be NULL.
+    // Not guaranteed to be non-NULL when is_valid() - see
+    // StatementRef::StatementRef(sqlite3_stmt* stmt).

[Scott Hess - ex-Googler, 2013/01/16 20:49:07]

Please rephrase the change something like:

// TODO(shess): connection_ can be NULL in case of GetUntrackedStatement(),
which prevents Statement::OnError() from forwarding errors.

Mostly, I'd rather aim to restore the guarantee in the future.

[glotov, 2013/01/17 10:58:22]

Done.

     Connection* connection() const { return connection_; }
 
     // Returns the sqlite statement if any. If the statement is not active,