Merge pull request #33 from mattn/master

otoolep · web-flow · commit ed04a340b119 · 2024-11-26T18:28:31.000-05:00
Merge upstream
diff --git a/.github/workflows/cifuzz.yaml b/.github/workflows/cifuzz.yaml
@@ -22,7 +22,7 @@ jobs:
        dry-run: false
        sanitizer: ${{ matrix.sanitizer }}
    - name: Upload Crash
-     uses: actions/upload-artifact@v1
+     uses: actions/upload-artifact@v4
      if: failure()
      with:
        name: ${{ matrix.sanitizer }}-artifacts
diff --git a/sqlite3.go b/sqlite3.go
@@ -942,6 +942,7 @@ func (c *SQLiteConn) query(ctx context.Context, query string, args []driver.Name
 		s.(*SQLiteStmt).cls = true
 		na := s.NumInput()
 		if len(args)-start < na {
+			s.Close()
 			return nil, fmt.Errorf("not enough args to execute query: want %d got %d", na, len(args)-start)
 		}
 		// consume the number of arguments used in the current

Original file line number	Diff line number	Diff line change
`@@ -942,6 +942,7 @@ func (c *SQLiteConn) query(ctx context.Context, query string, args []driver.Name`
`942`	`942`	`s.(*SQLiteStmt).cls = true`
`943`	`943`	`na := s.NumInput()`
`944`	`944`	`if len(args)-start < na {`
	`945`	`+ s.Close()`
`945`	`946`	`return nil, fmt.Errorf("not enough args to execute query: want %d got %d", na, len(args)-start)`
`946`	`947`	`}`
`947`	`948`	`// consume the number of arguments used in the current`