diff options
author | Paul Duncan <pabs@pablotron.org> | 2022-02-05 11:18:10 -0500 |
---|---|---|
committer | Paul Duncan <pabs@pablotron.org> | 2022-02-05 11:18:10 -0500 |
commit | 351c3531a932b30ad42d144b344edd23c1693559 (patch) | |
tree | 4a0fea83d70873e25bcb7ffa66396be6ef230b59 /dbstore | |
parent | 63a3351c9527f9f774862329b56f487339b09903 (diff) | |
download | cvez-351c3531a932b30ad42d144b344edd23c1693559.tar.bz2 cvez-351c3531a932b30ad42d144b344edd23c1693559.zip |
dbstore/dbstore.go: add comment about weird result set error handling
Diffstat (limited to 'dbstore')
-rw-r--r-- | dbstore/dbstore.go | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/dbstore/dbstore.go b/dbstore/dbstore.go index 7f22f3b..0ab6fcf 100644 --- a/dbstore/dbstore.go +++ b/dbstore/dbstore.go @@ -196,6 +196,10 @@ func (me DbStore) CpeSearch( } // close rows + // FIXME: is this correct? i am following the example from the + // database/sql documentation, but it is messy and it seems + // counterintuitive to close the row set and then do an additional + // test for iteration errors... if err = rows.Close(); err != nil { return r, err } |