NAS-Port is a text field. Don't cast it to integer origin/stable
authorAlan T. DeKok <aland@freeradius.org>
Thu, 11 Nov 2010 08:42:51 +0000 (09:42 +0100)
committerAlan T. DeKok <aland@freeradius.org>
Thu, 11 Nov 2010 09:34:06 +0000 (10:34 +0100)
This solves the 2^31+1 problem.  Closes bug #112

raddb/sql/postgresql/dialup.conf

index 5ff1e27..aa1907f 100644 (file)
@@ -182,7 +182,7 @@ accounting_update_query_alt = "INSERT INTO ${acct_table1} \
   ServiceType, FramedProtocol, FramedIPAddress, XAscendSessionSvrKey) \
   VALUES('%{Acct-Session-Id}', '%{Acct-Unique-Session-Id}', \
   '%{SQL-User-Name}', NULLIF('%{Realm}', ''), '%{NAS-IP-Address}', \
-  %{%{NAS-Port}:-NULL}::integer, '%{NAS-Port-Type}', \
+  %{%{NAS-Port}:-NULL}, '%{NAS-Port-Type}', \
   ('%S'::timestamp - '%{%{Acct-Delay-Time}:-0}'::interval - '%{%{Acct-Session-Time}:-0}'::interval), \
   '%{Acct-Session-Time}', '%{Acct-Authentic}', \
   (('%{%{Acct-Input-Gigawords}:-0}'::bigint << 32) + '%{%{Acct-Input-Octets}:-0}'::bigint), \
@@ -201,7 +201,7 @@ accounting_start_query = "INSERT INTO ${acct_table1} \
   '%{SQL-User-Name}', \
   NULLIF('%{Realm}', ''), \
   '%{NAS-IP-Address}', \
-  %{%{NAS-Port}:-NULL}::integer, \
+  %{%{NAS-Port}:-NULL}, \
   '%{NAS-Port-Type}', \
   ('%S'::timestamp - '%{%{Acct-Delay-Time}:-0}'::interval), \
   '%{Acct-Authentic}', \
@@ -249,7 +249,7 @@ accounting_stop_query_alt = "INSERT INTO ${acct_table2} \
   '%{SQL-User-Name}', \
   NULLIF('%{Realm}', ''), \
   '%{NAS-IP-Address}', \
-  %{%{NAS-Port}:-NULL}::integer, \
+  %{%{NAS-Port}:-NULL}, \
   '%{NAS-Port-Type}', \
   ('%S'::timestamp - '%{%{Acct-Delay-Time}:-0}'::interval - '%{%{Acct-Session-Time}:-0}'::interval), \
   ('%S'::timestamp - '%{%{Acct-Delay-Time}:-0}'::interval), \