r11859: Canonicalize whitespace
[clsql.git] / db-odbc / odbc-api.lisp
index a15d628f8c3c82120ed7cbf7e91f00ba5f6465b2..50ef4432d3485df1f6792b2cb01041c8a4e39106 100644 (file)
@@ -6,7 +6,7 @@
 ;;;; Purpose:  Low-level ODBC API using UFFI
 ;;;; Authors:  Kevin M. Rosenberg and Paul Meurer
 ;;;;
-;;;; $Id: odbc-package.lisp 7061 2003-09-07 06:34:45Z kevin $
+;;;; $Id$
 ;;;;
 ;;;; This file, part of CLSQL, is Copyright (c) 2004 by Kevin M. Rosenberg
 ;;;; and Copyright (C) Paul Meurer 1999 - 2001. All rights reserved.
 (defvar *null* nil
   "Lisp representation of SQL Null value, default = nil.
 May be locally bound to something else if a certain type is necessary.")
+
 
 (defvar *binary-format* :unsigned-byte-vector)
-(defvar *time-conversion-function* (lambda (universal-time &optional fraction)
-                                    (declare (ignore fraction))
-                                    universal-time)
+(defvar *time-conversion-function*
+    (lambda (universal-time &optional fraction)
+      (declare (ignore fraction))
+      (clsql-sys:format-time
+       nil (clsql-sys:utime->time universal-time)
+       :format :iso)
+      #+ignore
+      universal-time)
    "Bound to a function that converts from a Lisp universal time fixnum (and a fractional
-as possible second argument) to the desired representation of date/time/timestamp.")
+as possible second argument) to the desired representation of date/time/timestamp. By default, returns an iso-timestring.")
 
 (defvar +null-ptr+ (make-null-pointer :byte))
 (defparameter +null-handle-ptr+ (make-null-pointer :void))
@@ -39,13 +44,15 @@ as possible second argument) to the desired representation of date/time/timestam
   (let ((size (gensym)))
     `(let ((,size (length ,string)))
        (when (and ,max-length (> ,size ,max-length))
-         (error "string \"~a\" of length ~d is longer than max-length: ~d"
-                ,string ,size ,max-length))
+         (error 'clsql:sql-database-data-error
+                :message
+                (format nil "string \"~a\" of length ~d is longer than max-length: ~d"
+                        ,string ,size ,max-length)))
       (with-cast-pointer (char-ptr ,ptr :byte)
-       (dotimes (i ,size)
-         (setf (deref-array char-ptr '(:array :byte) i) 
-               (char-code (char ,string i))))
-       (setf (deref-array char-ptr '(:array :byte) ,size) 0)))))
+        (dotimes (i ,size)
+          (setf (deref-array char-ptr '(:array :byte) i)
+                (char-code (char ,string i))))
+        (setf (deref-array char-ptr '(:array :byte) ,size) 0)))))
 
 (defun %cstring-into-vector (ptr vector offset size-in-bytes)
   (dotimes (i size-in-bytes)
@@ -54,106 +61,130 @@ as possible second argument) to the desired representation of date/time/timestam
        (deref-array ptr '(:array :unsigned-char) i)))
     (incf offset))
   offset)
-  
+
 (defun handle-error (henv hdbc hstmt)
   (let ((sql-state (allocate-foreign-string 256))
-       (error-message (allocate-foreign-string #.$SQL_MAX_MESSAGE_LENGTH)))
-    (with-foreign-objects ((error-code :long)
-                          (msg-length :short))
+        (error-message (allocate-foreign-string #.$SQL_MAX_MESSAGE_LENGTH)))
+    (with-foreign-objects ((error-code #.$ODBC-LONG-TYPE)
+                           (msg-length :short))
       (SQLError henv hdbc hstmt sql-state
-               error-code error-message
-               #.$SQL_MAX_MESSAGE_LENGTH msg-length)
+                error-code error-message
+                #.$SQL_MAX_MESSAGE_LENGTH msg-length)
       (let ((err (convert-from-foreign-string error-message))
-           (state (convert-from-foreign-string sql-state)))
-       (free-foreign-object error-message)
-       (free-foreign-object sql-state)
-       (values
-        err
-        state
-        (deref-pointer msg-length :short) 
-        (deref-pointer error-code :long))))))
+            (state (convert-from-foreign-string sql-state)))
+        (free-foreign-object error-message)
+        (free-foreign-object sql-state)
+        (values
+         err
+         state
+         (deref-pointer msg-length :short)
+         (deref-pointer error-code #.$ODBC-LONG-TYPE))))))
 
 (defun sql-state (henv hdbc hstmt)
   (let ((sql-state (allocate-foreign-string 256))
-       (error-message (allocate-foreign-string #.$SQL_MAX_MESSAGE_LENGTH)))
-    (with-foreign-objects ((error-code :long)
-                          (msg-length :short))
+        (error-message (allocate-foreign-string #.$SQL_MAX_MESSAGE_LENGTH)))
+    (with-foreign-objects ((error-code #.$ODBC-LONG-TYPE)
+                           (msg-length :short))
       (SQLError henv hdbc hstmt sql-state error-code
-               error-message #.$SQL_MAX_MESSAGE_LENGTH msg-length)
-      (let ((state (convert-from-foreign-string sql-state))) 
-       (free-foreign-object error-message)
-       (free-foreign-object sql-state)
-       state
-       ;; test this: return a keyword for efficiency
-       ;;(%cstring-to-keyword state)
-       ))))
+                error-message #.$SQL_MAX_MESSAGE_LENGTH msg-length)
+      (let ((state (convert-from-foreign-string sql-state)))
+        (free-foreign-object error-message)
+        (free-foreign-object sql-state)
+        state
+        ;; test this: return a keyword for efficiency
+        ;;(%cstring-to-keyword state)
+        ))))
 
 (defmacro with-error-handling ((&key henv hdbc hstmt (print-info t))
                                    odbc-call &body body)
   (let ((result-code (gensym "RC-")))
     `(let ((,result-code ,odbc-call))
+
+      ;; Check for allegro v7 & v8 bug with ODBC calls returning
+      ;; 32-bit unsigned ints, not 16-bit signed ints
+      #+(and allegro mswindows)
+      (when (> ,result-code #xFFFF)
+        (warn (format nil "16-bit return bug: result-code #x~X for expression ~S"
+                      ,result-code (quote ,odbc-call)))
+        (setq ,result-code (logand ,result-code #xFFFF))
+        (when (> ,result-code #x7FFF)
+          (setq ,result-code (- ,result-code #x10000))))
+
        (case ,result-code
          (#.$SQL_SUCCESS
           (progn ,result-code ,@body))
          (#.$SQL_SUCCESS_WITH_INFO
           (when ,print-info
             (multiple-value-bind (error-message sql-state)
-               (handle-error (or ,henv +null-handle-ptr+)
-                             (or ,hdbc +null-handle-ptr+)
-                             (or ,hstmt +null-handle-ptr+))
-             (when *info-output*
-               (format *info-output* "[ODBC info ~A] ~A state: ~A"
-                       ,result-code error-message
-                       sql-state))))
+                (handle-error (or ,henv +null-handle-ptr+)
+                              (or ,hdbc +null-handle-ptr+)
+                              (or ,hstmt +null-handle-ptr+))
+              (when *info-output*
+                (format *info-output* "[ODBC info ~A] ~A state: ~A"
+                        ,result-code error-message
+                        sql-state))))
           (progn ,result-code ,@body))
          (#.$SQL_INVALID_HANDLE
           (error
-          'clsql-base-sys:clsql-odbc-error
-          :odbc-message "Invalid handle"))
+           'clsql-sys:sql-database-error
+           :message "ODBC: Invalid handle"))
          (#.$SQL_STILL_EXECUTING
           (error
-          'clsql-base-sys:clsql-odbc-error
-          :odbc-message "Still executing"))
+           'clsql-sys:sql-temporary-error
+           :message "ODBC: Still executing"))
          (#.$SQL_ERROR
           (multiple-value-bind (error-message sql-state)
-             (handle-error (or ,henv +null-handle-ptr+)
-                           (or ,hdbc +null-handle-ptr+)
-                           (or ,hstmt +null-handle-ptr+))
+              (handle-error (or ,henv +null-handle-ptr+)
+                            (or ,hdbc +null-handle-ptr+)
+                            (or ,hstmt +null-handle-ptr+))
             (error
-            'clsql-base-sys:clsql-odbc-error
-            :odbc-message error-message
-            :sql-state sql-state)))
-        (otherwise
+             'clsql-sys:sql-database-error
+             :message error-message
+             :secondary-error-id sql-state)))
+         (#.$SQL_NO_DATA_FOUND
+          (progn ,result-code ,@body))
+         ;; work-around for Allegro 7.0beta AMD64 which returns negative numbers
+         (otherwise
+          (multiple-value-bind (error-message sql-state)
+              (handle-error (or ,henv +null-handle-ptr+)
+                            (or ,hdbc +null-handle-ptr+)
+                            (or ,hstmt +null-handle-ptr+))
+            (error
+             'clsql-sys:sql-database-error
+             :message error-message
+             :secondary-error-id sql-state))
+          #+ignore
           (progn ,result-code ,@body))))))
 
 (defun %new-environment-handle ()
   (let ((henv
-        (with-foreign-object (phenv 'sql-handle)
-          (with-error-handling
-              ()
-            (SQLAllocEnv phenv)
-            (deref-pointer phenv 'sql-handle)))))
-    (%set-attr-odbc-version henv $SQL_OV_ODBC2)
+         (with-foreign-object (phenv 'sql-handle)
+           (with-error-handling
+               ()
+             (SQLAllocHandle $SQL_HANDLE_ENV +null-handle-ptr+ phenv)
+             (deref-pointer phenv 'sql-handle)))))
+    (%set-attr-odbc-version henv $SQL_OV_ODBC3)
     henv))
 
 
 (defun %sql-free-environment (henv)
-  (with-error-handling 
+  (with-error-handling
     (:henv henv)
     (SQLFreeEnv henv)))
 
 (defun %new-db-connection-handle (henv)
   (with-foreign-object (phdbc 'sql-handle)
+    (setf (deref-pointer phdbc 'sql-handle) +null-handle-ptr+)
     (with-error-handling
       (:henv henv)
-      (SQLAllocConnect henv phdbc)
+      (SQLAllocHandle $SQL_HANDLE_DBC henv phdbc)
       (deref-pointer phdbc 'sql-handle))))
 
 (defun %free-statement (hstmt option)
-  (with-error-handling 
+  (with-error-handling
       (:hstmt hstmt)
-      (SQLFreeStmt 
-       hstmt 
+      (SQLFreeStmt
+       hstmt
        (ecase option
          (:drop $SQL_DROP)
          (:close $SQL_CLOSE)
@@ -170,29 +201,43 @@ as possible second argument) to the desired representation of date/time/timestam
 
 (defun %sql-connect (hdbc server uid pwd)
   (with-cstrings ((server-ptr server)
-                 (uid-ptr uid)
-                 (pwd-ptr pwd))
-    (with-error-handling 
-       (:hdbc hdbc)
-      (SQLConnect hdbc server-ptr $SQL_NTS uid-ptr 
-                 $SQL_NTS pwd-ptr $SQL_NTS))))
-
+                  (uid-ptr uid)
+                  (pwd-ptr pwd))
+    (with-error-handling
+        (:hdbc hdbc)
+      (SQLConnect hdbc server-ptr $SQL_NTS uid-ptr
+                  $SQL_NTS pwd-ptr $SQL_NTS))))
+
+(defun %sql-driver-connect (hdbc connection-string completion window-handle)
+  (with-cstring (connection-ptr connection-string)
+    (let ((completed-connection-string (allocate-foreign-string $SQL_MAX_CONN_OUT)))
+      (unwind-protect
+          (with-foreign-object (completed-connection-length :short)
+                               (with-error-handling
+                                   (:hdbc hdbc)
+                                   (SQLDriverConnect hdbc
+                                                     window-handle
+                                                     connection-ptr $SQL_NTS
+                                                     completed-connection-string $SQL_MAX_CONN_OUT
+                                                     completed-connection-length
+                                                     completion)))
+        (free-foreign-object completed-connection-string)))))
 
 (defun %disconnect (hdbc)
-  (with-error-handling 
+  (with-error-handling
     (:hdbc hdbc)
     (SQLDisconnect hdbc)))
 
 (defun %commit (henv hdbc)
-  (with-error-handling 
+  (with-error-handling
     (:henv henv :hdbc hdbc)
-    (SQLTransact 
+    (SQLTransact
      henv hdbc $SQL_COMMIT)))
 
 (defun %rollback (henv hdbc)
-  (with-error-handling 
+  (with-error-handling
     (:henv henv :hdbc hdbc)
-    (SQLTransact 
+    (SQLTransact
      henv hdbc $SQL_ROLLBACK)))
 
 ; col-nr is zero-based in Lisp
@@ -211,7 +256,7 @@ as possible second argument) to the desired representation of date/time/timestam
   (with-error-handling
     (:hstmt hstmt)
     (SQLBindParameter hstmt (1+ parameter-nr)
-                      parameter-type ;$SQL_PARAM_INPUT 
+                      parameter-type ;$SQL_PARAM_INPUT
                       c-type ;$SQL_C_CHAR
                       sql-type ;$SQL_VARCHAR
                       precision ;(1- (length str))
@@ -222,16 +267,20 @@ as possible second argument) to the desired representation of date/time/timestam
                       )))
 
 (defun %sql-fetch (hstmt)
-  (with-error-handling 
+  (with-error-handling
       (:hstmt hstmt)
       (SQLFetch hstmt)))
 
 (defun %new-statement-handle (hdbc)
-  (with-foreign-object (hstmt-ptr 'sql-handle)
-    (with-error-handling 
-      (:hdbc hdbc)
-      (SQLAllocStmt hdbc hstmt-ptr) 
-      (deref-pointer hstmt-ptr 'sql-handle))))
+  (let ((statement-handle
+         (with-foreign-object (phstmt 'sql-handle)
+           (with-error-handling
+               (:hdbc hdbc)
+             (SQLAllocHandle $SQL_HANDLE_STMT hdbc phstmt)
+             (deref-pointer phstmt 'sql-handle)))))
+    (if (uffi:null-pointer-p statement-handle)
+        (error 'clsql:sql-database-error :message "Received null statement handle.")
+        statement-handle)))
 
 (defun %sql-get-info (hdbc info-type)
   (ecase info-type
@@ -269,14 +318,14 @@ as possible second argument) to the desired representation of date/time/timestam
       #.$SQL_SPECIAL_CHARACTERS
       #.$SQL_TABLE_TERM
       #.$SQL_USER_NAME)
-     (let ((info-ptr (allocate-foreign-string 1024))) 
+     (let ((info-ptr (allocate-foreign-string 1024)))
        (with-foreign-object (info-length-ptr :short)
-        (with-error-handling 
-            (:hdbc hdbc)
-            (SQLGetInfo hdbc info-type info-ptr 1023 info-length-ptr)
-          (let ((info (convert-from-foreign-string info-ptr)))
-            (free-foreign-object info-ptr)
-            info)))))
+         (with-error-handling
+             (:hdbc hdbc)
+             (SQLGetInfo hdbc info-type info-ptr 1023 info-length-ptr)
+           (let ((info (convert-from-foreign-string info-ptr)))
+             (free-foreign-object info-ptr)
+             info)))))
     ;; those returning a word
     ((#.$SQL_ACTIVE_CONNECTIONS
       #.$SQL_ACTIVE_STATEMENTS
@@ -306,18 +355,18 @@ as possible second argument) to the desired representation of date/time/timestam
       #.$SQL_QUOTED_IDENTIFIER_CASE
       #.$SQL_TXN_CAPABLE)
      (with-foreign-objects ((info-ptr :short)
-                           (info-length-ptr :short))
-       (with-error-handling 
-       (:hdbc hdbc)
+                            (info-length-ptr :short))
+       (with-error-handling
+        (:hdbc hdbc)
          (SQLGetInfo hdbc
-                    info-type
-                    info-ptr
-                    255
-                    info-length-ptr)
+                     info-type
+                     info-ptr
+                     255
+                     info-length-ptr)
          (deref-pointer info-ptr :short)))
      )
     ;; those returning a long bitmask
-    ((#.$SQL_ALTER_TABLE 
+    ((#.$SQL_ALTER_TABLE
       #.$SQL_BOOKMARK_PERSISTENCE
       #.$SQL_CONVERT_BIGINT
       #.$SQL_CONVERT_BINARY
@@ -362,16 +411,16 @@ as possible second argument) to the desired representation of date/time/timestam
       #.$SQL_TIMEDATE_FUNCTIONS
       #.$SQL_TXN_ISOLATION_OPTION
       #.$SQL_UNION)
-     (with-foreign-objects ((info-ptr :long)
-                           (info-length-ptr :short))
-       (with-error-handling 
+     (with-foreign-objects ((info-ptr #.$ODBC-LONG-TYPE)
+                            (info-length-ptr :short))
+       (with-error-handling
          (:hdbc hdbc)
          (SQLGetInfo hdbc
-                    info-type
-                    info-ptr
-                    255
-                    info-length-ptr)
-         (deref-pointer info-ptr :long)))
+                     info-type
+                     info-ptr
+                     255
+                     info-length-ptr)
+         (deref-pointer info-ptr #.$ODBC-LONG-TYPE)))
      )
     ;; those returning a long integer
     ((#.$SQL_DEFAULT_TXN_ISOLATION
@@ -385,13 +434,13 @@ as possible second argument) to the desired representation of date/time/timestam
       #.$SQL_MAX_CHAR_LITERAL_LEN
       #.$SQL_ACTIVE_ENVIRONMENTS
       )
-     (with-foreign-objects ((info-ptr :long)
-                           (info-length-ptr :short))
-       (with-error-handling 
+     (with-foreign-objects ((info-ptr #.$ODBC-LONG-TYPE)
+                            (info-length-ptr :short))
+       (with-error-handling
          (:hdbc hdbc)
          (SQLGetInfo hdbc info-type info-ptr 255 info-length-ptr)
-         (deref-pointer info-ptr :long))))))
-     
+         (deref-pointer info-ptr #.$ODBC-LONG-TYPE))))))
+
 (defun %sql-exec-direct (sql hstmt henv hdbc)
   (with-cstring (sql-ptr sql)
     (with-error-handling
@@ -415,127 +464,127 @@ as possible second argument) to the desired representation of date/time/timestam
       (deref-pointer columns-nr-ptr :short))))
 
 (defun result-rows-count (hstmt)
-  (with-foreign-objects ((row-count-ptr :long))
+  (with-foreign-objects ((row-count-ptr #.$ODBC-LONG-TYPE))
     (with-error-handling (:hstmt hstmt)
                          (SQLRowCount hstmt row-count-ptr)
-      (deref-pointer row-count-ptr :long))))
+      (deref-pointer row-count-ptr #.$ODBC-LONG-TYPE))))
 
 ;; column counting is 1-based
 (defun %describe-column (hstmt column-nr)
   (let ((column-name-ptr (allocate-foreign-string 256)))
     (with-foreign-objects ((column-name-length-ptr :short)
-                          (column-sql-type-ptr :short)
-                          (column-precision-ptr :unsigned-long)
-                          (column-scale-ptr :short)
-                          (column-nullable-p-ptr :short))
+                           (column-sql-type-ptr :short)
+                           (column-precision-ptr #.$ODBC-ULONG-TYPE)
+                           (column-scale-ptr :short)
+                           (column-nullable-p-ptr :short))
       (with-error-handling (:hstmt hstmt)
-         (SQLDescribeCol hstmt column-nr column-name-ptr 256
-                         column-name-length-ptr
-                         column-sql-type-ptr
-                         column-precision-ptr
-                         column-scale-ptr
-                         column-nullable-p-ptr)
-       (let ((column-name (convert-from-foreign-string column-name-ptr)))
-         (free-foreign-object column-name-ptr)
-         (values
-          column-name
-          (deref-pointer column-sql-type-ptr :short)
-          (deref-pointer column-precision-ptr :unsigned-long)
-          (deref-pointer column-scale-ptr :short)
-          (deref-pointer column-nullable-p-ptr :short)))))))
-    
+          (SQLDescribeCol hstmt column-nr column-name-ptr 256
+                          column-name-length-ptr
+                          column-sql-type-ptr
+                          column-precision-ptr
+                          column-scale-ptr
+                          column-nullable-p-ptr)
+        (let ((column-name (convert-from-foreign-string column-name-ptr)))
+          (free-foreign-object column-name-ptr)
+          (values
+           column-name
+           (deref-pointer column-sql-type-ptr :short)
+           (deref-pointer column-precision-ptr #.$ODBC-ULONG-TYPE)
+           (deref-pointer column-scale-ptr :short)
+           (deref-pointer column-nullable-p-ptr :short)))))))
+
 ;; parameter counting is 1-based
 (defun %describe-parameter (hstmt parameter-nr)
   (with-foreign-objects ((column-sql-type-ptr :short)
-                        (column-precision-ptr :unsigned-long)
-                        (column-scale-ptr :short)
-                        (column-nullable-p-ptr :short))
-    (with-error-handling 
+                         (column-precision-ptr #.$ODBC-ULONG-TYPE)
+                         (column-scale-ptr :short)
+                         (column-nullable-p-ptr :short))
+    (with-error-handling
       (:hstmt hstmt)
       (SQLDescribeParam hstmt parameter-nr
-                       column-sql-type-ptr
+                        column-sql-type-ptr
                         column-precision-ptr
-                       column-scale-ptr
+                        column-scale-ptr
                         column-nullable-p-ptr)
       (values
        (deref-pointer column-sql-type-ptr :short)
-       (deref-pointer column-precision-ptr :unsigned-long)
+       (deref-pointer column-precision-ptr #.$ODBC-ULONG-TYPE)
        (deref-pointer column-scale-ptr :short)
        (deref-pointer column-nullable-p-ptr :short)))))
 
 (defun %column-attributes (hstmt column-nr descriptor-type)
   (let ((descriptor-info-ptr (allocate-foreign-string 256)))
     (with-foreign-objects ((descriptor-length-ptr :short)
-                          (numeric-descriptor-ptr :long))
+                           (numeric-descriptor-ptr #.$ODBC-LONG-TYPE))
       (with-error-handling
-         (:hstmt hstmt) 
-         (SQLColAttributes hstmt column-nr descriptor-type descriptor-info-ptr
-                           256 descriptor-length-ptr
-                           numeric-descriptor-ptr)
-       (let ((desc (convert-from-foreign-string descriptor-info-ptr)))
-         (free-foreign-object descriptor-info-ptr)
-         (values
-          desc
-          (deref-pointer numeric-descriptor-ptr :long)))))))
-  
-(defun %prepare-describe-columns (hstmt table-qualifier table-owner 
+          (:hstmt hstmt)
+          (SQLColAttributes hstmt column-nr descriptor-type descriptor-info-ptr
+                            256 descriptor-length-ptr
+                            numeric-descriptor-ptr)
+        (let ((desc (convert-from-foreign-string descriptor-info-ptr)))
+          (free-foreign-object descriptor-info-ptr)
+          (values
+           desc
+           (deref-pointer numeric-descriptor-ptr #.$ODBC-LONG-TYPE)))))))
+
+(defun %prepare-describe-columns (hstmt table-qualifier table-owner
                                    table-name column-name)
   (with-cstrings ((table-qualifier-ptr table-qualifier)
-                 (table-owner-ptr table-owner) 
-                 (table-name-ptr table-name)
-                 (column-name-ptr column-name))
+                  (table-owner-ptr table-owner)
+                  (table-name-ptr table-name)
+                  (column-name-ptr column-name))
     (with-error-handling
-       (:hstmt hstmt) 
+        (:hstmt hstmt)
       (SQLColumns hstmt
-                 table-qualifier-ptr (length table-qualifier)
-                 table-owner-ptr (length table-owner)
-                 table-name-ptr (length table-name)
-                 column-name-ptr (length column-name)))))
+                  table-qualifier-ptr (length table-qualifier)
+                  table-owner-ptr (length table-owner)
+                  table-name-ptr (length table-name)
+                  column-name-ptr (length column-name)))))
 
-(defun %describe-columns (hdbc table-qualifier table-owner 
-                         table-name column-name)
+(defun %describe-columns (hdbc table-qualifier table-owner
+                          table-name column-name)
   (with-statement-handle (hstmt hdbc)
-    (%prepare-describe-columns hstmt table-qualifier table-owner 
+    (%prepare-describe-columns hstmt table-qualifier table-owner
                                table-name column-name)
     (fetch-all-rows hstmt)))
 
 (defun %sql-data-sources (henv &key (direction :first))
   (let ((name-ptr (allocate-foreign-string (1+ $SQL_MAX_DSN_LENGTH)))
-       (description-ptr (allocate-foreign-string 1024)))
+        (description-ptr (allocate-foreign-string 1024)))
     (with-foreign-objects ((name-length-ptr :short)
-                          (description-length-ptr :short))
+                           (description-length-ptr :short))
       (let ((res (with-error-handling
-                    (:henv henv)
-                    (SQLDataSources henv
-                                    (ecase direction
-                                      (:first $SQL_FETCH_FIRST)
-                                      (:next $SQL_FETCH_NEXT))
-                                    name-ptr
-                                    (1+ $SQL_MAX_DSN_LENGTH)
-                                    name-length-ptr
-                                    description-ptr
-                                    1024
-                                    description-length-ptr))))
-       (cond
-         ((= res $SQL_NO_DATA_FOUND)
-          (let ((name (convert-from-foreign-string name-ptr))
-                (desc (convert-from-foreign-string description-ptr)))
-            (free-foreign-object name-ptr)
-            (free-foreign-object description-ptr)
-            (values
-             name
-             desc)))
-         (t
-          (free-foreign-object name-ptr)
-          (free-foreign-object description-ptr)
-          nil))))))
-          
+                     (:henv henv)
+                     (SQLDataSources henv
+                                     (ecase direction
+                                       (:first $SQL_FETCH_FIRST)
+                                       (:next $SQL_FETCH_NEXT))
+                                     name-ptr
+                                     (1+ $SQL_MAX_DSN_LENGTH)
+                                     name-length-ptr
+                                     description-ptr
+                                     1024
+                                     description-length-ptr))))
+        (cond
+          ((= res $SQL_NO_DATA_FOUND)
+           (let ((name (convert-from-foreign-string name-ptr))
+                 (desc (convert-from-foreign-string description-ptr)))
+             (free-foreign-object name-ptr)
+             (free-foreign-object description-ptr)
+             (values
+              name
+              desc)))
+          (t
+           (free-foreign-object name-ptr)
+           (free-foreign-object description-ptr)
+           nil))))))
+
 
 
 (defun sql-to-c-type (sql-type)
   (ecase sql-type
-    ((#.$SQL_CHAR #.$SQL_VARCHAR #.$SQL_LONGVARCHAR 
-      #.$SQL_NUMERIC #.$SQL_DECIMAL #.$SQL_BIGINT -8 -9) $SQL_C_CHAR)
+    ((#.$SQL_CHAR #.$SQL_VARCHAR #.$SQL_LONGVARCHAR
+      #.$SQL_NUMERIC #.$SQL_DECIMAL #.$SQL_BIGINT -8 -9 -10) $SQL_C_CHAR) ;; Added -10 for MSSQL ntext type
     (#.$SQL_INTEGER $SQL_C_SLONG)
     (#.$SQL_SMALLINT $SQL_C_SSHORT)
     (#.$SQL_DOUBLE $SQL_C_DOUBLE)
@@ -544,17 +593,20 @@ as possible second argument) to the desired representation of date/time/timestam
     (#.$SQL_DATE $SQL_C_DATE)
     (#.$SQL_TIME $SQL_C_TIME)
     (#.$SQL_TIMESTAMP $SQL_C_TIMESTAMP)
+    (#.$SQL_TYPE_DATE $SQL_C_TYPE_DATE)
+    (#.$SQL_TYPE_TIME $SQL_C_TYPE_TIME)
+    (#.$SQL_TYPE_TIMESTAMP $SQL_C_TYPE_TIMESTAMP)
     ((#.$SQL_BINARY #.$SQL_VARBINARY #.$SQL_LONGVARBINARY) $SQL_C_BINARY)
     (#.$SQL_TINYINT $SQL_C_STINYINT)
     (#.$SQL_BIT $SQL_C_BIT)))
 
-(def-type byte-pointer-type '(* :byte))
-(def-type short-pointer-type '(* :short))
-(def-type int-pointer-type '(* :int))
-(def-type long-pointer-type '(* :long))
-(def-type float-pointer-type '(* :float))
-(def-type double-pointer-type '(* :double))
-(def-type string-pointer-type '(* :unsigned-char))
+(def-type byte-pointer-type (* :byte))
+(def-type short-pointer-type (* :short))
+(def-type int-pointer-type (* :int))
+(def-type long-pointer-type (* #.$ODBC-LONG-TYPE))
+(def-type float-pointer-type (* :float))
+(def-type double-pointer-type (* :double))
+(def-type string-pointer-type (* :unsigned-char))
 
 (defun get-cast-byte (ptr)
   (locally (declare (type byte-pointer-type ptr))
@@ -570,7 +622,7 @@ as possible second argument) to the desired representation of date/time/timestam
 
 (defun get-cast-long (ptr)
   (locally (declare (type long-pointer-type ptr))
-    (deref-pointer ptr :long)))
+    (deref-pointer ptr #.$ODBC-LONG-TYPE)))
 
 (defun get-cast-single-float (ptr)
   (locally (declare (type float-pointer-type ptr))
@@ -590,76 +642,77 @@ as possible second argument) to the desired representation of date/time/timestam
     (ecase format
       (:unsigned-byte-vector
        (let ((vector (make-array len :element-type '(unsigned-byte 8))))
-        (dotimes (i len)
-          (setf (aref vector i)
-                (deref-array casted '(:array :byte) i)))
-        vector))
+         (dotimes (i len)
+           (setf (aref vector i)
+                 (deref-array casted '(:array :byte) i)))
+         vector))
       (:bit-vector
        (let ((vector (make-array (ash len 3) :element-type 'bit)))
-        (dotimes (i len)
-          (let ((byte (deref-array casted '(:array :byte) i)))
-            (dotimes (j 8)
-              (setf (bit vector (+ (ash i 3) j))
-                    (logand (ash byte (- j 7)) 1)))))
-        vector)))))
+         (dotimes (i len)
+           (let ((byte (deref-array casted '(:array :byte) i)))
+             (dotimes (j 8)
+               (setf (bit vector (+ (ash i 3) j))
+                     (logand (ash byte (- j 7)) 1)))))
+         vector)))))
 
 
 (defun read-data (data-ptr c-type sql-type out-len-ptr result-type)
   (declare (type long-ptr-type out-len-ptr))
-  (let* ((out-len (deref-pointer out-len-ptr :long))
-        (value
-         (cond ((= out-len $SQL_NULL_DATA)
-                *null*)
-               (t
-                (case sql-type
-                  ;; SQL extended datatypes
-                  (#.$SQL_TINYINT  (get-cast-byte data-ptr))
-                  (#.$SQL_C_STINYINT (get-cast-byte data-ptr)) ;; ?
-                  (#.$SQL_C_SSHORT (get-cast-short data-ptr)) ;; ?
-                  (#.$SQL_SMALLINT (get-cast-short data-ptr)) ;; ??
-                  (#.$SQL_INTEGER (get-cast-int data-ptr))
-                  (#.$SQL_BIGINT (read-from-string
-                                  (get-cast-foreign-string data-ptr)))
-                  (#.$SQL_TINYINT (get-cast-byte data-ptr))
-                  (#.$SQL_DECIMAL 
-                   (let ((*read-base* 10))
-                     (read-from-string (get-cast-foreign-string data-ptr))))
-                  (t 
-                   (case c-type
-                     (#.$SQL_C_DATE
-                      (funcall *time-conversion-function* (date-to-universal-time data-ptr)))
-                     (#.$SQL_C_TIME
-                      (multiple-value-bind (universal-time frac) (time-to-universal-time data-ptr)
-                        (funcall *time-conversion-function* universal-time frac)))
-                     (#.$SQL_C_TIMESTAMP
-                      (multiple-value-bind (universal-time frac) (timestamp-to-universal-time data-ptr)
-                        (funcall *time-conversion-function* universal-time frac)))
-                     (#.$SQL_INTEGER
-                      (get-cast-int data-ptr))
-                     (#.$SQL_C_FLOAT
-                      (get-cast-single-float data-ptr))
-                     (#.$SQL_C_DOUBLE
-                      (get-cast-double-float data-ptr))
-                     (#.$SQL_C_SLONG
-                      (get-cast-long data-ptr))
-                     #+lispworks
-                     (#.$SQL_C_BIT     ; encountered only in Access
-                      (get-cast-byte data-ptr))
-                     (#.$SQL_C_BINARY
-                      (get-cast-binary data-ptr out-len *binary-format*))
-                     ((#.$SQL_C_SSHORT #.$SQL_C_STINYINT) ; LMH short ints
-                      (get-cast-short data-ptr)) ; LMH
-                     #+ignore
-                     (#.$SQL_C_CHAR
-                      (code-char (get-cast-short data-ptr)))
-                     (t
-                      (get-cast-foreign-string data-ptr)))))))))
-    
+  (let* ((out-len (deref-pointer out-len-ptr #.$ODBC-LONG-TYPE))
+         (value
+          (cond ((= out-len $SQL_NULL_DATA)
+                 *null*)
+                (t
+                 (case sql-type
+                   ;; SQL extended datatypes
+                   (#.$SQL_TINYINT  (get-cast-byte data-ptr))
+                   (#.$SQL_C_STINYINT (get-cast-byte data-ptr)) ;; ?
+                   (#.$SQL_C_SSHORT (get-cast-short data-ptr)) ;; ?
+                   (#.$SQL_SMALLINT (get-cast-short data-ptr)) ;; ??
+                   (#.$SQL_INTEGER (get-cast-int data-ptr))
+                   (#.$SQL_BIGINT (read-from-string
+                                   (get-cast-foreign-string data-ptr)))
+                   (#.$SQL_DECIMAL
+                    (let ((*read-base* 10))
+                      (read-from-string (get-cast-foreign-string data-ptr))))
+                   (#.$SQL_BIT (get-cast-byte data-ptr))
+                   (t
+                    (case c-type
+                      ((#.$SQL_C_DATE #.$SQL_C_TYPE_DATE)
+                       (funcall *time-conversion-function* (date-to-universal-time data-ptr)))
+                      ((#.$SQL_C_TIME #.$SQL_C_TYPE_TIME)
+                       (multiple-value-bind (universal-time frac) (time-to-universal-time data-ptr)
+                         (funcall *time-conversion-function* universal-time frac)))
+                      ((#.$SQL_C_TIMESTAMP #.$SQL_C_TYPE_TIMESTAMP)
+                       (multiple-value-bind (universal-time frac) (timestamp-to-universal-time data-ptr)
+                         (funcall *time-conversion-function* universal-time frac)))
+                      (#.$SQL_INTEGER
+                       (get-cast-int data-ptr))
+                      (#.$SQL_C_FLOAT
+                       (get-cast-single-float data-ptr))
+                      (#.$SQL_C_DOUBLE
+                       (get-cast-double-float data-ptr))
+                      (#.$SQL_C_SLONG
+                       (get-cast-long data-ptr))
+                      #+lispworks
+                      (#.$SQL_C_BIT     ; encountered only in Access
+                       (get-cast-byte data-ptr))
+                      (#.$SQL_C_BINARY
+                       (get-cast-binary data-ptr out-len *binary-format*))
+                      ((#.$SQL_C_SSHORT #.$SQL_C_STINYINT) ; LMH short ints
+                       (get-cast-short data-ptr)) ; LMH
+                      #+ignore
+                      (#.$SQL_C_CHAR
+                       (code-char (get-cast-short data-ptr)))
+                      (t
+                       (get-cast-foreign-string data-ptr)))))))))
+
     ;; FIXME: this could be better optimized for types which use READ-FROM-STRING above
-    
+
     (if (and (or (eq result-type t) (eq result-type :string))
-            (not (stringp value)))
-       (write-to-string value)
+             value
+             (not (stringp value)))
+        (write-to-string value)
       value)))
 
 ;; which value is appropriate?
@@ -679,24 +732,24 @@ as possible second argument) to the desired representation of date/time/timestam
          (long-p (= size +max-precision+))
          (data-ptr
           (case c-type ;; add more?
-            (#.$SQL_C_SLONG (uffi:allocate-foreign-object :long))
-            (#.$SQL_C_DATE (allocate-foreign-object 'sql-c-date))
-            (#.$SQL_C_TIME (allocate-foreign-object 'sql-c-time))
-            (#.$SQL_C_TIMESTAMP (allocate-foreign-object 'sql-c-timestamp))
-           (#.$SQL_C_FLOAT (uffi:allocate-foreign-object :float))
+            (#.$SQL_C_SLONG (uffi:allocate-foreign-object #.$ODBC-LONG-TYPE))
+            ((#.$SQL_C_DATE #.$SQL_C_TYPE_DATE) (allocate-foreign-object 'sql-c-date))
+            ((#.$SQL_C_TIME #.$SQL_C_TYPE_TIME) (allocate-foreign-object 'sql-c-time))
+            ((#.$SQL_C_TIMESTAMP #.$SQL_C_TYPE_TIMESTAMP) (allocate-foreign-object 'sql-c-timestamp))
+            (#.$SQL_C_FLOAT (uffi:allocate-foreign-object :float))
             (#.$SQL_C_DOUBLE (uffi:allocate-foreign-object :double))
             (#.$SQL_C_BIT (uffi:allocate-foreign-object :byte))
-            (#.$SQL_C_STINYINT (uffi:allocate-foreign-object :short))
+            (#.$SQL_C_STINYINT (uffi:allocate-foreign-object :byte))
             (#.$SQL_C_SSHORT (uffi:allocate-foreign-object :short))
             (#.$SQL_C_CHAR (uffi:allocate-foreign-string (1+ size)))
             (#.$SQL_C_BINARY (uffi:allocate-foreign-string (1+ (* 2 size))))
-            (t 
+            (t
                 ;; Maybe should signal a restartable condition for this?
                 (when *break-on-unknown-data-type*
-                  (break "SQL type is ~A, precision ~D, size ~D, C type is ~A" 
+                  (break "SQL type is ~A, precision ~D, size ~D, C type is ~A"
                          sql-type precision size c-type))
                 (uffi:allocate-foreign-object :byte (1+ size)))))
-         (out-len-ptr (uffi:allocate-foreign-object :long)))
+         (out-len-ptr (uffi:allocate-foreign-object #.$ODBC-LONG-TYPE)))
     (values c-type data-ptr out-len-ptr size long-p)))
 
 (defun fetch-all-rows (hstmt &key free-option flatp)
@@ -731,12 +784,13 @@ as possible second argument) to the desired representation of date/time/timestam
                          (aref out-len-ptrs col-nr) out-len-ptr))))
              ;; the main loop
              (prog1
-               (cond (flatp 
+               (cond (flatp
                       (when (> column-count 1)
-                        (error "If more than one column is to be fetched, flatp has to be nil."))
+                        (error 'clsql:sql-database-error
+                               :message "If more than one column is to be fetched, flatp has to be nil."))
                       (loop until (= (%sql-fetch hstmt) $SQL_NO_DATA_FOUND)
                             collect
-                            (read-data (aref data-ptrs 0) 
+                            (read-data (aref data-ptrs 0)
                                        (aref c-types 0)
                                        (aref sql-types 0)
                                        (aref out-len-ptrs 0)
@@ -746,7 +800,7 @@ as possible second argument) to the desired representation of date/time/timestam
                             collect
                             (loop for col-nr from 0 to (1- column-count)
                                   collect
-                                  (read-data (aref data-ptrs col-nr) 
+                                  (read-data (aref data-ptrs col-nr)
                                              (aref c-types col-nr)
                                              (aref sql-types col-nr)
                                              (aref out-len-ptrs col-nr)
@@ -770,10 +824,10 @@ as possible second argument) to the desired representation of date/time/timestam
 
 ;; depending on option, we return a long int or a string; string not implemented
 (defun get-connection-option (hdbc option)
-  (with-foreign-object (param-ptr :long)
+  (with-foreign-object (param-ptr #.$ODBC-LONG-TYPE)
     (with-error-handling (:hdbc hdbc)
                          (SQLGetConnectOption hdbc option param-ptr)
-      (deref-pointer param-ptr :long))))
+      (deref-pointer param-ptr #.$ODBC-LONG-TYPE))))
 
 (defun set-connection-option (hdbc option param)
   (with-error-handling (:hdbc hdbc)
@@ -786,17 +840,17 @@ as possible second argument) to the desired representation of date/time/timestam
   (set-connection-option hdbc $SQL_AUTOCOMMIT $SQL_AUTOCOMMIT_ON))
 
 (defun %sql-set-pos (hstmt row option lock)
-  (with-error-handling 
+  (with-error-handling
     (:hstmt hstmt)
     (SQLSetPos hstmt row option lock)))
 
 (defun %sql-extended-fetch (hstmt fetch-type row)
-  (with-foreign-objects ((row-count-ptr :unsigned-long)
-                        (row-status-ptr :short))
+  (with-foreign-objects ((row-count-ptr #.$ODBC-ULONG-TYPE)
+                         (row-status-ptr :short))
     (with-error-handling (:hstmt hstmt)
       (SQLExtendedFetch hstmt fetch-type row row-count-ptr
-                       row-status-ptr)
-      (values (deref-pointer row-count-ptr :unsigned-long)
+                        row-status-ptr)
+      (values (deref-pointer row-count-ptr #.$ODBC-ULONG-TYPE)
               (deref-pointer row-status-ptr :short)))))
 
 ; column-nr is zero-based
@@ -817,67 +871,70 @@ as possible second argument) to the desired representation of date/time/timestam
 
 (defconstant $sql-data-truncated (intern "01004" :keyword))
 
-(defun read-data-in-chunks (hstmt column-nr data-ptr c-type sql-type 
+(defun read-data-in-chunks (hstmt column-nr data-ptr c-type sql-type
                                       out-len-ptr result-type)
-  (declare (type long-ptr-type out-len-ptr))
-  (let* ((res (%sql-get-data hstmt column-nr c-type data-ptr 
+  (declare (type long-ptr-type out-len-ptr)
+           (ignore result-type))
+  (let* ((res (%sql-get-data hstmt column-nr c-type data-ptr
                              +max-precision+ out-len-ptr))
-         (out-len (deref-pointer out-len-ptr :long))
-         (offset 0))
-    (case out-len
-      (#.$SQL_NULL_DATA
-       (return-from read-data-in-chunks *null*))
-      (#.$SQL_NO_TOTAL ;; don't know how long it is going to be
-       (let ((str (make-array 0 :element-type 'character :adjustable t)))
-         (loop do (if (= c-type #.$SQL_CHAR)
-                      (let ((data-length (foreign-string-length data-ptr)))
-                        (adjust-array str (+ offset data-length)
-                                      :initial-element #\?)
-                        (setf offset (%cstring-into-vector
-                                      data-ptr str 
-                                      offset 
-                                      data-length)))
-                    (error "wrong type. preliminary."))
-               while (and (= res $SQL_SUCCESS_WITH_INFO)
-                          (equal (sql-state +null-handle-ptr+ +null-handle-ptr+ hstmt)
-                                 "01004"))
-               do (setf res (%sql-get-data hstmt column-nr c-type data-ptr 
-                                           +max-precision+ out-len-ptr)))
-         (setf str (coerce str 'string))
-         (if (= sql-type $SQL_DECIMAL)
-             (let ((*read-base* 10))
-               (read-from-string str))
-           str)))
-      (otherwise
-       (let ((str (make-string out-len)))
-         (loop do (if (= c-type #.$SQL_CHAR)
-                      (setf offset (%cstring-into-vector ;string
-                                    data-ptr str 
-                                    offset 
-                                    (min out-len (1- +max-precision+))))
-                    (error "wrong type. preliminary."))
-               while 
-               (and (= res $SQL_SUCCESS_WITH_INFO)
-                    #+ingore(eq (sql-state +null-handle-ptr+ +null-handle-ptr+ hstmt)
-                                $sql-data-truncated)
-                    (equal (sql-state +null-handle-ptr+ +null-handle-ptr+ hstmt)
-                           "01004"))
-               do (setf res (%sql-get-data hstmt column-nr c-type data-ptr 
-                                           +max-precision+ out-len-ptr)
-                        out-len (deref-pointer out-len-ptr :long)))
-         (if (= sql-type $SQL_DECIMAL)
-             (let ((*read-base* 10))
-               (read-from-string str))
-           str))))))
-
-(def-type c-timestamp-ptr-type '(* (:struct sql-c-timestamp)))
-(def-type c-time-ptr-type '(* (:struct sql-c-time)))
-(def-type c-date-ptr-type '(* (:struct sql-c-date)))
+         (out-len (deref-pointer out-len-ptr #.$ODBC-LONG-TYPE))
+         (offset 0)
+         (result (case out-len
+                   (#.$SQL_NULL_DATA
+                    (return-from read-data-in-chunks *null*))
+                   (#.$SQL_NO_TOTAL ;; don't know how long it is going to be
+                                    (let ((str (make-array 0 :element-type 'character :adjustable t)))
+                                      (loop do (if (= c-type #.$SQL_CHAR)
+                                                   (let ((data-length (foreign-string-length data-ptr)))
+                                                     (adjust-array str (+ offset data-length)
+                                                                   :initial-element #\?)
+                                                     (setf offset (%cstring-into-vector
+                                                                   data-ptr str
+                                                                   offset
+                                                                   data-length)))
+                                                 (error 'clsql:sql-database-error :message "wrong type. preliminary."))
+                                            while (and (= res $SQL_SUCCESS_WITH_INFO)
+                                                       (equal (sql-state +null-handle-ptr+ +null-handle-ptr+ hstmt)
+                                                              "01004"))
+                                            do (setf res (%sql-get-data hstmt column-nr c-type data-ptr
+                                                                        +max-precision+ out-len-ptr)))
+                                      (setf str (coerce str 'string))
+                                      (if (= sql-type $SQL_DECIMAL)
+                                          (let ((*read-base* 10))
+                                            (read-from-string str))
+                                        str)))
+                   (otherwise
+                    (let ((str (make-string out-len)))
+                      (loop do (if (= c-type #.$SQL_CHAR)
+                                   (setf offset (%cstring-into-vector ;string
+                                                 data-ptr str
+                                                 offset
+                                                 (min out-len (1- +max-precision+))))
+                                 (error 'clsql:sql-database-error :message "wrong type. preliminary."))
+                            while
+                            (and (= res $SQL_SUCCESS_WITH_INFO)
+                                 #+ingore(eq (sql-state +null-handle-ptr+ +null-handle-ptr+ hstmt)
+                                             $sql-data-truncated)
+                                 (equal (sql-state +null-handle-ptr+ +null-handle-ptr+ hstmt)
+                                        "01004"))
+                            do (setf res (%sql-get-data hstmt column-nr c-type data-ptr
+                                                        +max-precision+ out-len-ptr)
+                                     out-len (deref-pointer out-len-ptr #.$ODBC-LONG-TYPE)))
+                      (if (= sql-type $SQL_DECIMAL)
+                          (let ((*read-base* 10))
+                            (read-from-string str))
+                        str))))))
+    (setf (deref-pointer out-len-ptr #.$ODBC-LONG-TYPE) #.$SQL_NO_TOTAL) ;; reset the out length for the next row
+    result))
+
+(def-type c-timestamp-ptr-type (* (:struct sql-c-timestamp)))
+(def-type c-time-ptr-type (* (:struct sql-c-time)))
+(def-type c-date-ptr-type (* (:struct sql-c-date)))
 
 (defun timestamp-to-universal-time (ptr)
   (declare (type c-timestamp-ptr-type ptr))
   (values
-   (encode-universal-time 
+   (encode-universal-time
     (get-slot-value ptr 'sql-c-timestamp 'second)
     (get-slot-value ptr 'sql-c-timestamp 'minute)
     (get-slot-value ptr 'sql-c-timestamp 'hour)
@@ -922,7 +979,7 @@ as possible second argument) to the desired representation of date/time/timestam
 
 (defun time-to-universal-time (ptr)
   (declare (type c-time-ptr-type ptr))
-  (encode-universal-time 
+  (encode-universal-time
    (get-slot-value ptr 'sql-c-timestamp 'second)
    (get-slot-value ptr 'sql-c-timestamp 'minute)
    (get-slot-value ptr 'sql-c-timestamp 'hour)
@@ -933,9 +990,52 @@ as possible second argument) to the desired representation of date/time/timestam
 
 (defun %set-attr-odbc-version (henv version)
   (with-error-handling (:henv henv)
-      (SQLSetEnvAttr henv $SQL_ATTR_ODBC_VERSION 
-                    (make-pointer version :void) 0)))
+      (SQLSetEnvAttr henv $SQL_ATTR_ODBC_VERSION
+                     (make-pointer version :void) 0)))
 
 (defun %list-tables (hstmt)
   (with-error-handling (:hstmt hstmt)
     (SQLTables hstmt +null-ptr+ 0 +null-ptr+ 0 +null-ptr+ 0 +null-ptr+ 0)))
+
+(defun %table-statistics (table hstmt &key unique (ensure t))
+  (with-cstrings ((table-cs table))
+    (with-error-handling (:hstmt hstmt)
+      (SQLStatistics
+       hstmt
+       +null-ptr+ 0
+       +null-ptr+ 0
+       table-cs $SQL_NTS
+       (if unique $SQL_INDEX_UNIQUE $SQL_INDEX_ALL)
+       (if ensure $SQL_ENSURE $SQL_QUICK)))))
+
+(defun %list-data-sources (henv)
+  (let ((dsn (allocate-foreign-string (1+ $SQL_MAX_DSN_LENGTH)))
+        (desc (allocate-foreign-string 256))
+        (results nil))
+    (unwind-protect
+         (with-foreign-objects ((dsn-len :short)
+                                (desc-len :short))
+           (let ((res (with-error-handling (:henv henv)
+                        (SQLDataSources henv $SQL_FETCH_FIRST dsn
+                                        (1+ $SQL_MAX_DSN_LENGTH)
+                                        dsn-len desc 256 desc-len))))
+             (when (or (eql res $SQL_SUCCESS)
+                       (eql res $SQL_SUCCESS_WITH_INFO))
+               (push (convert-from-foreign-string dsn) results))
+
+             (do ((res (with-error-handling (:henv henv)
+                         (SQLDataSources henv $SQL_FETCH_NEXT dsn
+                                         (1+ $SQL_MAX_DSN_LENGTH)
+                                         dsn-len desc 256 desc-len))
+                       (with-error-handling (:henv henv)
+                         (SQLDataSources henv $SQL_FETCH_NEXT dsn
+                                         (1+ $SQL_MAX_DSN_LENGTH)
+                                         dsn-len desc 256 desc-len))))
+                 ((not (or (eql res $SQL_SUCCESS)
+                           (eql res $SQL_SUCCESS_WITH_INFO))))
+               (push (convert-from-foreign-string dsn) results))))
+      (progn
+        (free-foreign-object dsn)
+        (free-foreign-object desc)))
+    (nreverse results)))
+