Yup. I spotted that also. I will try to clean them up over the next few days.

If you have some time can you give current CVS a whirl and see if it works for 
you.

The function now looks like:

static int sql_destroy_socket(SQLSOCK *sqlsocket, SQL_CONFIG *config)
{
        rlm_sql_postgres_sock *pg_sock = sqlsocket->conn;

        free(pg_sock);
        return 0;
}

There is some NASTY code in some of the DB drivers.. Other people are welcome 
to help me clean them up :-)

Peter

On Thu July 31 2003 21:15, Alex Chen wrote:
> This not_implemented function is also present in other SQL driver modules,
> e.g. DB2.
>
> > -----Original Message-----
> > From: [EMAIL PROTECTED]
> > [mailto:[EMAIL PROTECTED] Behalf Of
> > Alan DeKok
> > Sent: Thursday, July 31, 2003 7:08 AM
> > To: [EMAIL PROTECTED]
> > Subject: Re: Newbie-ish HUP question (0.9.0) (solution - a bug?)
> >
> > Peter Nixon <[EMAIL PROTECTED]> wrote:
> > > If I have time (and no one else beats me to it) I will
> >
> > implement this
> >
> > > tonight..
> >
> >   Please also get rid of the "not_implemented" function in
> > sql_postgresql.c.  It's evil.
> >
> >   Alan DeKok.
> >
> > -
> > List info/subscribe/unsubscribe? See
> > http://www.freeradius.org/list/users.html
>
> -
> List info/subscribe/unsubscribe? See
> http://www.freeradius.org/list/users.html

-- 

Peter Nixon
http://www.peternixon.net/
PGP Key: http://www.peternixon.net/public.asc


- 
List info/subscribe/unsubscribe? See http://www.freeradius.org/list/users.html

Reply via email to