Comment 55 for bug 216990

Revision history for this message
codeslinger (codeslinger) wrote :

so here we are, many months later, and the problem is still not solved even though it should be trivial to fix. I recently installed Ubuntu Server and the default install does not include samba which is good but my log files are being flooded with this bogus error msg which is very bad. DoS anyone?

The actual fix is totally trivial, just comment out one line in each of two different files.

What I see above looks like some pretty complicated suggested fixes being proposed to something that would benefit from being kept simple.

Why not split the config file? after all it is a ".d" directory that should be able to support the use separate config files for each app. So put all the samba related config options in a samba specific file and only install that file when samba is installed. Bottom line is that an App Specific config should not be in a "common" shared config file.

Proposed Changes:

Remove:
  From File: /etc/pam.d/common-password
  Line: password optional pam_smbpass.so nullok use_authtok use_first_pass

  From File: /etc/pam.d/common-auth
  Line: auth optional pam_smbpass.so migrate

Add:
  New File: /etc/pam.d/samba-options
  Line: password optional pam_smbpass.so nullok use_authtok use_first_pass
  Line: auth optional pam_smbpass.so migrate

End Of Problem... no code changes or new config flags needed.