Date: Thu, 7 May 2009 13:13:31 GMT From: Oleg Gawriloff <barzog@telecom.by> To: freebsd-gnats-submit@FreeBSD.org Subject: ports/134335: IDSIZE patch to prevent 'auth string to long' in net/gnu-radius Message-ID: <200905071313.n47DDVH4088965@www.freebsd.org> Resent-Message-ID: <200905071320.n47DK1E6017354@freefall.freebsd.org>
next in thread | raw e-mail | index | archive | help
>Number: 134335
>Category: ports
>Synopsis: IDSIZE patch to prevent 'auth string to long' in net/gnu-radius
>Confidential: no
>Severity: non-critical
>Priority: low
>Responsible: freebsd-ports-bugs
>State: open
>Quarter:
>Keywords:
>Date-Required:
>Class: sw-bug
>Submitter-Id: current-users
>Arrival-Date: Thu May 07 13:20:01 UTC 2009
>Closed-Date:
>Last-Modified:
>Originator: Oleg Gawriloff
>Release: 7.1
>Organization:
Atlant Telecom
>Environment:
FreeBSD radius_grodno.telecom.by 7.1-RELEASE-p4 FreeBSD 7.1-RELEASE-p4 #0: Mon Apr 13 15:22:34 UTC 2009 root@test.telecom.by:/usr/obj/usr/src/sys/RADIUS-GRODNO amd64
>Description:
When using gnu-radis for PPPoE authorization from Cisco BRAS it complains about:
May 18 13:09:35 penguin radiusd: Auth.error: string too long: 0/0/0/252_0001D876
Long time ago (at 2004) I convinced author to increase this string from 16 bytes to 32, but it seems in time this patch was somewhat forgot. I've already reported it upstream, but temporarily pls include it in port distribution for 1.6.1.
>How-To-Repeat:
>Fix:
Patch attached with submission follows:
--- include/radius/radutmp.h.orig 2008-12-15 10:03:24.000000000 +0200
+++ include/radius/radutmp.h 2009-05-07 10:57:42.000000000 +0300
@@ -30,7 +30,7 @@
#define P_ACCT_ENABLED 131
#define RUT_NAMESIZE 32
-#define RUT_IDSIZE 16
+#define RUT_IDSIZE 32
#define RUT_PNSIZE 24 /* Phone number size */
struct radutmp {
>Release-Note:
>Audit-Trail:
>Unformatted:
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200905071313.n47DDVH4088965>