[PATCH] using lp_passdb_module_path()
Andrew Bartlett
abartlet at pcug.org.au
Fri Mar 8 04:09:04 GMT 2002
Jelmer Vernooij wrote:
>
> Hi!
>
> This small patch allows lp_passdb_module_path() to have an actual use
> :) You can now do things like this in your smb.conf:
>
> passdb module path = /home/jelmer
> passdb backend = plugin:pdb_mysql.so:location
>
> of course, this also still works:
> passdb backend = plugin:/some/path/to/pdb_mysql.so:location
>
> and this even should work:
> passdb module path = /home/jelmer
> passdb backend = plugin:../user/pdb_mysql.so:location
>
> A default for passdb backend might be nice..
It defaults to smbpasswd, or did you mean a default for 'passdb module
path'?
The patch needs two things:
Style: I have a serious objection to 'tight c' where the minimum
whitespace is used. Please ensure that your code is formatted in the
same way as the rest of the samba source.
Documentation: While this parameter has been declared for a while, it
has no documentation. A patch for the docbook sources that explains
both the plugin passdb module and this new code would be appreciated.
Other than that, it looks fine - but is is really needed? We could kill
of this module entirely - why can't we just specify the full path in the
passdb backend line?
Andrew Bartlett
--
Andrew Bartlett abartlet at pcug.org.au
Manager, Authentication Subsystems, Samba Team abartlet at samba.org
Student Network Administrator, Hawker College abartlet at hawkerc.net
http://samba.org http://build.samba.org http://hawkerc.net
More information about the samba-technical
mailing list