Hi,

This kind-of makes sense because the pua module only deals with one table, when using a normal DB module, having set the table name once you never need to do it again.

If db_cluster() is different perhaps it'd be good to make it explicitly check for this and log an error in this case?

The pua DB is accessed in several different files so if this change is made all files will need to be checked and updated.

Regards,

Peter

On Thu, 2012-09-27 at 17:14 +0200, Daniel-Constantin Mierla wrote:
Hello,

a quick look over the pua_db.c file shows that dbf.use_table(...) is not 
used before doing the queries. That is required for db_cluster. Perhaps 
someone that is able to test the module can add the fix quickly -- I 
cannot do it for the next few days.

Cheers,
Daniel


On 9/27/12 4:58 PM, Juha Heinanen wrote:
> Peter Dunkley writes:
>
>> >From a quick look at the code this could happen if the table name or
>> length was a bad pointer.
> peter,
>
> i didn't change table names when i turned on cluster.  i just changed
> this:
>
> modparam("pua", "db_url", "cluster://pres_serv_cluster")
>
> crash happens on timer before presence server has got any requests.
>
> without db_cluster, presence works fine. have you tried presence with
> db_cluster?
>
> -- juha
>
> _______________________________________________
> sr-dev mailing list
> sr-dev@lists.sip-router.org
> http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev


-- 
Peter Dunkley
Technical Director
Crocodile RCS Ltd