> I just noticed that there is a dialback secret, which is a child tag in 
> my config file. But dialback.so entry point does this
> 
> d->secret = xmlnode_get_attrib(cfg,"secret");
> 
> looking for an attribute. Secret just feels a lot more like a child tag. 
> Changing the code to
> 
> d->secret = pstrdup(i->p,xmlnode_get_tag_data(cfg,"secret"));

Yes, I'd say that was what it was supposed to be. Although I'd suggest:

d->secret = xmlnode_get_tag_data(cfg, "secret");

No need to copy it, since the server config xmlnode never gets
deallocated.

Rob.

-- 
Robert Norris                                       GPG: 1024D/FC18E6C2
Email+Jabber: [EMAIL PROTECTED]                Web: http://cataclysm.cx/

Attachment: msg08430/pgp00000.pgp
Description: PGP signature

Reply via email to