Pablu/go-sqlite3
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #1424 from mattn/fix-preupdate-null-deref

Browse Source 
Check preupdate value fetch result to avoid NULL dereference
This commit is contained in:
mattn
2026-07-13 03:55:44 +00:00
committed by GitHub
parent 66371d29b6 82d5507d2c
commit a5fd1f63a0
1 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
sqlite3_opt_preupdate_hook.go
View File

@@ -59,13 +59,17 @@ func (d *SQLitePreUpdateData) row(dest []any, new bool) error {
for i := 0; i < d.Count() && i < len(dest); i++ { for i := 0; i < d.Count() && i < len(dest); i++ {
var val *C.sqlite3_value var val *C.sqlite3_value
var src any var src any
var rc C.int
// Initially I tried making this just a function pointer argument, but // Initially I tried making this just a function pointer argument, but
// it's absurdly complicated to pass C function pointers. // it's absurdly complicated to pass C function pointers.
if new { if new {
C.sqlite3_preupdate_new(d.Conn.db, C.int(i), &val) rc = C.sqlite3_preupdate_new(d.Conn.db, C.int(i), &val)
} else { } else {
C.sqlite3_preupdate_old(d.Conn.db, C.int(i), &val) rc = C.sqlite3_preupdate_old(d.Conn.db, C.int(i), &val)
}
if rc != C.SQLITE_OK {
return Error{Code: ErrNo(rc)}
} }
switch C.sqlite3_value_type(val) { switch C.sqlite3_value_type(val) {