> You seem to be missing a '-' here. But why are we setting %U in this > function at all! > > The whole passdb area has been cleaned up, and should no longer rely on > auto-magic expansion. (And if it does, things like this break pretty > badly).
It's still needed I think. The passdb doesn't get its hooks into things like 'include = smb.conf.%U', so to get that right we need to set the smb_name used for %U expansions in quite a few places.