Hi all,
I got problems after compiling OpenSER Rev. 2492. Including Module mysql, after compiling and executing openser, there occurs an error message about an undefined mysql method.
In modules/mysql/dbase.c, there are the methods line 682 db_mysql_print_values() and on line 692 db_mysql_print_set. I think they have been replaces to db/db_ut.c as db_print_values() and db_print_set respectively. Also, these new methods have a parameter val2str as last parameter. Can this be something that has been forgotten to change in Rev. 2353? If I change the functions according db_ut.h, openser runs!
Regards from hot Bratislava Sebastian
Sebastian Schumann Diploma Student Architecture and Design Department
Slovak Telekom, a. s. T-Com, Operations Unit Production and Service Division Innovations and Enterprise Solution Subunit
Address: Namestie Slobody 6, 817 62 Bratislava, Slovakia Office: Room 449, Jarabinkova 1, 821 09 Bratislava, Slovakia
+421 2 588 13332 (tel) +421 910 643010 (mobile) +49 175 1925928 (mobile)
sebastian.schumann@t-com.sk http://www.t-com.sk http://www.t-com.sk/
On Friday 20 July 2007, Schumann Sebastian wrote:
Hi all,
I got problems after compiling OpenSER Rev. 2492. Including Module mysql, after compiling and executing openser, there occurs an error message about an undefined mysql method.
In modules/mysql/dbase.c, there are the methods line 682 db_mysql_print_values() and on line 692 db_mysql_print_set. I think they have been replaces to db/db_ut.c as db_print_values() and db_print_set respectively. Also, these new methods have a parameter val2str as last parameter. Can this be something that has been forgotten to change in Rev. 2353? If I change the functions according db_ut.h, openser runs!
Hello Sebastian,
thank you for pointing this out. I forgot that the patch for the extension of the db interface was still using the old db utility function names.
Should be fixed now.
Cheers,
Henning
Dear Henning
I think you just changed the function names, not the passed parameters acc. declaration.
Compiling the files I receive the following messages: dbase.c:684: warning: passing argument 1 of 'db_print_values' from incompatible pointer type dbase.c:684: error: too few arguments to function 'db_print_values' dbase.c:694: warning: passing argument 1 of 'db_print_set' from incompatible pointer type dbase.c:694: error: too few arguments to function 'db_print_set'
The functions db_print_values(CON_CONNECTION(_h), sql_buf + off, SQL_BUF_LEN - off, _v, _n) and db_print_set(CON_CONNECTION(_h), sql_buf + off, SQL_BUF_LEN - off, _k, _v, _n) respectively in the dbase.c files require acc. db/db_ut.h a different connection parameter as 1st parameter and the val2str as last parameter.
Pls. check it.
Best regards Sebastian
________________________________
Von: Henning Westerholt [mailto:henning.westerholt@1und1.de] Gesendet: Fr 7/20/2007 17:35 An: users@openser.org Cc: Schumann Sebastian Betreff: Re: [OpenSER-Users] Possible Problem in code mysql/dbase.c, MySQL issue
On Friday 20 July 2007, Schumann Sebastian wrote:
Hi all,
I got problems after compiling OpenSER Rev. 2492. Including Module mysql, after compiling and executing openser, there occurs an error message about an undefined mysql method.
In modules/mysql/dbase.c, there are the methods line 682 db_mysql_print_values() and on line 692 db_mysql_print_set. I think they have been replaces to db/db_ut.c as db_print_values() and db_print_set respectively. Also, these new methods have a parameter val2str as last parameter. Can this be something that has been forgotten to change in Rev. 2353? If I change the functions according db_ut.h, openser runs!
Hello Sebastian,
thank you for pointing this out. I forgot that the patch for the extension of the db interface was still using the old db utility function names.
Should be fixed now.
Cheers,
Henning
On Saturday 21 July 2007, Schumann Sebastian wrote:
Dear Henning
I think you just changed the function names, not the passed parameters acc. declaration.
Hi Sebastian,
yes, you're right, thank you. One should not commit to the public svn tree five minutes before the weekend, when you're tired from the week.. ;-)
It should now really work, sorry about the problems.
Henning
-
Henning Westerholt -
Schumann Sebastian