• Daniel Theophanes's avatar
    database/sql: check for arg counts after eliminating arguments · 262141a1
    Daniel Theophanes authored
    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: 's avatarIan Lance Taylor <iant@golang.org>
    262141a1
Name
Last commit
Last update
..
driver Loading commit data...
convert.go Loading commit data...
convert_test.go Loading commit data...
ctxutil.go Loading commit data...
doc.txt Loading commit data...
example_test.go Loading commit data...
fakedb_test.go Loading commit data...
sql.go Loading commit data...
sql_test.go Loading commit data...