aboutsummaryrefslogtreecommitdiff
path: root/dbstore
diff options
context:
space:
mode:
authorPaul Duncan <pabs@pablotron.org>2022-02-05 11:18:10 -0500
committerPaul Duncan <pabs@pablotron.org>2022-02-05 11:18:10 -0500
commit351c3531a932b30ad42d144b344edd23c1693559 (patch)
tree4a0fea83d70873e25bcb7ffa66396be6ef230b59 /dbstore
parent63a3351c9527f9f774862329b56f487339b09903 (diff)
downloadcvez-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.go4
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
}