]> Cypherpunks repositories - gostls13.git/commit
database/sql: check for arg counts after eliminating arguments
authorDaniel Theophanes <kardianos@gmail.com>
Thu, 9 Nov 2017 22:14:44 +0000 (14:14 -0800)
committerDaniel Theophanes <kardianos@gmail.com>
Sat, 18 Nov 2017 01:52:49 +0000 (01:52 +0000)
commit262141a12a6ef0d914fe7aff8b3bde45ea59abb0
tree3901e8b2f7f2dc4fe79d5b4206ce6fd863e8718b
parentce68e1b81c470ef29ece6a33980ab05e2b8bb30c
database/sql: check for arg counts after eliminating arguments

Check for the expected number of arguments in a SQL statement
after arguments are eliminated in the argument converter.

This situation was already tested for in TestNamedValueChecker.
However the test used Exec which didn't have any check for
NumInput on it at all, thus this issue was never caught.

In addition to moving the NumInput check on the Query
methods after the converter, add the NumInput check
to the Exec methods as well.

Fixes #22630

Change-Id: If45920c6e1cf70dca63822a0cedec2cdc5cc611c
Reviewed-on: https://go-review.googlesource.com/76732
Run-TryBot: Daniel Theophanes <kardianos@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
src/database/sql/sql.go