Add support for pam_kwallet5.so (boo#1029942)

SUSE / pam-config

Tool to modify common PAM configuration files

GNU General Public License v2.0

9 stars 16 forks source link

Add support for pam_kwallet5.so (boo#1029942) #2

Closed Vogtinator closed 7 years ago

Vogtinator commented 7 years ago

Tried this on a second system, for some reason it breaks sudo, I'll have to look into that.

Vogtinator commented 7 years ago

Nope, that's a bug in pam-kwallet. It does a call to gcry_control(GCRYCTL_INIT_SECMEM, 32768, 0) which drops privileges and thus breaks setuid binaries.

Vogtinator commented 7 years ago

I found some more issues internal to pam-kwallet and I'll have to raise them upstream. By itself the commits here should be ok though.

Vogtinator commented 7 years ago

Ping.

Vogtinator commented 7 years ago

Ping?

Vogtinator commented 7 years ago

Apparently I'm semi-blind. Thanks for fixing and merging!