* Fixed a bug that caused a panic if no rows were returned in Insert/…

…InsertAll.
DvdSpijker · Mar 2, 2022 · be5639a · be5639a
1 parent 3c5969e
commit be5639a
Showing 1 changed file with 12 additions and 2 deletions.
diff --git a/mssql.go b/mssql.go
@@ -39,7 +39,12 @@ func (m MSSQL) Insert(ctx context.Context, query rel.Query, primaryField string,
 		rows, err       = m.DoQuery(ctx, statement, args)
 	)
 
-	defer rows.Close()
+	defer func() {
+		if rows != nil {
+			rows.Close()
+		}
+	}()
+
 	if err == nil && rows.Next() {
 		rows.Scan(&id)
 	}
@@ -62,7 +67,12 @@ func (m MSSQL) InsertAll(ctx context.Context, query rel.Query, primaryField stri
 		rows, err       = m.DoQuery(ctx, statement, args)
 	)
 
-	defer rows.Close()
+	defer func() {
+		if rows != nil {
+			rows.Close()
+		}
+	}()
+
 	if err == nil {
 		for rows.Next() {
 			var id int64