Blame SOURCES/coverity-fixes.patch

caac36
Fix assorted small errors detected by a Coverity scan, per bug #760877.
caac36
caac36
caac36
diff -Naur unixODBC-2.3.1.orig/DriverManager/SQLGetCursorName.c unixODBC-2.3.1/DriverManager/SQLGetCursorName.c
caac36
--- unixODBC-2.3.1.orig/DriverManager/SQLGetCursorName.c	2011-08-04 09:06:56.000000000 -0400
caac36
+++ unixODBC-2.3.1/DriverManager/SQLGetCursorName.c	2013-03-19 13:12:19.022156796 -0400
caac36
@@ -189,7 +189,7 @@
caac36
 
caac36
     thread_protect( SQL_HANDLE_STMT, statement );
caac36
 
caac36
-    if ( !buffer_length < 0 )
caac36
+    if ( buffer_length < 0 )
caac36
     {
caac36
         dm_log_write( __FILE__, 
caac36
                 __LINE__, 
caac36
diff -Naur unixODBC-2.3.1.orig/DriverManager/SQLGetCursorNameW.c unixODBC-2.3.1/DriverManager/SQLGetCursorNameW.c
caac36
--- unixODBC-2.3.1.orig/DriverManager/SQLGetCursorNameW.c	2011-08-04 09:07:01.000000000 -0400
caac36
+++ unixODBC-2.3.1/DriverManager/SQLGetCursorNameW.c	2013-03-19 13:12:19.023156773 -0400
caac36
@@ -172,7 +172,7 @@
caac36
 
caac36
     thread_protect( SQL_HANDLE_STMT, statement );
caac36
 
caac36
-    if ( !buffer_length < 0 )
caac36
+    if ( buffer_length < 0 )
caac36
     {
caac36
         dm_log_write( __FILE__, 
caac36
                 __LINE__, 
caac36
diff -Naur unixODBC-2.3.1.orig/DriverManager/__info.c unixODBC-2.3.1/DriverManager/__info.c
caac36
--- unixODBC-2.3.1.orig/DriverManager/__info.c	2011-11-15 06:43:15.000000000 -0500
caac36
+++ unixODBC-2.3.1/DriverManager/__info.c	2013-03-19 13:12:19.022156796 -0400
caac36
@@ -3997,7 +3997,7 @@
caac36
 
caac36
 void setup_error_head( EHEAD *error_header, void *handle, int type )
caac36
 {
caac36
-    memset( error_header, 0, sizeof( error_header ));
caac36
+    memset( error_header, 0, sizeof( *error_header ));
caac36
 
caac36
     error_header -> owning_handle = handle;
caac36
     error_header -> handle_type = type;
caac36
diff -Naur unixODBC-2.3.1.orig/Drivers/Postgre7.1/info.c unixODBC-2.3.1/Drivers/Postgre7.1/info.c
caac36
--- unixODBC-2.3.1.orig/Drivers/Postgre7.1/info.c	2009-12-11 11:54:08.000000000 -0500
caac36
+++ unixODBC-2.3.1/Drivers/Postgre7.1/info.c	2013-03-19 13:12:19.025156733 -0400
caac36
@@ -1007,7 +1007,7 @@
caac36
 	strcpy(prefixes, globals.extra_systable_prefixes);
caac36
 	i = 0;
caac36
 	prefix[i] = strtok(prefixes, ";");
caac36
-	while (prefix[i] && i<32) {
caac36
+	while (prefix[i] && i
caac36
 		prefix[++i] = strtok(NULL, ";");
caac36
 	}
caac36
 
caac36
@@ -1023,7 +1023,7 @@
caac36
 		free(tableType);
caac36
 		i = 0;
caac36
 		table_type[i] = strtok(table_types, ",");
caac36
-		while (table_type[i] && i<32) {
caac36
+		while (table_type[i] && i
caac36
 			table_type[++i] = strtok(NULL, ",");
caac36
 		}
caac36
 
caac36
diff -Naur unixODBC-2.3.1.orig/Drivers/nn/convert.c unixODBC-2.3.1/Drivers/nn/convert.c
caac36
--- unixODBC-2.3.1.orig/Drivers/nn/convert.c	2009-12-11 11:54:09.000000000 -0500
caac36
+++ unixODBC-2.3.1/Drivers/nn/convert.c	2013-03-19 13:12:19.024156752 -0400
caac36
@@ -246,7 +246,7 @@
caac36
 	if( cidx == -1 )
caac36
 		return 0;
caac36
 
caac36
-	for(i=0; i< sizeof(ctype_idx_tab); i++ )
caac36
+	for(i=0; i< sizeof(sqltype_idx_tab); i++ )
caac36
 	{
caac36
 		if( sqltype_idx_tab[i].sqltype == sqltype )
caac36
 		{
caac36
@@ -432,7 +432,7 @@
caac36
 	if( cidx == -1 )
caac36
 		return 0;
caac36
 
caac36
-	for(i=0; i< sizeof(ctype_idx_tab); i++ )
caac36
+	for(i=0; i< sizeof(sqltype_idx_tab); i++ )
caac36
 	{
caac36
 		if( sqltype_idx_tab[i].sqltype == sqltype )
caac36
 		{
caac36
diff -Naur unixODBC-2.3.1.orig/cur/SQLBindCol.c unixODBC-2.3.1/cur/SQLBindCol.c
caac36
--- unixODBC-2.3.1.orig/cur/SQLBindCol.c	2011-08-04 09:11:51.000000000 -0400
caac36
+++ unixODBC-2.3.1/cur/SQLBindCol.c	2013-03-19 13:12:19.019156876 -0400
caac36
@@ -90,7 +90,7 @@
caac36
     {
caac36
         CLBCOL *next;
caac36
         
caac36
-        if ( bcol -> local_buffer );
caac36
+        if ( bcol -> local_buffer )
caac36
         {
caac36
             free( bcol -> local_buffer );
caac36
         }
caac36
diff -Naur unixODBC-2.3.1.orig/cur/SQLGetInfo.c unixODBC-2.3.1/cur/SQLGetInfo.c
caac36
--- unixODBC-2.3.1.orig/cur/SQLGetInfo.c	2011-08-04 09:13:48.000000000 -0400
caac36
+++ unixODBC-2.3.1/cur/SQLGetInfo.c	2013-03-19 13:12:19.020156848 -0400
caac36
@@ -140,6 +140,7 @@
caac36
         value = SQL_PS_POSITIONED_DELETE |
caac36
                     SQL_PS_POSITIONED_UPDATE | 
caac36
                     SQL_PS_SELECT_FOR_UPDATE;
caac36
+        break;
caac36
 
caac36
       case SQL_ROW_UPDATES:
caac36
         cval = "Y";
caac36
diff -Naur unixODBC-2.3.1.orig/exe/isql.c unixODBC-2.3.1/exe/isql.c
caac36
--- unixODBC-2.3.1.orig/exe/isql.c	2011-09-15 12:57:14.000000000 -0400
caac36
+++ unixODBC-2.3.1/exe/isql.c	2013-03-19 13:27:09.907545641 -0400
caac36
@@ -121,6 +121,7 @@
caac36
                     break;
caac36
                 case '3':
caac36
                     version3 = 1;
caac36
+                    break;
caac36
                 case 'v':
caac36
                     bVerbose = 1;
caac36
                     break;
caac36
@@ -611,8 +612,7 @@
caac36
         {
caac36
             if ( bVerbose ) DumpODBCLog( hEnv, hDbc, hStmt );
caac36
             fprintf( stderr, "[ISQL]ERROR: Could not SQLNumResultCols\n" );
caac36
-            SQLFreeStmt( hStmt, SQL_DROP );
caac36
-            free(szSepLine);
caac36
+            break;
caac36
         }
caac36
 
caac36
         if ( cols > 0 )