nodejs
diff --git a/Collapse file
‎src/node_sqlite.cc‎
Copy file name to clipboardExpand all lines: src/node_sqlite.cc
+3-1Lines changed: 3 additions & 1 deletion b/Collapse file
‎src/node_sqlite.cc‎
Copy file name to clipboardExpand all lines: src/node_sqlite.cc
+3-1Lines changed: 3 additions & 1 deletion
diff --git a/Collapse file
‎test/parallel/test-sqlite-statement-sync.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-sqlite-statement-sync.js
+30Lines changed: 30 additions & 0 deletions b/Collapse file
‎test/parallel/test-sqlite-statement-sync.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-sqlite-statement-sync.js
+30Lines changed: 30 additions & 0 deletions
@@ -1932,7 +1932,9 @@ bool StatementSync::BindParams(const FunctionCallbackInfo<Value>& args) {
   }
 
   for (int i = anon_start; i < args.Length(); ++i) {
-    while (sqlite3_bind_parameter_name(statement_, anon_idx) != nullptr) {
+    while (1) {
+      const char* param = sqlite3_bind_parameter_name(statement_, anon_idx);
+      if (param == nullptr || param[0] == '?') break;
       anon_idx++;
     }
 
 
@@ -240,6 +240,36 @@ suite('StatementSync.prototype.run()', () => {
       stmt.run({ k: 3, v: 30 }), { changes: 1, lastInsertRowid: 3 }
     );
   });
+
+  test('SQLite defaults unbound ?NNN parameters', (t) => {
+    const db = new DatabaseSync(nextDb());
+    t.after(() => { db.close(); });
+    const setup = db.exec(
+      'CREATE TABLE data(key INTEGER PRIMARY KEY, val INTEGER NOT NULL) STRICT;'
+    );
+    t.assert.strictEqual(setup, undefined);
+    const stmt = db.prepare('INSERT INTO data (key, val) VALUES (?1, ?3)');
+
+    t.assert.throws(() => {
+      stmt.run(1);
+    }, {
+      code: 'ERR_SQLITE_ERROR',
+      message: 'NOT NULL constraint failed: data.val',
+      errcode: 1299,
+      errstr: 'constraint failed',
+    });
+  });
+
+  test('binds ?NNN params by position', (t) => {
+    const db = new DatabaseSync(nextDb());
+    t.after(() => { db.close(); });
+    const setup = db.exec(
+      'CREATE TABLE data(key INTEGER PRIMARY KEY, val INTEGER NOT NULL) STRICT;'
+    );
+    t.assert.strictEqual(setup, undefined);
+    const stmt = db.prepare('INSERT INTO data (key, val) VALUES (?1, ?2)');
+    t.assert.deepStrictEqual(stmt.run(1, 2), { changes: 1, lastInsertRowid: 1 });
+  });
 });
 
 suite('StatementSync.prototype.sourceSQL', () => {
Original file line number	Diff line number	Diff line change
`@@ -1932,7 +1932,9 @@ bool StatementSync::BindParams(const FunctionCallbackInfo<Value>& args) {`
`1932`	`1932`	`}`
`1933`	`1933`
`1934`	`1934`	`for (int i = anon_start; i < args.Length(); ++i) {`
`1935`		`- while (sqlite3_bind_parameter_name(statement_, anon_idx) != nullptr) {`
	`1935`	`+ while (1) {`
	`1936`	`+ const char* param = sqlite3_bind_parameter_name(statement_, anon_idx);`
	`1937`	`+ if (param == nullptr \|\| param[0] == '?') break;`
`1936`	`1938`	`anon_idx++;`
`1937`	`1939`	`}`
`1938`	`1940`