From: | Alan T DeKok <aland(at)freeradius(dot)org> |
---|---|
To: | Magnus Hagander <magnus(at)hagander(dot)net> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-bugs(at)postgresql(dot)org |
Subject: | Re: BUG #5687: RADIUS Authentication issues |
Date: | 2010-10-05 17:11:21 |
Message-ID: | 4CAB5C39.5030402@freeradius.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-bugs |
Magnus Hagander wrote:
> Actually, nevermind that one. Here's a patch I worked up from your
> description, and that turns out to be fairly similar to yours in what
> it does I think - except I'm not rearranging the code into a separate
> function. We already have a while-loop.
Thanks. The only comment I have is that the hard-code 100000 could be
USECS_PER_SEC.
> See attached context diff, and I've also included a diff without
> whitespace changes since the majority of the diff is otherwise coming
> from indenting the code one tab...
>
> (so far untested, I seem to have deleted my test-instance of the
> radius server, but I figured I should post my attempt anyway)
I can set up a test server if you want.
> Also, my patch does not change from log to warning - note that warning
> is actually *below* log when it comes to the logfile (see
> log_min_messages comments in postgresql.conf). I keep making that
> mistake myself...
OK. My only interest there was to ensure that a DoS attack wouldn't
result in the log being flooded with "invalid packet" messages.
Alan DeKok.
From | Date | Subject | |
---|---|---|---|
Next Message | Kevin Grittner | 2010-10-05 17:55:21 | Re: BUG #5687: RADIUS Authentication issues |
Previous Message | Jeff Hubbach | 2010-10-05 16:03:34 | Re: [BUGS] Mapping Hibernate boolean to smallint(Postgresql) |