winbindd does not provide geocs to libnss_winbind in ad configuration with winbind nss info = template (default)

Bug #1388091 reported by msaxl
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
samba (Ubuntu)
Fix Released
Medium
Unassigned

Bug Description

affected with Ubuntu version 14.04.1 and 14.10.

If the user loggs in, he does not see his own full name (geocs).

it is also reporduceable by getent passwd $USER. usually there should be a field containing the users full name.

if winbindd enumerates all users or winbind rpc only = yes is set, the full name is displayed.
In the first case only until the internal cache expires.

The reason is that nss info template simply does not provide this information and resets this field, even if the values is known somewhere else in the code path.

Revision history for this message
msaxl (saxl) wrote :
Revision history for this message
msaxl (saxl) wrote :

A samba version containing this patch is built in my ppa ppa:saxl/ppa (for utopic)
There is also a version for trusty, but its also samba 4.1.11 backported from utopic

Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "Give a hint to the nss info backend if we know the full name. not every backend implements fetching its value (ex. template)" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.]

tags: added: patch
tags: added: trusty utopic
Revision history for this message
Sebastien Bacher (seb128) wrote :

Thanks, is that fixed in newer upstream version?

Mathew Hodson (mhodson)
Changed in samba (Ubuntu):
importance: Undecided → Medium
Revision history for this message
Iain Lane (laney) wrote :

Hi msaxl - thanks for your patch.

This is a patch to samba's upstream code. We shouldn't be maintaining such patches in Ubuntu (if nothing else, we don't have the expertise to review it properly). Is there any chance you could send it upstream please? Instructions appear to be here <https://wiki.samba.org/index.php/Using_Git_for_Samba_Development>.

I'm setting this bug to Incomplete - please reset to New once you have done the above and ideally after you have had it reviewed there.

Changed in samba (Ubuntu):
status: New → Incomplete
Revision history for this message
Sebastien Bacher (seb128) wrote :

unsubscribing sponsors until it's reported upstream which is the right place (doesn't make sense to have a fix in Ubuntu only/upstream knows the code/samba better to do a review)

Revision history for this message
msaxl (saxl) wrote :

in samba 4.3.3 in ubuntu xenial the problem is resolved.

The upstream bug is/was https://bugzilla.samba.org/show_bug.cgi?id=10440
As you can see there the proper fix is quite big, maybe not the best idea to backport to 4.1,
but at least in the next lts version of ubuntu it should be fixed

Revision history for this message
Robie Basak (racb) wrote :

Thanks msaxl. As you report this was fixed in Xenial, I'm marking this as Fix Released. Xenial is an LTS release.

Changed in samba (Ubuntu):
status: Incomplete → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.