roxen.lists.roxen.general

Subject Author Date
[PATCH 11/17] Cater for lost connections during the last query. Stephen R. van den Berg <srb[at]cuci[dot]nl> 20-01-2009 
This shouldn't happen, but if it does, it becomes
very confusing to localise because of the surrounding
catch.  Errors in errormessages are bad news.
---

 server/modules/database/sqltag.pike |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/server/modules/database/sqltag.pike
b/server/modules/database/sqltag.pike
index 71b6c98..8b14eaf 100644
--- a/server/modules/database/sqltag.pike
+++ b/server/modules/database/sqltag.pike
@@ -523,7 +523,7 @@ array|Sql.sql_result do_sql_query(mapping args, RequestID id,
     function query_fn = (big_query ? con->big_query : con->query); 
     if( error = catch( result = (bindings ? query_fn(args->query, bindings) :
query_fn(args->query))) ) {
       error = sprintf("Query failed: %s\n",
-		      con->error() || describe_error(error));
+		      con && con->error() || describe_error(error));
       RXML.run_error(error);
     }
   }