Fix rlm_sql noop for accounting start
authorDmitry Borodaenko <angdraug@debian.org>
Sat, 6 Aug 2011 17:15:59 +0000 (20:15 +0300)
committerAlan T. DeKok <aland@freeradius.org>
Sun, 18 Sep 2011 11:56:48 +0000 (13:56 +0200)
commite1c6a580085d133ddee5a4f2b26e059f4d79587c
tree6f09eb5c8c4a2b16d50df27fd85c742c2a277db8
parented3b0e32afb16e1211aa76f168df1df64760278b
Fix rlm_sql noop for accounting start

When 6ed9727 was merged, else{} in the START case got placed against the
wrong if(). Unlike STOP and ALIVE cases, in START insert comes first,
and we only care if that affects 0 rows. If insert fails and we have to
go for an update, we don't have to check for NOOP because we can assume
the insert failed due to a conflicting row already in the database.
src/modules/rlm_sql/rlm_sql.c