Fix for the freeradius patch (within cdrtool) for version 1.1.7

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

Fix for the freeradius patch (within cdrtool) for version 1.1.7

Joan-2-2
When following the details on how to compile freeradius with to
support cdrtool from CDRTool/setup/radius/FreeRadius/readme.txt

The patch there applies cleanly, there's only a issue when applying
the patch I attached to the mail, because it is already fixed in 1.1.7
version. Other than that, the patch applies cleanly.

--- freeradius.orig/src/modules/rlm_sql/drivers/rlm_sql_mysql/sql_mysql.c
      2005-12-09 17:10:08.000000000 +0100
+++ freeradius.new/src/modules/rlm_sql/drivers/rlm_sql_mysql/sql_mysql.c
       2006-12-12 23:59:11.000000000 +0100
@@ -82,7 +82,7 @@
                                                    config->sql_db,
                                                    atoi(config->sql_port),
                                                    NULL,
-                                                   CLIENT_FOUND_ROWS))) {
+
CLIENT_FOUND_ROWS|CLIENT_MULTI_RESULTS))) {
                radlog(L_ERR, "rlm_sql_mysql: Couldn't connect socket
to MySQL server %s@%s:%s", config->sql_login, config->sql_server,
config->sql_db);
                radlog(L_ERR, "rlm_sql_mysql: Mysql error '%s'",
mysql_error(&mysql_sock->conn));
                mysql_sock->sock = NULL;

_______________________________________________
Users mailing list
[hidden email]
http://lists.opensips.org/cgi-bin/mailman/listinfo/users
Reply | Threaded
Open this post in threaded view
|

Re: Fix for the freeradius patch (within cdrtool) for version 1.1.7

Adrian Georgescu
Just to double check where did you get the latest sources from?

Adrian


On Jan 23, 2009, at 12:34 PM, Joan wrote:

When following the details on how to compile freeradius with to
support cdrtool from CDRTool/setup/radius/FreeRadius/readme.txt

The patch there applies cleanly, there's only a issue when applying
the patch I attached to the mail, because it is already fixed in 1.1.7
version. Other than that, the patch applies cleanly.

--- freeradius.orig/src/modules/rlm_sql/drivers/rlm_sql_mysql/sql_mysql.c
     2005-12-09 17:10:08.000000000 +0100
+++ freeradius.new/src/modules/rlm_sql/drivers/rlm_sql_mysql/sql_mysql.c
      2006-12-12 23:59:11.000000000 +0100
@@ -82,7 +82,7 @@
                                                   config->sql_db,
                                                   atoi(config->sql_port),
                                                   NULL,
-                                                   CLIENT_FOUND_ROWS))) {
+
CLIENT_FOUND_ROWS|CLIENT_MULTI_RESULTS))) {
               radlog(L_ERR, "rlm_sql_mysql: Couldn't connect socket
to MySQL server %s@%s:%s", config->sql_login, config->sql_server,
config->sql_db);
               radlog(L_ERR, "rlm_sql_mysql: Mysql error '%s'",
mysql_error(&mysql_sock->conn));
               mysql_sock->sock = NULL;

_______________________________________________
Users mailing list
[hidden email]
http://lists.opensips.org/cgi-bin/mailman/listinfo/users


_______________________________________________
Users mailing list
[hidden email]
http://lists.opensips.org/cgi-bin/mailman/listinfo/users
Reply | Threaded
Open this post in threaded view
|

Re: Fix for the freeradius patch (within cdrtool) for version 1.1.7

Joan-2-2
2009/1/23 Adrian Georgescu <[hidden email]>:
> Just to double check where did you get the latest sources from?
I got the cdrtool code from your site, the lastest packet version.
 .- wget http://download.ag-projects.com/CDRTool/cdrtool_6.7.2.tar.gz
I finally could checkout your code from trunk, and the
freeradius.patch file is the same than the one I've got in the tar.gz.

So it seems that between 1.1.3 and 1.1.7 your patch was applied to the
official freeradius sources.

>
> On Jan 23, 2009, at 12:34 PM, Joan wrote:
>
> When following the details on how to compile freeradius with to
> support cdrtool from CDRTool/setup/radius/FreeRadius/readme.txt
>
> The patch there applies cleanly, there's only a issue when applying
> the patch I attached to the mail, because it is already fixed in 1.1.7
> version. Other than that, the patch applies cleanly.
>
> --- freeradius.orig/src/modules/rlm_sql/drivers/rlm_sql_mysql/sql_mysql.c
>      2005-12-09 17:10:08.000000000 +0100
> +++ freeradius.new/src/modules/rlm_sql/drivers/rlm_sql_mysql/sql_mysql.c
>       2006-12-12 23:59:11.000000000 +0100
> @@ -82,7 +82,7 @@
>                                                    config->sql_db,
>                                                    atoi(config->sql_port),
>                                                    NULL,
> -                                                   CLIENT_FOUND_ROWS))) {
> +
> CLIENT_FOUND_ROWS|CLIENT_MULTI_RESULTS))) {
>                radlog(L_ERR, "rlm_sql_mysql: Couldn't connect socket
> to MySQL server %s@%s:%s", config->sql_login, config->sql_server,
> config->sql_db);
>                radlog(L_ERR, "rlm_sql_mysql: Mysql error '%s'",
> mysql_error(&mysql_sock->conn));
>                mysql_sock->sock = NULL;
>
> _______________________________________________
> Users mailing list
> [hidden email]
> http://lists.opensips.org/cgi-bin/mailman/listinfo/users
>
>

_______________________________________________
Users mailing list
[hidden email]
http://lists.opensips.org/cgi-bin/mailman/listinfo/users
Reply | Threaded
Open this post in threaded view
|

Re: Fix for the freeradius patch (within cdrtool) for version 1.1.7

Adrian Georgescu
OK thanks


On Jan 26, 2009, at 10:49 AM, Joan wrote:

2009/1/23 Adrian Georgescu <[hidden email]>:
Just to double check where did you get the latest sources from?
I got the cdrtool code from your site, the lastest packet version.
.- wget http://download.ag-projects.com/CDRTool/cdrtool_6.7.2.tar.gz
I finally could checkout your code from trunk, and the
freeradius.patch file is the same than the one I've got in the tar.gz.

So it seems that between 1.1.3 and 1.1.7 your patch was applied to the
official freeradius sources.


On Jan 23, 2009, at 12:34 PM, Joan wrote:

When following the details on how to compile freeradius with to
support cdrtool from CDRTool/setup/radius/FreeRadius/readme.txt

The patch there applies cleanly, there's only a issue when applying
the patch I attached to the mail, because it is already fixed in 1.1.7
version. Other than that, the patch applies cleanly.

--- freeradius.orig/src/modules/rlm_sql/drivers/rlm_sql_mysql/sql_mysql.c
    2005-12-09 17:10:08.000000000 +0100
+++ freeradius.new/src/modules/rlm_sql/drivers/rlm_sql_mysql/sql_mysql.c
     2006-12-12 23:59:11.000000000 +0100
@@ -82,7 +82,7 @@
                                                  config->sql_db,
                                                  atoi(config->sql_port),
                                                  NULL,
-                                                   CLIENT_FOUND_ROWS))) {
+
CLIENT_FOUND_ROWS|CLIENT_MULTI_RESULTS))) {
              radlog(L_ERR, "rlm_sql_mysql: Couldn't connect socket
to MySQL server %s@%s:%s", config->sql_login, config->sql_server,
config->sql_db);
              radlog(L_ERR, "rlm_sql_mysql: Mysql error '%s'",
mysql_error(&mysql_sock->conn));
              mysql_sock->sock = NULL;

_______________________________________________
Users mailing list
[hidden email]
http://lists.opensips.org/cgi-bin/mailman/listinfo/users



_______________________________________________
Users mailing list
[hidden email]
http://lists.opensips.org/cgi-bin/mailman/listinfo/users


_______________________________________________
Users mailing list
[hidden email]
http://lists.opensips.org/cgi-bin/mailman/listinfo/users