From 13c88fe2d2cd2135db609a909c03af243b258dae Mon Sep 17 00:00:00 2001
From: Artem Goncharov <Artem.goncharov@gmail.com>
Date: Tue, 22 May 2018 17:29:00 +0200
Subject: [PATCH] fixes issue 39472: (#40379)

With python 3.6 spwd.getspnam returns PermissionError instead of
KeyError if user does not have privileges

(cherry picked from commit 0023b1ee9eca1286493c56ce5e1aa1b2272c3e39)
---
 changelogs/fragments/py36-spwd.yaml | 3 +++
 lib/ansible/modules/system/user.py  | 8 ++++++++
 2 files changed, 11 insertions(+)
 create mode 100644 changelogs/fragments/py36-spwd.yaml

diff --git a/changelogs/fragments/py36-spwd.yaml b/changelogs/fragments/py36-spwd.yaml
new file mode 100644
index 00000000000..7b867baa8e6
--- /dev/null
+++ b/changelogs/fragments/py36-spwd.yaml
@@ -0,0 +1,3 @@
+---
+bugfixes:
+- spwd - With python 3.6 spwd.getspnam returns PermissionError instead of KeyError if user does not have privileges (https://github.com/ansible/ansible/issues/39472)
diff --git a/lib/ansible/modules/system/user.py b/lib/ansible/modules/system/user.py
index f5255c526f2..4c1b7d0a3cf 100644
--- a/lib/ansible/modules/system/user.py
+++ b/lib/ansible/modules/system/user.py
@@ -218,6 +218,7 @@ EXAMPLES = '''
     expires: 1422403387
 '''
 
+import errno
 import grp
 import os
 import platform
@@ -616,6 +617,13 @@ class User(object):
                 return passwd, expires
             except KeyError:
                 return passwd, expires
+            except OSError as e:
+                # Python 3.6 raises PermissionError instead of KeyError
+                # Due to absence of PermissionError in python2.7 need to check
+                # errno
+                if e.errno in (errno.EACCES, errno.EPERM):
+                    return passwd, expires
+                raise
 
         if not self.user_exists():
             return passwd, expires